diff options
| author | Tommy Chung <tommy.chung@quanta.corp-partner.google.com> | 2021-08-04 18:55:02 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-08-09 16:29:17 +0000 |
| commit | 48f83b8b045e4105c4811bf8982cc109e7408b9e (patch) | |
| tree | c5a139e7ff093ea1e37e48a691f3733a1a985f6b /common | |
| parent | 871c4115c23ee7ad79131d55edfb178237f9327f (diff) | |
| download | chrome-ec-48f83b8b045e4105c4811bf8982cc109e7408b9e.tar.gz | |
chgstv2: call sustain_battery_soc without checking previous states
When we enalbe battery sustain after battery returns that it is full,
we need to add this condition for sustain_battery_soc(). Also, add this
test condition for battery sustainer.
BUG=b:188457962
BRANCH=none
TEST=make sure the battery sustain works when enabling it after battery
returns that it is full. Also, make sure that "make run-sbs_charging_v2"
pass.
Signed-off-by: Tommy Chung <tommy.chung@quanta.corp-partner.google.com>
Change-Id: If10b9fd0264717abfb7cdbb7ddc947b370291895
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3070946
Reviewed-by: Devin Lu <Devin.Lu@quantatw.com>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Commit-Queue: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'common')
| -rw-r--r-- | common/charge_state_v2.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/common/charge_state_v2.c b/common/charge_state_v2.c index 48a9f35b02..e5f62fff60 100644 --- a/common/charge_state_v2.c +++ b/common/charge_state_v2.c @@ -2168,6 +2168,10 @@ wait_for_it: /* And the EC console */ is_full = calc_is_full(); + + /* Run battery sustainer (no-op if not applicable). */ + sustain_battery_soc(); + if ((!(curr.batt.flags & BATT_FLAG_BAD_STATE_OF_CHARGE) && curr.batt.state_of_charge != prev_charge) || #ifdef CONFIG_EC_EC_COMM_BATTERY_CLIENT @@ -2176,7 +2180,6 @@ wait_for_it: (is_full != prev_full) || (curr.state != prev_state) || (charge_get_display_charge() != prev_disp_charge)) { - sustain_battery_soc(); show_charging_progress(); prev_charge = curr.batt.state_of_charge; prev_disp_charge = charge_get_display_charge(); |