lkml.org 
[lkml]   [2012]   [Mar]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Subject[PATCH 3/4] regulator: wm8350: Fix the logic to choose best current limit setting
    From
    Date
    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>
    ---
    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;
    --
    1.7.5.4




    \
     
     \ /
      Last update: 2012-03-27 09:23    [W:0.022 / U:0.904 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site