diff options
| author | Shawn Nematbakhsh <shawnn@chromium.org> | 2017-10-23 15:45:16 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-11-02 23:21:52 -0700 |
| commit | 2f127f3081be00dc318b4e0b9a9947c68b24849d (patch) | |
| tree | a5ca33f3e2654f788e676790dee4d5d65a18b773 /common/charge_state_v2.c | |
| parent | f28ab5c2ecd0549a72dc042564374faade58abcc (diff) | |
| download | chrome-ec-2f127f3081be00dc318b4e0b9a9947c68b24849d.tar.gz | |
charge_manager: Enter safe mode at boot
Charge port / current selection often needs to be significantly altered
when a battery cannot provide sufficient charge, so have charge_manager
initially enter safe mode. After a battery with sufficient capacity has
been identified, charge manager will leave safe mode, and port / current
selection will return to standard rules.
BUG=chromium:777596
BRANCH=None
TEST=Pass charge_manager unit tests. On kevin, remove battery, attach
Apple PD charger, verify safe mode is not exited and device does not
brown out. Hot-plug battery and verify safe mode is exited. Next,
remove battery, attach to Samus, verify safe mode is not exited and
device doesn't brown out. Hot-plug battery, verify that safe mode is
exited and no active charge port, due to dual-role exclusion.
Change-Id: I7784865750087a037aad8dbbac058b22c77ba6d4
Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/733954
Commit-Ready: Shawn N <shawnn@chromium.org>
Tested-by: Shawn N <shawnn@chromium.org>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'common/charge_state_v2.c')
| -rw-r--r-- | common/charge_state_v2.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/common/charge_state_v2.c b/common/charge_state_v2.c index df87184813..cb742f1bb6 100644 --- a/common/charge_state_v2.c +++ b/common/charge_state_v2.c @@ -875,6 +875,13 @@ wait_for_it: } #endif +#ifdef CONFIG_CHARGE_MANAGER + if (curr.batt.state_of_charge >= + CONFIG_CHARGE_MANAGER_BAT_PCT_SAFE_MODE_EXIT && + !battery_seems_to_be_disconnected) + charge_manager_leave_safe_mode(); +#endif + /* Keep the AP informed */ if (need_static) need_static = update_static_battery_info(); |