From a63ae25e23ac05e96812facb5211e68b1245b653 Mon Sep 17 00:00:00 2001 From: Yilun Lin Date: Thu, 22 Aug 2019 16:52:20 +0800 Subject: battery/mm8013: fix battery present status Tighter battery present status by returning BP_NO. charger task will try to charge battery when battery present status is BP_NOT_SURE, and this would put PD and charger tasks in a weird state (and keep resetting) if the battery is actually not present. TEST=boot krane w/o battery and w/ PD charger, and see it doesn't reset for over 5 mins. BUG=b:139509751 BRANCH=none Change-Id: I3e3d818b80ffcfdb12fbd83daafa792d837a0fe0 Signed-off-by: Yilun Lin Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1765108 Reviewed-by: Ting Shen Reviewed-by: Daisuke Nojiri --- driver/battery/mm8013.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'driver/battery') diff --git a/driver/battery/mm8013.c b/driver/battery/mm8013.c index d7fba89076..6013146be9 100644 --- a/driver/battery/mm8013.c +++ b/driver/battery/mm8013.c @@ -170,7 +170,7 @@ enum battery_present battery_is_present(void) int temp; if (mm8013_read16(REG_TEMPERATURE, &temp)) - return BP_NOT_SURE; + return BP_NO; return BP_YES; } -- cgit v1.2.1