diff options
author | Julia Lawall <julia.lawall@lip6.fr> | 2015-06-16 06:45:14 -0700 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-06-16 15:58:53 +0100 |
commit | 0f94bffad401faa045c2062f1652d21e3e89f439 (patch) | |
tree | 7420b2da5ae1372389c8339f0b221703871486d5 /drivers/regulator | |
parent | e92a4047419c805d08ad136fbc72368249d9f091 (diff) | |
download | op-kernel-dev-0f94bffad401faa045c2062f1652d21e3e89f439.zip op-kernel-dev-0f94bffad401faa045c2062f1652d21e3e89f439.tar.gz |
regulator: fix simple_return.cocci warnings
Simplify a trivial if-return sequence and combine with a
preceding function call.
Generated by: scripts/coccinelle/misc/simple_return.cocci
CC: Mark Brown <broonie@kernel.org>
Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator')
-rw-r--r-- | drivers/regulator/qcom_spmi-regulator.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/regulator/qcom_spmi-regulator.c b/drivers/regulator/qcom_spmi-regulator.c index 162b865..15bba2b 100644 --- a/drivers/regulator/qcom_spmi-regulator.c +++ b/drivers/regulator/qcom_spmi-regulator.c @@ -710,12 +710,8 @@ static int spmi_regulator_ult_lo_smps_set_voltage(struct regulator_dev *rdev, if (range_sel == 1) voltage_sel |= ULT_SMPS_RANGE_SPLIT; - ret = spmi_vreg_update_bits(vreg, SPMI_COMMON_REG_VOLTAGE_SET, + return spmi_vreg_update_bits(vreg, SPMI_COMMON_REG_VOLTAGE_SET, voltage_sel, 0xff); - if (ret) - return ret; - - return 0; } static int spmi_regulator_ult_lo_smps_get_voltage(struct regulator_dev *rdev) |