diff options
author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2024-08-06 15:52:24 +0200 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2024-08-13 12:41:40 +0100 |
commit | ab73c7c0e5800a44690023cfdfeac72d3b6b95e8 (patch) | |
tree | 77983986dae2a2f691089fda9b51c9feb084de63 | |
parent | 6024b86b4a618b6973cf6fc5ed3fa21280e395b9 (diff) | |
download | lwn-ab73c7c0e5800a44690023cfdfeac72d3b6b95e8.tar.gz lwn-ab73c7c0e5800a44690023cfdfeac72d3b6b95e8.zip |
ASoC: rt1318: Constify struct reg_sequence
'struct reg_sequence' is not modified in this driver.
Constifying this structure moves some data to a read-only section, so
increase overall security.
While at it, remove rt1318_INIT_REG_LEN which is ununsed.
On a x86_64, with allmodconfig:
Before:
======
text data bss dec hex filename
22062 4859 32 26953 6949 sound/soc/codecs/rt1318.o
After:
=====
text data bss dec hex filename
24742 2171 32 26945 6941 sound/soc/codecs/rt1318.o
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://patch.msgid.link/96561dd2962d4312eb0e68ab850027f44350d070.1722952334.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | sound/soc/codecs/rt1318.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sound/soc/codecs/rt1318.c b/sound/soc/codecs/rt1318.c index 83b29b441be9..e12b1e96a53a 100644 --- a/sound/soc/codecs/rt1318.c +++ b/sound/soc/codecs/rt1318.c @@ -30,7 +30,7 @@ #include "rt1318.h" -static struct reg_sequence init_list[] = { +static const struct reg_sequence init_list[] = { { 0x0000C000, 0x01}, { 0x0000F20D, 0x00}, { 0x0000F212, 0x3E}, @@ -254,7 +254,6 @@ static struct reg_sequence init_list[] = { { 0x0000C320, 0x20}, { 0x0000C203, 0x9C}, }; -#define rt1318_INIT_REG_LEN ARRAY_SIZE(init_list) static const struct reg_default rt1318_reg[] = { { 0xc000, 0x00 }, |