diff options
| author | Daisuke Nojiri <dnojiri@chromium.org> | 2020-09-25 13:42:38 -0700 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-09-13 17:34:03 +0000 |
| commit | f6367fbbdb0c5ba224cb37967b2bdef5081aad65 (patch) | |
| tree | b26b349774ba08747e6f391b49ebe7cc2ea1d00d | |
| parent | 193b7fcffe40413fd70e222ee52106baf0d844aa (diff) | |
| download | chrome-ec-f6367fbbdb0c5ba224cb37967b2bdef5081aad65.tar.gz | |
Battery: Apply faked state of charge to remaining capacity
Currently, only the state of charge is modified by battfake command.
This patch applies the faked state of charge to the remaining capacity
as well.
BUG=b:163721887
BRANCH=Octopus
TEST=Verified Fleex wakes up host on BATTERY_LEVEL_CRITICAL.
TEST=Verified powerd reports faked state of charge.
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Change-Id: I34eb409bab129804a983abbe2a1d021298ce044e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2432865
Reviewed-by: Todd Broch <tbroch@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3872713
Reviewed-by: Jett Rink <jettrink@chromium.org>
| -rw-r--r-- | driver/battery/smart.c | 28 |
1 files changed, 22 insertions, 6 deletions
diff --git a/driver/battery/smart.c b/driver/battery/smart.c index 51a04d3b6b..643793a7ac 100644 --- a/driver/battery/smart.c +++ b/driver/battery/smart.c @@ -306,6 +306,24 @@ int battery_get_avg_voltage(void) } #endif /* CONFIG_CMD_PWR_AVG */ +static void apply_fake_state_of_charge(struct batt_params *batt) +{ + int full; + + if (fake_state_of_charge < 0) + return; + + if (batt->flags & BATT_FLAG_BAD_FULL_CAPACITY) + battery_design_capacity(&full); + else + full = batt->full_capacity; + + batt->state_of_charge = fake_state_of_charge; + batt->remaining_capacity = full * fake_state_of_charge / 100; + batt->flags &= ~BATT_FLAG_BAD_STATE_OF_CHARGE; + batt->flags &= ~BATT_FLAG_BAD_REMAINING_CAPACITY; +} + void battery_get_params(struct batt_params *batt) { struct batt_params batt_new = {0}; @@ -318,10 +336,6 @@ void battery_get_params(struct batt_params *batt) && fake_state_of_charge < 0) batt_new.flags |= BATT_FLAG_BAD_STATE_OF_CHARGE; - /* If soc is faked, override with faked data */ - if (fake_state_of_charge >= 0) - batt_new.state_of_charge = fake_state_of_charge; - if (sb_read(SB_VOLTAGE, &batt_new.voltage)) batt_new.flags |= BATT_FLAG_BAD_VOLTAGE; @@ -401,6 +415,9 @@ void battery_get_params(struct batt_params *batt) board_battery_compensate_params(&batt_new); #endif + if (IS_ENABLED(CONFIG_CMD_BATTFAKE)) + apply_fake_state_of_charge(&batt_new); + /* Update visible battery parameters */ memcpy(batt, &batt_new, sizeof(*batt)); } @@ -441,8 +458,7 @@ static int command_battfake(int argc, char **argv) } if (fake_state_of_charge >= 0) - ccprintf("Fake batt %d%%\n", - fake_state_of_charge); + ccprintf("Fake batt %d%%\n", fake_state_of_charge); return EC_SUCCESS; } |