diff options
| author | Andrew McRae <amcrae@google.com> | 2022-03-29 17:12:33 +1100 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-03-29 21:53:43 +0000 |
| commit | cd70889abfe59865efa29ce47bd1bff11dbb9acd (patch) | |
| tree | 11c3e425823039421a7821c919186fd80425333d /util | |
| parent | 9b2ef47ab9ee59606ca4e7d0a3a5594bdd756fff (diff) | |
| download | chrome-ec-cd70889abfe59865efa29ce47bd1bff11dbb9acd.tar.gz | |
ectool: Allow a negative value for battery charge rate
A negative value indicates the battery is discharging, so
allow ectool to display it.
BUG=b:227275730
TEST=cros deploy dut1 ec-utils
BRANCH=none
Signed-off-by: Andrew McRae <amcrae@google.com>
Change-Id: I17191691d5e0eb29180cfed39b9991fa48797930
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3555952
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'util')
| -rw-r--r-- | util/ectool.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/util/ectool.c b/util/ectool.c index 1f7826a447..d17ba521c7 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -8021,9 +8021,8 @@ int cmd_battery(int argc, char *argv[]) printf(" Present voltage %u mV\n", val); val = read_mapped_mem32(EC_MEMMAP_BATT_RATE); - if (!is_battery_range(val)) - goto cmd_error; - printf(" Present current %u mA\n", val); + /* Current can be negative */ + printf(" Present current %d mA\n", val); val = read_mapped_mem32(EC_MEMMAP_BATT_CAP); if (!is_battery_range(val)) |