]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
ASoC: Intel: avs: Constify struct snd_soc_tplg_ops
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 13 May 2024 17:37:21 +0000 (19:37 +0200)
committerMark Brown <broonie@kernel.org>
Mon, 27 May 2024 00:31:16 +0000 (01:31 +0100)
Constifying "struct snd_soc_tplg_ops" moves some data to a read-only
section, so increase overall security.

On a x86_64, with allmodconfig:
Before:
   text    data     bss     dec     hex filename
  28046     794       0   28840    70a8 sound/soc/intel/avs/topology.o

After:
   text    data     bss     dec     hex filename
  28206     614       0   28820    7094 sound/soc/intel/avs/topology.o

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
Link: https://msgid.link/r/01de0157088de0475970c79738284d6c86d702fe.1715526069.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/avs/topology.c

index 02bae207f6ece33823635197635f412aa8ca1147..35381a835c931a73917c20b1b158c6fd3ceb4207 100644 (file)
@@ -1889,7 +1889,7 @@ avs_control_load(struct snd_soc_component *comp, int index, struct snd_kcontrol_
        return 0;
 }
 
-static struct snd_soc_tplg_ops avs_tplg_ops = {
+static const struct snd_soc_tplg_ops avs_tplg_ops = {
        .io_ops                 = avs_control_ops,
        .io_ops_count           = ARRAY_SIZE(avs_control_ops),
        .control_load           = avs_control_load,