regulator: tps65090: constify static regulator_ops (7d844ac3) · Commits · 戴 / test

The only usages of these are to assign their address 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-6-rikard.falkeborn@gmail.com Signed-off-by:

Mark Brown <broonie@kernel.org>

drivers/regulator/tps65090-regulator.c

+4 −4

Original line number	Diff line number	Diff line
		@@ -47,7 +47,7 @@ struct tps65090_regulator {
		int overcurrent_wait;
		};

		static struct regulator_ops tps65090_ext_control_ops = {
		static const struct regulator_ops tps65090_ext_control_ops = {
		};

		/**
		@@ -167,19 +167,19 @@ err:
		return ret;
		}

		static struct regulator_ops tps65090_reg_control_ops = {
		static const struct regulator_ops tps65090_reg_control_ops = {
		.enable = regulator_enable_regmap,
		.disable = regulator_disable_regmap,
		.is_enabled = regulator_is_enabled_regmap,
		};

		static struct regulator_ops tps65090_fet_control_ops = {
		static const struct regulator_ops tps65090_fet_control_ops = {
		.enable = tps65090_fet_enable,
		.disable = regulator_disable_regmap,
		.is_enabled = regulator_is_enabled_regmap,
		};

		static struct regulator_ops tps65090_ldo_ops = {
		static const struct regulator_ops tps65090_ldo_ops = {
		};

		#define tps65090_REG_DESC(_id, _sname, _en_reg, _en_bits, _nvolt, _volt, _ops) \

Admin message