Commit 578df8ba authored by Axel Lin's avatar Axel Lin Committed by Mark Brown

regulator: core: Return correct delay time in regulator_set_voltage_time_sel

rdev->desc->uV_step * abs(new_selector - old_selector) returns uV.
The unit of ramp_delay is mV/us.

Current code multiples 1000 at wrong place.
Signed-off-by: default avatarAxel Lin <axel.lin@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 23ca6bf2
...@@ -2299,7 +2299,7 @@ int regulator_set_voltage_time_sel(struct regulator_dev *rdev, ...@@ -2299,7 +2299,7 @@ int regulator_set_voltage_time_sel(struct regulator_dev *rdev,
if (rdev->desc->ramp_delay && rdev->desc->uV_step) if (rdev->desc->ramp_delay && rdev->desc->uV_step)
return DIV_ROUND_UP(rdev->desc->uV_step * return DIV_ROUND_UP(rdev->desc->uV_step *
abs(new_selector - old_selector), abs(new_selector - old_selector),
rdev->desc->ramp_delay) * 1000; rdev->desc->ramp_delay * 1000);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment