diff options
| author | Axel Lin <axel.lin@gmail.com> | 2012-03-27 15:20:08 +0800 |
|---|---|---|
| committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-03-28 12:44:07 +0100 |
| commit | 3a744038b3709cd467b693f3e146c6d5b8120a18 (patch) | |
| tree | 840ff5477c11763260773054bff60096cbe1576b /drivers/regulator | |
| parent | ed3be9a0e3c1050fe07d69a8c600d86cac76cdc4 (diff) | |
| download | op-kernel-dev-3a744038b3709cd467b693f3e146c6d5b8120a18.zip op-kernel-dev-3a744038b3709cd467b693f3e146c6d5b8120a18.tar.gz | |
regulator: wm8350: Fix the logic to choose best current limit setting
Current implementation in get_isink_val actually choose the biggest current
limit setting falls within the specified range.
What we want is to choose the smallest current limit setting falls within the
specified range. Fix it.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'drivers/regulator')
| -rw-r--r-- | drivers/regulator/wm8350-regulator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/wm8350-regulator.c b/drivers/regulator/wm8350-regulator.c index ff34654..f29803c 100644 --- a/drivers/regulator/wm8350-regulator.c +++ b/drivers/regulator/wm8350-regulator.c @@ -99,7 +99,7 @@ static int get_isink_val(int min_uA, int max_uA, u16 *setting) { int i; - for (i = ARRAY_SIZE(isink_cur) - 1; i >= 0; i--) { + for (i = 0; i < ARRAY_SIZE(isink_cur); i++) { if (min_uA <= isink_cur[i] && max_uA >= isink_cur[i]) { *setting = i; return 0; |
