Commit 23ca6bf2 authored by Axel Lin's avatar Axel Lin Committed by Mark Brown

regulator: max77686: Fix the delay time for set_voltage_time_sel

rdev->desc->uV_step * abs(new_selector - old_selector) returns uV.
The unit of ramp_rate is mV/us.
Thus 1000 should be multiplied.
Signed-off-by: default avatarAxel Lin <axel.lin@gmail.com>
Reviewed-by: default avatarYadwinder Singh Brar <yadi.brar@samsung.com>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 98a175b6
...@@ -77,7 +77,7 @@ static int max77686_set_dvs_voltage_time_sel(struct regulator_dev *rdev, ...@@ -77,7 +77,7 @@ static int max77686_set_dvs_voltage_time_sel(struct regulator_dev *rdev,
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),
ramp_rate[max77686->ramp_delay]); ramp_rate[max77686->ramp_delay] * 1000);
} }
static int max77686_set_voltage_time_sel(struct regulator_dev *rdev, static int max77686_set_voltage_time_sel(struct regulator_dev *rdev,
...@@ -85,7 +85,7 @@ static int max77686_set_voltage_time_sel(struct regulator_dev *rdev, ...@@ -85,7 +85,7 @@ static int max77686_set_voltage_time_sel(struct regulator_dev *rdev,
{ {
/* Unconditionally 100 mV/us */ /* Unconditionally 100 mV/us */
return DIV_ROUND_UP(rdev->desc->uV_step * return DIV_ROUND_UP(rdev->desc->uV_step *
abs(new_selector - old_selector), 100); abs(new_selector - old_selector), 100 * 1000);
} }
static struct regulator_ops max77686_ops = { static struct regulator_ops max77686_ops = {
......
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