Commit c0948245 authored by Takashi Iwai's avatar Takashi Iwai

ALSA: vx: More constifications

Apply const prefix to every possible place: the static tables for DSP
commands, the string tables, and register/offset tables.

Just for minor optimization and no functional changes.

Link: https://lore.kernel.org/r/20200105144823.29547-8-tiwai@suse.deSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 731922a5
...@@ -15,7 +15,7 @@ ...@@ -15,7 +15,7 @@
/* /*
* Array of DSP commands * Array of DSP commands
*/ */
static struct vx_cmd_info vx_dsp_cmds[] = { static const struct vx_cmd_info vx_dsp_cmds[] = {
[CMD_VERSION] = { 0x010000, 2, RMH_SSIZE_FIXED, 1 }, [CMD_VERSION] = { 0x010000, 2, RMH_SSIZE_FIXED, 1 },
[CMD_SUPPORTED] = { 0x020000, 1, RMH_SSIZE_FIXED, 2 }, [CMD_SUPPORTED] = { 0x020000, 1, RMH_SSIZE_FIXED, 2 },
[CMD_TEST_IT] = { 0x040000, 1, RMH_SSIZE_FIXED, 1 }, [CMD_TEST_IT] = { 0x040000, 1, RMH_SSIZE_FIXED, 1 },
......
...@@ -39,7 +39,7 @@ MODULE_LICENSE("GPL"); ...@@ -39,7 +39,7 @@ MODULE_LICENSE("GPL");
int snd_vx_check_reg_bit(struct vx_core *chip, int reg, int mask, int bit, int time) int snd_vx_check_reg_bit(struct vx_core *chip, int reg, int mask, int bit, int time)
{ {
unsigned long end_time = jiffies + (time * HZ + 999) / 1000; unsigned long end_time = jiffies + (time * HZ + 999) / 1000;
static char *reg_names[VX_REG_MAX] = { static const char * const reg_names[VX_REG_MAX] = {
"ICR", "CVR", "ISR", "IVR", "RXH", "RXM", "RXL", "ICR", "CVR", "ISR", "IVR", "RXH", "RXM", "RXL",
"DMA", "CDSP", "RFREQ", "RUER/V2", "DATA", "MEMIRQ", "DMA", "CDSP", "RFREQ", "RUER/V2", "DATA", "MEMIRQ",
"ACQ", "BIT0", "BIT1", "MIC0", "MIC1", "MIC2", "ACQ", "BIT0", "BIT1", "MIC0", "MIC1", "MIC2",
...@@ -588,11 +588,11 @@ static void vx_reset_board(struct vx_core *chip, int cold_reset) ...@@ -588,11 +588,11 @@ static void vx_reset_board(struct vx_core *chip, int cold_reset)
static void vx_proc_read(struct snd_info_entry *entry, struct snd_info_buffer *buffer) static void vx_proc_read(struct snd_info_entry *entry, struct snd_info_buffer *buffer)
{ {
struct vx_core *chip = entry->private_data; struct vx_core *chip = entry->private_data;
static char *audio_src_vxp[] = { "Line", "Mic", "Digital" }; static const char * const audio_src_vxp[] = { "Line", "Mic", "Digital" };
static char *audio_src_vx2[] = { "Analog", "Analog", "Digital" }; static const char * const audio_src_vx2[] = { "Analog", "Analog", "Digital" };
static char *clock_mode[] = { "Auto", "Internal", "External" }; static const char * const clock_mode[] = { "Auto", "Internal", "External" };
static char *clock_src[] = { "Internal", "External" }; static const char * const clock_src[] = { "Internal", "External" };
static char *uer_type[] = { "Consumer", "Professional", "Not Present" }; static const char * const uer_type[] = { "Consumer", "Professional", "Not Present" };
snd_iprintf(buffer, "%s\n", chip->card->longname); snd_iprintf(buffer, "%s\n", chip->card->longname);
snd_iprintf(buffer, "Xilinx Firmware: %s\n", snd_iprintf(buffer, "Xilinx Firmware: %s\n",
......
...@@ -32,7 +32,7 @@ MODULE_FIRMWARE("vx/l_1_vp4.d56"); ...@@ -32,7 +32,7 @@ MODULE_FIRMWARE("vx/l_1_vp4.d56");
int snd_vx_setup_firmware(struct vx_core *chip) int snd_vx_setup_firmware(struct vx_core *chip)
{ {
static char *fw_files[VX_TYPE_NUMS][4] = { static const char * const fw_files[VX_TYPE_NUMS][4] = {
[VX_TYPE_BOARD] = { [VX_TYPE_BOARD] = {
NULL, "x1_1_vx2.xlx", "bd56002.boot", "l_1_vx2.d56", NULL, "x1_1_vx2.xlx", "bd56002.boot", "l_1_vx2.d56",
}, },
......
...@@ -961,7 +961,7 @@ int snd_vx_mixer_new(struct vx_core *chip) ...@@ -961,7 +961,7 @@ int snd_vx_mixer_new(struct vx_core *chip)
return err; return err;
/* VU, peak, saturation meters */ /* VU, peak, saturation meters */
for (c = 0; c < 2; c++) { for (c = 0; c < 2; c++) {
static char *dir[2] = { "Output", "Input" }; static const char * const dir[2] = { "Output", "Input" };
for (i = 0; i < chip->hw->num_ins; i++) { for (i = 0; i < chip->hw->num_ins; i++) {
int val = (i * 2) | (c << 8); int val = (i * 2) | (c << 8);
if (c == 1) { if (c == 1) {
......
...@@ -19,7 +19,7 @@ ...@@ -19,7 +19,7 @@
#include "vx222.h" #include "vx222.h"
static int vx2_reg_offset[VX_REG_MAX] = { static const int vx2_reg_offset[VX_REG_MAX] = {
[VX_ICR] = 0x00, [VX_ICR] = 0x00,
[VX_CVR] = 0x04, [VX_CVR] = 0x04,
[VX_ISR] = 0x08, [VX_ISR] = 0x08,
...@@ -45,7 +45,7 @@ static int vx2_reg_offset[VX_REG_MAX] = { ...@@ -45,7 +45,7 @@ static int vx2_reg_offset[VX_REG_MAX] = {
[VX_GPIOC] = 0x54, // VX_GPIOC (new with PLX9030) [VX_GPIOC] = 0x54, // VX_GPIOC (new with PLX9030)
}; };
static int vx2_reg_index[VX_REG_MAX] = { static const int vx2_reg_index[VX_REG_MAX] = {
[VX_ICR] = 1, [VX_ICR] = 1,
[VX_CVR] = 1, [VX_CVR] = 1,
[VX_ISR] = 1, [VX_ISR] = 1,
......
...@@ -15,7 +15,7 @@ ...@@ -15,7 +15,7 @@
#include "vxpocket.h" #include "vxpocket.h"
static int vxp_reg_offset[VX_REG_MAX] = { static const int vxp_reg_offset[VX_REG_MAX] = {
[VX_ICR] = 0x00, // ICR [VX_ICR] = 0x00, // ICR
[VX_CVR] = 0x01, // CVR [VX_CVR] = 0x01, // CVR
[VX_ISR] = 0x02, // ISR [VX_ISR] = 0x02, // ISR
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment