diff options
author | Bhumika Goyal <bhumirks@gmail.com> | 2017-08-03 21:30:19 +0530 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-08-10 16:10:50 +0100 |
commit | a180ba45b1cf630b3bd5912ce235b2ee16606b8e (patch) | |
tree | 7357cb3beb7fa05c24bc9a472f34c5e8a8daabb1 /sound/soc/codecs/rt5616.c | |
parent | 5a36aedb087ba16a2690b377fa25693dc7e52d34 (diff) | |
download | linux-a180ba45b1cf630b3bd5912ce235b2ee16606b8e.tar.bz2 |
ASoC: codecs: add const to snd_soc_codec_driver structures
Declare snd_soc_codec_driver structures as const as they are only passed
as an argument to the function snd_soc_register_codec. This argument is
of type const, so declare the structures with this property as const.
In file codecs/sn95031.c, snd_soc_codec_driver structure is also used in
a copy operation along with getting passed to snd_soc_register_codec.
So, it can be made const too.
Done using Coccinelle:
@match disable optional_qualifier@
identifier s;
position p;
@@
static struct snd_soc_codec_driver s@p={...};
@good1@
identifier match.s;
position p;
@@
snd_soc_register_codec(...,&s@p,...)
@bad@
identifier match.s;
position p!={match.p,good1.p};
@@
s@p
@depends on !bad disable optional_qualifier@
identifier match.s;
@@
static
+const
struct snd_soc_codec_driver s={...};
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/codecs/rt5616.c')
-rw-r--r-- | sound/soc/codecs/rt5616.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/codecs/rt5616.c b/sound/soc/codecs/rt5616.c index 7d6e0823f98f..3c5f555ae6d0 100644 --- a/sound/soc/codecs/rt5616.c +++ b/sound/soc/codecs/rt5616.c @@ -1294,7 +1294,7 @@ static struct snd_soc_dai_driver rt5616_dai[] = { }, }; -static struct snd_soc_codec_driver soc_codec_dev_rt5616 = { +static const struct snd_soc_codec_driver soc_codec_dev_rt5616 = { .probe = rt5616_probe, .suspend = rt5616_suspend, .resume = rt5616_resume, |