diff options
| author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2024-09-09 20:56:19 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2024-09-09 21:08:43 +0100 |
| commit | 63a68ee1c27f5d1a17b78a2c937b86b0fb1fd99a (patch) | |
| tree | d9adecc723586840931f4525fc7569d6c528f7d9 /drivers/regulator | |
| parent | 48cc042bd68e0225f1e6b137452e3d867e7c0942 (diff) | |
| download | lwn-63a68ee1c27f5d1a17b78a2c937b86b0fb1fd99a.tar.gz lwn-63a68ee1c27f5d1a17b78a2c937b86b0fb1fd99a.zip | |
regulator: tps6287x: Constify struct regulator_desc
'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>
Diffstat (limited to 'drivers/regulator')
| -rw-r--r-- | drivers/regulator/tps6287x-regulator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/tps6287x-regulator.c b/drivers/regulator/tps6287x-regulator.c index 7a0551f0c8c0..97f5ce138548 100644 --- a/drivers/regulator/tps6287x-regulator.c +++ b/drivers/regulator/tps6287x-regulator.c @@ -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, |