diff options
| author | Axel Lin <axel.lin@ingics.com> | 2015-07-30 09:06:43 -0400 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2015-08-03 12:22:37 -0400 |
| commit | 08b472f7b602c60c69f278ba1e15535f5c86ee24 (patch) | |
| tree | 5536bd93028056e8c4437204890e84134316887f | |
| parent | 53e381627d34c4c7a3ea00c041c38f447828f755 (diff) | |
regulator: mt6311: Trivial clean up
Make mt6311_buck_ops, mt6311_ldo_ops and mt6311_regulators const and remove
unneeded error variable in mt6311_i2c_probe().
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/regulator/mt6311-regulator.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/regulator/mt6311-regulator.c b/drivers/regulator/mt6311-regulator.c index b140da04a657..2c2c85bf08c5 100644 --- a/drivers/regulator/mt6311-regulator.c +++ b/drivers/regulator/mt6311-regulator.c | |||
| @@ -41,7 +41,7 @@ static const struct regulator_linear_range buck_volt_range[] = { | |||
| 41 | REGULATOR_LINEAR_RANGE(MT6311_MIN_UV, 0, 0x7f, MT6311_STEP_UV), | 41 | REGULATOR_LINEAR_RANGE(MT6311_MIN_UV, 0, 0x7f, MT6311_STEP_UV), |
| 42 | }; | 42 | }; |
| 43 | 43 | ||
| 44 | static struct regulator_ops mt6311_buck_ops = { | 44 | static const struct regulator_ops mt6311_buck_ops = { |
| 45 | .list_voltage = regulator_list_voltage_linear_range, | 45 | .list_voltage = regulator_list_voltage_linear_range, |
| 46 | .map_voltage = regulator_map_voltage_linear_range, | 46 | .map_voltage = regulator_map_voltage_linear_range, |
| 47 | .set_voltage_sel = regulator_set_voltage_sel_regmap, | 47 | .set_voltage_sel = regulator_set_voltage_sel_regmap, |
| @@ -52,7 +52,7 @@ static struct regulator_ops mt6311_buck_ops = { | |||
| 52 | .is_enabled = regulator_is_enabled_regmap, | 52 | .is_enabled = regulator_is_enabled_regmap, |
| 53 | }; | 53 | }; |
| 54 | 54 | ||
| 55 | static struct regulator_ops mt6311_ldo_ops = { | 55 | static const struct regulator_ops mt6311_ldo_ops = { |
| 56 | .enable = regulator_enable_regmap, | 56 | .enable = regulator_enable_regmap, |
| 57 | .disable = regulator_disable_regmap, | 57 | .disable = regulator_disable_regmap, |
| 58 | .is_enabled = regulator_is_enabled_regmap, | 58 | .is_enabled = regulator_is_enabled_regmap, |
| @@ -91,7 +91,7 @@ static struct regulator_ops mt6311_ldo_ops = { | |||
| 91 | .enable_mask = MT6311_PMIC_RG_VBIASN_EN_MASK,\ | 91 | .enable_mask = MT6311_PMIC_RG_VBIASN_EN_MASK,\ |
| 92 | } | 92 | } |
| 93 | 93 | ||
| 94 | static struct regulator_desc mt6311_regulators[] = { | 94 | static const struct regulator_desc mt6311_regulators[] = { |
| 95 | MT6311_BUCK(VDVFS), | 95 | MT6311_BUCK(VDVFS), |
| 96 | MT6311_LDO(VBIASN), | 96 | MT6311_LDO(VBIASN), |
| 97 | }; | 97 | }; |
| @@ -105,15 +105,15 @@ static int mt6311_i2c_probe(struct i2c_client *i2c, | |||
| 105 | struct regulator_config config = { }; | 105 | struct regulator_config config = { }; |
| 106 | struct regulator_dev *rdev; | 106 | struct regulator_dev *rdev; |
| 107 | struct regmap *regmap; | 107 | struct regmap *regmap; |
| 108 | int error, i, ret; | 108 | int i, ret; |
| 109 | unsigned int data; | 109 | unsigned int data; |
| 110 | 110 | ||
| 111 | regmap = devm_regmap_init_i2c(i2c, &mt6311_regmap_config); | 111 | regmap = devm_regmap_init_i2c(i2c, &mt6311_regmap_config); |
| 112 | if (IS_ERR(regmap)) { | 112 | if (IS_ERR(regmap)) { |
| 113 | error = PTR_ERR(regmap); | 113 | ret = PTR_ERR(regmap); |
| 114 | dev_err(&i2c->dev, "Failed to allocate register map: %d\n", | 114 | dev_err(&i2c->dev, "Failed to allocate register map: %d\n", |
| 115 | error); | 115 | ret); |
| 116 | return error; | 116 | return ret; |
| 117 | } | 117 | } |
| 118 | 118 | ||
| 119 | ret = regmap_read(regmap, MT6311_SWCID, &data); | 119 | ret = regmap_read(regmap, MT6311_SWCID, &data); |
