Unverified Commit 72f2746c authored by Rikard Falkeborn's avatar Rikard Falkeborn Committed by Mark Brown
Browse files

regulator: pca9450: Constify static regulator_ops 



The only usages of these is 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: default avatarRikard Falkeborn <rikard.falkeborn@gmail.com>
Link: https://lore.kernel.org/r/20200913084114.8851-5-rikard.falkeborn@gmail.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 9032693e

drivers/regulator/pca9450-regulator.c

+3 −3
Original line number Diff line number Diff line
@@ -90,7 +90,7 @@ static int pca9450_dvs_set_ramp_delay(struct regulator_dev *rdev, 
				  BUCK1_RAMP_MASK, ramp_value << 6);

}



static struct regulator_ops pca9450_dvs_buck_regulator_ops = {

static const struct regulator_ops pca9450_dvs_buck_regulator_ops = {

	.enable = regulator_enable_regmap,

	.disable = regulator_disable_regmap,

	.is_enabled = regulator_is_enabled_regmap,
@@ -101,7 +101,7 @@ static struct regulator_ops pca9450_dvs_buck_regulator_ops = { 
	.set_ramp_delay = pca9450_dvs_set_ramp_delay,

};



static struct regulator_ops pca9450_buck_regulator_ops = {

static const struct regulator_ops pca9450_buck_regulator_ops = {

	.enable = regulator_enable_regmap,

	.disable = regulator_disable_regmap,

	.is_enabled = regulator_is_enabled_regmap,
@@ -111,7 +111,7 @@ static struct regulator_ops pca9450_buck_regulator_ops = { 
	.set_voltage_time_sel = regulator_set_voltage_time_sel,

};



static struct regulator_ops pca9450_ldo_regulator_ops = {

static const struct regulator_ops pca9450_ldo_regulator_ops = {

	.enable = regulator_enable_regmap,

	.disable = regulator_disable_regmap,

	.is_enabled = regulator_is_enabled_regmap,