Commit 130dbe04 authored by Simon Trimmer's avatar Simon Trimmer Committed by Mark Brown

ASoC: wm_adsp: mark more data structures with the const qualifier

The callback structures and memory region type table can be marked as
const as they will not change during use.

Fix checkpatch warning against wm_adsp_find_region function by moving
const keyword to form the 'static const struct' pattern.
Signed-off-by: default avatarSimon Trimmer <simont@opensource.cirrus.com>
Acked-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20210511171459.270169-1-simont@opensource.cirrus.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 4ac9b48a
...@@ -303,9 +303,9 @@ ...@@ -303,9 +303,9 @@
#define HALO_MPU_VIO_ERR_SRC_MASK 0x00007fff #define HALO_MPU_VIO_ERR_SRC_MASK 0x00007fff
#define HALO_MPU_VIO_ERR_SRC_SHIFT 0 #define HALO_MPU_VIO_ERR_SRC_SHIFT 0
static struct wm_adsp_ops wm_adsp1_ops; static const struct wm_adsp_ops wm_adsp1_ops;
static struct wm_adsp_ops wm_adsp2_ops[]; static const struct wm_adsp_ops wm_adsp2_ops[];
static struct wm_adsp_ops wm_halo_ops; static const struct wm_adsp_ops wm_halo_ops;
struct wm_adsp_buf { struct wm_adsp_buf {
struct list_head list; struct list_head list;
...@@ -824,7 +824,7 @@ const struct soc_enum wm_adsp_fw_enum[] = { ...@@ -824,7 +824,7 @@ const struct soc_enum wm_adsp_fw_enum[] = {
}; };
EXPORT_SYMBOL_GPL(wm_adsp_fw_enum); EXPORT_SYMBOL_GPL(wm_adsp_fw_enum);
static struct wm_adsp_region const *wm_adsp_find_region(struct wm_adsp *dsp, static const struct wm_adsp_region *wm_adsp_find_region(struct wm_adsp *dsp,
int type) int type)
{ {
int i; int i;
...@@ -2240,7 +2240,7 @@ static void wmfw_v3_parse_id_header(struct wm_adsp *dsp, ...@@ -2240,7 +2240,7 @@ static void wmfw_v3_parse_id_header(struct wm_adsp *dsp,
} }
static int wm_adsp_create_regions(struct wm_adsp *dsp, __be32 id, int nregions, static int wm_adsp_create_regions(struct wm_adsp *dsp, __be32 id, int nregions,
int *type, __be32 *base) const int *type, __be32 *base)
{ {
struct wm_adsp_alg_region *alg_region; struct wm_adsp_alg_region *alg_region;
int i; int i;
...@@ -2487,7 +2487,7 @@ static int wm_adsp2_setup_algs(struct wm_adsp *dsp) ...@@ -2487,7 +2487,7 @@ static int wm_adsp2_setup_algs(struct wm_adsp *dsp)
static int wm_halo_create_regions(struct wm_adsp *dsp, __be32 id, static int wm_halo_create_regions(struct wm_adsp *dsp, __be32 id,
__be32 xm_base, __be32 ym_base) __be32 xm_base, __be32 ym_base)
{ {
int types[] = { static const int types[] = {
WMFW_ADSP2_XM, WMFW_HALO_XM_PACKED, WMFW_ADSP2_XM, WMFW_HALO_XM_PACKED,
WMFW_ADSP2_YM, WMFW_HALO_YM_PACKED WMFW_ADSP2_YM, WMFW_HALO_YM_PACKED
}; };
...@@ -4500,13 +4500,13 @@ irqreturn_t wm_halo_wdt_expire(int irq, void *data) ...@@ -4500,13 +4500,13 @@ irqreturn_t wm_halo_wdt_expire(int irq, void *data)
} }
EXPORT_SYMBOL_GPL(wm_halo_wdt_expire); EXPORT_SYMBOL_GPL(wm_halo_wdt_expire);
static struct wm_adsp_ops wm_adsp1_ops = { static const struct wm_adsp_ops wm_adsp1_ops = {
.validate_version = wm_adsp_validate_version, .validate_version = wm_adsp_validate_version,
.parse_sizes = wm_adsp1_parse_sizes, .parse_sizes = wm_adsp1_parse_sizes,
.region_to_reg = wm_adsp_region_to_reg, .region_to_reg = wm_adsp_region_to_reg,
}; };
static struct wm_adsp_ops wm_adsp2_ops[] = { static const struct wm_adsp_ops wm_adsp2_ops[] = {
{ {
.sys_config_size = sizeof(struct wm_adsp_system_config_xm_hdr), .sys_config_size = sizeof(struct wm_adsp_system_config_xm_hdr),
.parse_sizes = wm_adsp2_parse_sizes, .parse_sizes = wm_adsp2_parse_sizes,
...@@ -4567,7 +4567,7 @@ static struct wm_adsp_ops wm_adsp2_ops[] = { ...@@ -4567,7 +4567,7 @@ static struct wm_adsp_ops wm_adsp2_ops[] = {
}, },
}; };
static struct wm_adsp_ops wm_halo_ops = { static const struct wm_adsp_ops wm_halo_ops = {
.sys_config_size = sizeof(struct wm_halo_system_config_xm_hdr), .sys_config_size = sizeof(struct wm_halo_system_config_xm_hdr),
.parse_sizes = wm_adsp2_parse_sizes, .parse_sizes = wm_adsp2_parse_sizes,
.validate_version = wm_halo_validate_version, .validate_version = wm_halo_validate_version,
......
...@@ -64,7 +64,7 @@ struct wm_adsp { ...@@ -64,7 +64,7 @@ struct wm_adsp {
struct regmap *regmap; struct regmap *regmap;
struct snd_soc_component *component; struct snd_soc_component *component;
struct wm_adsp_ops *ops; const struct wm_adsp_ops *ops;
unsigned int base; unsigned int base;
unsigned int base_sysinfo; unsigned int base_sysinfo;
......
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