diff options
| author | Daisuke Nojiri <dnojiri@chromium.org> | 2018-07-30 16:44:06 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-08-10 18:33:23 -0700 |
| commit | 8713f42d278d05d06c6b5db6b7cfa44e769814d5 (patch) | |
| tree | b41a191bebb9f4205d04e635446477db0e37919d /common/charge_state_v2.c | |
| parent | 894c310d39dba59e8ba8e751f584d31f59a00460 (diff) | |
| download | chrome-ec-8713f42d278d05d06c6b5db6b7cfa44e769814d5.tar.gz | |
chgstv2: Notify HOOK_BATTERY_SOC_CHANGE on battery removal
Currently, when a battery isn't present when charger_task starts, the
state machine doesn't trigger HOOK_BATTERY_SOC_CHANGE hooks. This causes
Nami's LED driver to miss BP_NO state after sysjump.
This patch makes charger_task call hook_notify by initialzing prev_bp
to BP_NOT_INIT.
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
BUG=b:111969858
BRANCH=none
TEST=Verify battery & power LEDs reflect correct BP_NO status on Sona
when a battery is disconnected on boot.
Change-Id: I23bd7446dfacbf33f5719b7da567d07ef79788cc
Reviewed-on: https://chromium-review.googlesource.com/1155970
Commit-Ready: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'common/charge_state_v2.c')
| -rw-r--r-- | common/charge_state_v2.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/common/charge_state_v2.c b/common/charge_state_v2.c index 5cca92f181..35c495d310 100644 --- a/common/charge_state_v2.c +++ b/common/charge_state_v2.c @@ -1527,8 +1527,9 @@ void charger_task(void *u) * as needed. */ battery_get_params(&curr.batt); - prev_bp = curr.batt.is_present; - curr.desired_input_current = get_desired_input_current(prev_bp, info); + prev_bp = BP_NOT_INIT; + curr.desired_input_current = get_desired_input_current( + curr.batt.is_present, info); while (1) { |