Re: [PATCH] power: bq25890: Use the correct range for IILIM register

[Michał Mirosław]

On Wed, Nov 25, 2020 at 04:48:05AM +0100, Sebastian Krzyszkowiak wrote:
> I've checked bq25890, bq25892, bq25895 and bq25896 datasheets and
> they all define IILIM to be between 100mA-3.25A with 50mA steps.

That's what DS says, indeed. 

Reviewed-by: Michał Mirosław 

[PATCH] power: bq25890: Use the correct range for IILIM register

[Sebastian Krzyszkowiak]

I've checked bq25890, bq25892, bq25895 and bq25896 datasheets and
they all define IILIM to be between 100mA-3.25A with 50mA steps.

Fixes: 478efc79ee32 ("power: bq25890: implement INPUT_CURRENT_LIMIT property")

Signed-off-by: Sebastian Krzyszkowiak 
---
 drivers/power/supply/bq25890_charger.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/supply/bq25890_charger.c 
b/drivers/power/supply/bq25890_charger.c
index 5ab7d2c5549c..e95557e46a6d 100644
--- a/drivers/power/supply/bq25890_charger.c
+++ b/drivers/power/supply/bq25890_charger.c
@@ -303,7 +303,7 @@ static const union {
/* TODO: BQ25896 has max ICHG 3008 mA */
[TBL_ICHG] ={ .rt = {0,   5056000, 64000} }, /* uA */
[TBL_ITERM] =   { .rt = {64000,   1024000, 64000} }, /* uA */
-   [TBL_IILIM] =   { .rt = {5,   320, 5} }, /* uA */
+   [TBL_IILIM] =   { .rt = {10,  325, 5} }, /* uA */
[TBL_VREG] ={ .rt = {384, 4608000, 16000} }, /* uV */
[TBL_BOOSTV] =  { .rt = {455, 551, 64000} }, /* uV */
[TBL_SYSVMIN] = { .rt = {300, 370, 10} },/* uV */
-- 
2.29.2