| Message ID | 20190213164648.26579-3-krzk@kernel.org |
|---|---|
| State | Changes Requested |
| Delegated to: | Minkyu Kang |
| Headers | show |
| Series | arm: exynos: Fix reboot on Odroid HC1 | expand |
On Wed, 13 Feb 2019 17:46:41 +0100 Krzysztof Kozlowski <krzk@kernel.org> wrote: > LDO27 and LDO35 have 25 mV step, not 50 mV. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > --- > drivers/power/regulator/s2mps11_regulator.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/power/regulator/s2mps11_regulator.c > b/drivers/power/regulator/s2mps11_regulator.c index > ced504eb1476..723d27f67c9a 100644 --- > a/drivers/power/regulator/s2mps11_regulator.c +++ > b/drivers/power/regulator/s2mps11_regulator.c @@ -346,6 +346,8 @@ > static int s2mps11_ldo_hex2volt(int ldo, int hex) case 11: > case 22: > case 23: > + case 27: > + case 35: > uV = hex * S2MPS11_LDO_STEP + S2MPS11_LDO_UV_MIN; > break; > default: > @@ -366,6 +368,8 @@ static int s2mps11_ldo_volt2hex(int ldo, int uV) > case 11: > case 22: > case 23: > + case 27: > + case 35: > hex = (uV - S2MPS11_LDO_UV_MIN) / S2MPS11_LDO_STEP; > break; > default: Reviewed-by: Lukasz Majewski <lukma@denx.de> Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de
diff --git a/drivers/power/regulator/s2mps11_regulator.c b/drivers/power/regulator/s2mps11_regulator.c index ced504eb1476..723d27f67c9a 100644 --- a/drivers/power/regulator/s2mps11_regulator.c +++ b/drivers/power/regulator/s2mps11_regulator.c @@ -346,6 +346,8 @@ static int s2mps11_ldo_hex2volt(int ldo, int hex) case 11: case 22: case 23: + case 27: + case 35: uV = hex * S2MPS11_LDO_STEP + S2MPS11_LDO_UV_MIN; break; default: @@ -366,6 +368,8 @@ static int s2mps11_ldo_volt2hex(int ldo, int uV) case 11: case 22: case 23: + case 27: + case 35: hex = (uV - S2MPS11_LDO_UV_MIN) / S2MPS11_LDO_STEP; break; default:
LDO27 and LDO35 have 25 mV step, not 50 mV. Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- drivers/power/regulator/s2mps11_regulator.c | 4 ++++ 1 file changed, 4 insertions(+)