ASoC: mediatek: Constify static snd_soc_ops
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Tue, 12 Oct 2021 20:55:21 +0000 (22:55 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 13 Oct 2021 11:59:28 +0000 (12:59 +0100)
These are only assigned to the ops field in the snd_soc_dai_link struct
which is a pointer to const struct snd_soc_ops. Make them const to allow
the compiler to put them in read-only memory.

Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Link: https://lore.kernel.org/r/20211012205521.14098-1-rikard.falkeborn@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/mediatek/mt2701/mt2701-cs42448.c
sound/soc/mediatek/mt2701/mt2701-wm8960.c
sound/soc/mediatek/mt8183/mt8183-mt6358-ts3a227-max98357.c

index 44a8d5c..d9fd6eb 100644 (file)
@@ -146,7 +146,7 @@ static int mt2701_cs42448_be_ops_hw_params(struct snd_pcm_substream *substream,
        return 0;
 }
 
-static struct snd_soc_ops mt2701_cs42448_be_ops = {
+static const struct snd_soc_ops mt2701_cs42448_be_ops = {
        .hw_params = mt2701_cs42448_be_ops_hw_params
 };
 
index 414e422..f56de1b 100644 (file)
@@ -40,7 +40,7 @@ static int mt2701_wm8960_be_ops_hw_params(struct snd_pcm_substream *substream,
        return 0;
 }
 
-static struct snd_soc_ops mt2701_wm8960_be_ops = {
+static const struct snd_soc_ops mt2701_wm8960_be_ops = {
        .hw_params = mt2701_wm8960_be_ops_hw_params
 };
 
index 94dcbd3..aeb1af8 100644 (file)
@@ -335,7 +335,7 @@ static void mt8183_mt6358_tdm_shutdown(struct snd_pcm_substream *substream)
                        __func__, ret);
 }
 
-static struct snd_soc_ops mt8183_mt6358_tdm_ops = {
+static const struct snd_soc_ops mt8183_mt6358_tdm_ops = {
        .startup = mt8183_mt6358_tdm_startup,
        .shutdown = mt8183_mt6358_tdm_shutdown,
 };