regulator: tps65086: Constify static regulator_ops
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Sat, 29 Aug 2020 22:11:00 +0000 (00:11 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 1 Sep 2020 14:48:54 +0000 (15:48 +0100)
The only usage of reg_ops and switch_ops is to assign their addresses to
the ops field in the regulator_desc struct, which is a const pointer.
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/20200829221104.20870-5-rikard.falkeborn@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/tps65086-regulator.c

index 23528475a962b2a9d309ae0fb2b283dc8f3ea846..070c956216b0319ed47d381b64201af85b0695f0 100644 (file)
@@ -101,7 +101,7 @@ static const struct linear_range tps65086_ldoa23_ranges[] = {
 };
 
 /* Operations permitted on regulators */
-static struct regulator_ops reg_ops = {
+static const struct regulator_ops reg_ops = {
        .enable                 = regulator_enable_regmap,
        .disable                = regulator_disable_regmap,
        .is_enabled             = regulator_is_enabled_regmap,
@@ -112,7 +112,7 @@ static struct regulator_ops reg_ops = {
 };
 
 /* Operations permitted on load switches */
-static struct regulator_ops switch_ops = {
+static const struct regulator_ops switch_ops = {
        .enable                 = regulator_enable_regmap,
        .disable                = regulator_disable_regmap,
        .is_enabled             = regulator_is_enabled_regmap,