Skip to contentSkip to conversation

power: bq25890: Use the correct range for IILIM register

PATCH In review Sebastian Krzyszkowiak 2 participants 2 messages Nov 25, 2020 Power supply

lore
Share & tools
lore

Sebastian Krzyszkowiak

author
#0 lore raw

PATCH 1/1 power: bq25890: Use the correct range for IILIM register

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 <[email protected]>
---
 drivers/power/supply/bq25890_charger.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/power/supply/bq25890_charger.c +1 -1
index 5ab7d2c5549c..e95557e46a6d 100644
@@ -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 = {50000,   3200000, 50000} },	 /* uA */
+	[TBL_IILIM] =   { .rt = {100000,  3250000, 50000} },	 /* uA */
 	[TBL_VREG] =	{ .rt = {3840000, 4608000, 16000} },	 /* uV */
 	[TBL_BOOSTV] =	{ .rt = {4550000, 5510000, 64000} },	 /* uV */
 	[TBL_SYSVMIN] = { .rt = {3000000, 3700000, 100000} },	 /* uV */
Signature · 1 line
2.29.2
Reviewed-by on v1 1/1
#1 lore raw
Quoted 2 lines · 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.

That's what DS says, indeed.

Reviewed-by Michał Mirosław <[email protected]>