From cc9b3464e26e112b2c220455a3938235f8553a2c Mon Sep 17 00:00:00 2001 From: Bill Richardson Date: Thu, 13 Jun 2013 14:11:49 -0700 Subject: Falco: Disable IFAULT_HI on the BQ24738 charger Partners are adamant that this needs doing. Whatever... BUG=chrome-os-partner:19868 BRANCH=none TEST=none You can run the "charger" command on the EC console to see that the option bits have been changed. I couldn't reproduce the original complaint, so I don't know if this solves it. Change-Id: I428697f69a7ba1b360e2acb123b42ed41244ebc5 Signed-off-by: Bill Richardson Reviewed-on: https://gerrit.chromium.org/gerrit/58576 --- common/charger_bq24738.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/common/charger_bq24738.c b/common/charger_bq24738.c index a578196ddc..1bcbf22f1d 100644 --- a/common/charger_bq24738.c +++ b/common/charger_bq24738.c @@ -174,11 +174,9 @@ int charger_set_voltage(int voltage) int charger_post_init(void) { int rv; - -/* FIXME(crosbug.com/p/19868): Do we want this or not? */ -#if 0 int val; + /* Disable IFAULT_HI. See crosbug.com/p/19868 */ rv = charger_get_option(&val); if (rv) return rv; @@ -186,7 +184,6 @@ int charger_post_init(void) rv = charger_set_option(val); if (rv) return rv; -#endif /* Set charger input current limit */ rv = charger_set_input_current(CONFIG_CHARGER_INPUT_CURRENT); -- cgit v1.2.1