]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
regulator: tps6287x: Constify struct regulator_desc
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 9 Sep 2024 18:56:19 +0000 (20:56 +0200)
committerMark Brown <broonie@kernel.org>
Mon, 9 Sep 2024 20:08:43 +0000 (21:08 +0100)
'struct regulator_desc' is not modified in this driver.

Constifying this structure moves some data to a read-only section, so
increases overall security, especially when the structure holds some
function pointers.

On a x86_64, with allmodconfig:
Before:
======
   text    data     bss     dec     hex filename
   4974     736      16    5726    165e drivers/regulator/tps6287x-regulator.o

After:
=====
   text    data     bss     dec     hex filename
   5294     416      16    5726    165e drivers/regulator/tps6287x-regulator.o

--
Compile tested only

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://patch.msgid.link/7727e493490d37775a653905dfe0cc1d8478f8e0.1725908163.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/tps6287x-regulator.c

index 7a0551f0c8c0aa414abc03313a40e0d17871a280..97f5ce138548ed73166a2e4d1f701a1b6ee60ac0 100644 (file)
@@ -103,7 +103,7 @@ static const struct regulator_ops tps6287x_regulator_ops = {
        .set_ramp_delay = regulator_set_ramp_delay_regmap,
 };
 
-static struct regulator_desc tps6287x_reg = {
+static const struct regulator_desc tps6287x_reg = {
        .name = "tps6287x",
        .owner = THIS_MODULE,
        .ops = &tps6287x_regulator_ops,