Skip to content

Commit ad7403b

Browse files
committed
power: supply: ln8411: Improves charger health reporting
Refactors charger health reporting to prioritze overcurrent reporting and eliminates checking the general volt fault bit. Change-Id: I7090fd81b60c48f2d1d476460c6826318182ffd7 Signed-off-by: Ricardo Rivera-Matos <[email protected]>
1 parent a89bfa3 commit ad7403b

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

drivers/power/supply/ln8411_charger.c

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -201,15 +201,12 @@ static int ln8411_get_charger_health(struct ln8411_device *ln8411, union power_s
201201

202202
if (buf[0] & LN8411_WATCHDOG_TIMER_STS) {
203203
val->intval = POWER_SUPPLY_HEALTH_WATCHDOG_TIMER_EXPIRE;
204-
} else if (buf[0] & LN8411_PMID2OUT_OV_STS) {
205-
val->intval = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
206-
dev_dbg(ln8411->dev, "PMID2OUT overvoltage condition detected!\n");
207204
} else if (buf[2] & LN8411_IBUS_OC_DETECTED) {
208205
val->intval = POWER_SUPPLY_HEALTH_OVERCURRENT;
209206
dev_dbg(ln8411->dev, "IBUS overcurrent condition detected!\n");
210-
} else if (buf[0] & LN8411_VOLT_FAULT_DETECTED) {
211-
val->intval = POWER_SUPPLY_HEALTH_UNSPEC_FAILURE;
212-
dev_dbg(ln8411->dev, "Voltage fault condition detected! FAULT1_STS:0x%x, FAULT2_STS:0x%x, FAULT3_STS:0x%x\n", buf[0], buf[1], buf[2]);
207+
} else if (buf[0] & LN8411_PMID2OUT_OV_STS) {
208+
val->intval = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
209+
dev_dbg(ln8411->dev, "PMID2OUT overvoltage condition detected!\n");
213210
} else {
214211
val->intval = POWER_SUPPLY_HEALTH_GOOD;
215212
}

0 commit comments

Comments
 (0)