diff options
author | Bill Richardson <wfrichar@chromium.org> | 2013-06-13 14:11:49 -0700 |
---|---|---|
committer | Bill Richardson <wfrichar@chromium.org> | 2013-06-13 15:52:29 -0700 |
commit | cc9b3464e26e112b2c220455a3938235f8553a2c (patch) | |
tree | 7e00d7de8ffe2f167b891b997bb4c2f38853217c | |
parent | 74f5aaaa50ae4457b1a01fd3251f575a9b4a0687 (diff) | |
download | chrome-ec-cc9b3464e26e112b2c220455a3938235f8553a2c.tar.gz |
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 <wfrichar@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/58576
-rw-r--r-- | common/charger_bq24738.c | 5 |
1 files changed, 1 insertions, 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); |