diff options
| author | Aseda Aboagye <aaboagye@google.com> | 2018-03-22 20:07:39 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-04-10 19:13:32 -0700 |
| commit | 46ca9738f23d91148519fdaf31e8791be49855e3 (patch) | |
| tree | 44c57af9f4710e84ef6735888386b4582cf2719f /board/fizz/board.c | |
| parent | dccaf9d9fce04a1e2fd1eb51fb85627ef2976146 (diff) | |
| download | chrome-ec-46ca9738f23d91148519fdaf31e8791be49855e3.tar.gz | |
chgstv2: Check charger power in prevent_power_on.
charge_prevent_power_on() had sections which were gated on the following
CONFIG_* option:
CONFIG_CHARGER_LIMIT_POWER_THRESH_BAT_PCT
However, the block of code that this gated didn't even take the battery
percentage into account and made it very confusing as to why.
This commit simply changes the CONFIG_* option used to gate to be the
following:
CONFIG_CHARGER_MIN_POWER_MW_FOR_POWER_ON
This better reflects the checks that were actually being made.
Additionally, this CONFIG_* option is defined by default for boards that
have a chipset task and is initialized to 15W, which is the power that
indicates that the charger is likely to speak USB PD.
BUG=b:76174140
BRANCH=None
TEST=make -j buildall
Change-Id: Ic9158dd7109ce6082c6d00157ff266842363b295
Signed-off-by: Aseda Aboagye <aaboagye@google.com>
Reviewed-on: https://chromium-review.googlesource.com/977431
Commit-Ready: Aseda Aboagye <aaboagye@chromium.org>
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Edward Hill <ecgh@chromium.org>
Diffstat (limited to 'board/fizz/board.c')
| -rw-r--r-- | board/fizz/board.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/fizz/board.c b/board/fizz/board.c index 24d31a99c8..98f5f36c68 100644 --- a/board/fizz/board.c +++ b/board/fizz/board.c @@ -511,7 +511,7 @@ void board_set_charge_limit(int port, int supplier, int charge_ma, * is called. */ led_alert(charge_ma * charge_mv < - CONFIG_CHARGER_LIMIT_POWER_THRESH_CHG_MW * 1000); + CONFIG_CHARGER_MIN_POWER_MW_FOR_POWER_ON * 1000); /* * In terms of timing, this should always work because |