From d2108bbcaa19260582d2cf6f2b5a19dd6a553c39 Mon Sep 17 00:00:00 2001 From: Philip Chen Date: Wed, 16 Aug 2017 13:49:30 -0700 Subject: [PATCH] battery/max17055: Wait longer for initial config max17055 sometimes fails to finish configuration due to timeout. Let's wait longer. BUG=b:63870414 BRANCH=none TEST=boot scarlet rev1 10 times and ensure MODELCFG.REFRESH bit is cleared when retries value > 10. Change-Id: I62984c19014a244a378180bab6683e2e6f842cd1 Signed-off-by: Philip Chen Reviewed-on: https://chromium-review.googlesource.com/617653 Commit-Ready: Philip Chen Tested-by: Philip Chen Reviewed-by: Aseda Aboagye --- driver/battery/max17055.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/driver/battery/max17055.c b/driver/battery/max17055.c index c676ce22a5..1814fc57ea 100644 --- a/driver/battery/max17055.c +++ b/driver/battery/max17055.c @@ -279,7 +279,7 @@ static int max17055_init_config(void) { int reg; int hib_cfg; - int retries = 20; + int retries = 50; if (max17055_write(REG_DESIGN_CAPACITY, BATTERY_MAX17055_DESIGNCAP) || max17055_write(REG_DQACC, BATTERY_MAX17055_DESIGNCAP / 32) || @@ -304,7 +304,7 @@ static int max17055_init_config(void) if (max17055_write(REG_MODELCFG, (MODELCFG_REFRESH | MODELCFG_VCHG))) return EC_ERROR_UNKNOWN; - /* Delay up to 200 ms until MODELCFG.REFRESH bit == 0. */ + /* Delay up to 500 ms until MODELCFG.REFRESH bit == 0. */ while (--retries) { if (max17055_read(REG_MODELCFG, ®)) return EC_ERROR_UNKNOWN;