diff options
| author | ye xingchen <ye.xingchen@zte.com.cn> | 2022-08-24 07:47:07 +0000 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2022-08-24 12:21:30 +0100 |
| commit | 48aa47308de609e687dc187b72de92e3346c4187 (patch) | |
| tree | ce6a5a2615aca61c583ba25536bc0dcf825c8d85 /drivers/regulator | |
| parent | d9270292e6174551d3b02a49b310ddf56c0225fe (diff) | |
| download | linux-48aa47308de609e687dc187b72de92e3346c4187.tar.bz2 | |
regulator: max597x: Remove the unneeded result variable
Return the value from regmap_write() directly instead of storing it
in another redundant variable.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: ye xingchen <ye.xingchen@zte.com.cn>
Link: https://lore.kernel.org/r/20220824074707.221159-1-ye.xingchen@zte.com.cn
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator')
| -rw-r--r-- | drivers/regulator/max597x-regulator.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/regulator/max597x-regulator.c b/drivers/regulator/max597x-regulator.c index 03c6027682d8..39f803ff0a90 100644 --- a/drivers/regulator/max597x-regulator.c +++ b/drivers/regulator/max597x-regulator.c @@ -137,7 +137,7 @@ static int max597x_set_ovp(struct regulator_dev *rdev, int lim_uV, int severity, static int max597x_set_ocp(struct regulator_dev *rdev, int lim_uA, int severity, bool enable) { - int ret, val, reg; + int val, reg; unsigned int vthst, vthfst; struct max597x_regulator *data = rdev_get_drvdata(rdev); @@ -183,9 +183,8 @@ static int max597x_set_ocp(struct regulator_dev *rdev, int lim_uA, val = 0xFF; reg = MAX5970_REG_DAC_FAST(rdev_id); - ret = regmap_write(rdev->regmap, reg, val); - return ret; + return regmap_write(rdev->regmap, reg, val); } static int max597x_get_status(struct regulator_dev *rdev) |