diff options
| author |   Axel Lin <axel.lin@gmail.com> | 2012-04-10 23:07:52 +0800 |
|---|---|---|
| committer |   Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-04-10 17:28:52 +0100 |
| commit | 0cb2f1239dd7acf72fb60adb53c1fdaa65b719a2 (patch) | |
| tree | 8226f9e3054e927dc50c38e8fabb3cb0ff82e4f4 /drivers/regulator/anatop-regulator.c | |
| parent | regulator: tps65023: Use devm_* APIs (diff) | |
| download | linux-dev-0cb2f1239dd7acf72fb60adb53c1fdaa65b719a2.tar.xz linux-dev-0cb2f1239dd7acf72fb60adb53c1fdaa65b719a2.zip | |
regulator: twl-regulator: Fix off-by-one vsel setting in twl6030smps_set_voltage
commit 268a1 "regulator: twl-regulator: Use DIV_ROUND_UP at appropriate places"
introduced an off-by-one bug for setting vsel.
The linear calculation code in twl6030smps_list_voltage() does subtract index
by 1 so we need the vsel++ after DIV_ROUND_UP.
The original code use this trick to differentiate if we are going to set the
voltage to 0 or the voltage falls within the linear calculation range.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'drivers/regulator/anatop-regulator.c')
0 files changed, 0 insertions, 0 deletions