regulator: MAX8998: set_voltage bugfix. ramp_up delay and min/max voltage
authorMyungJoo Ham <myungjoo.ham@samsung.com>
Tue, 20 Jul 2010 06:07:07 +0000 (08:07 +0200)
committerLiam Girdwood <lrg@slimlogic.co.uk>
Wed, 11 Aug 2010 10:38:01 +0000 (11:38 +0100)
commitc5a4655db9e0b5e49a0b17e1bf96462e93e3c326
treede2afadd98d171248fc0f2a2edb4760f44470b29
parentc789ca202cae5adfab0208e7e8becbc602e5d079
regulator: MAX8998: set_voltage bugfix. ramp_up delay and min/max voltage

Two issues are addressed for max8998_set_voltage function.

1. Min/Max Voltage.

max8998_set_voltage had been using the voltage value of

min ( voltage[i] >= max_vol , i )

This is corrected to use:

min ( voltage[i] >= min_vol , i )

2. Ramp Up Delay.

max8998_set_voltage should provide delay for BUCK1/2
if ENRAMP is on. It reads RAMP value from ONOFF4 register to determine
RAMP delay length. However, when max8998_set_voltage's new voltage is
lower than the previous, we don't care because it does not deteriorate
the stability.

Changes since v1:
- rebased onto latest regulator-for-next tree

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
drivers/regulator/max8998.c