diff options
| author | Yilun Lin <yllin@chromium.org> | 2019-10-30 16:19:14 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2019-10-30 10:38:46 +0000 |
| commit | 4548b81d8ec8d58339c7981c81f9edae5dd91c6f (patch) | |
| tree | 5b7c4e199b877bd5c602a711a71cdfa699e47f0f /driver | |
| parent | 1b54e57c7e7bf50dbf31f24fe5a4c293d6698ffd (diff) | |
| download | chrome-ec-4548b81d8ec8d58339c7981c81f9edae5dd91c6f.tar.gz | |
battery/mm8013: Fix reversed WANT_CHG flag.
battery_flag()'s return code is 0 on success.
TEST=see allowing charge in command battery
BUG=None
BRANCH=kukui
Change-Id: I26d6a69deadca1ad5a0f5d7faaa031ed0dffe78d
Signed-off-by: Yilun Lin <yllin@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1888004
Reviewed-by: Ting Shen <phoenixshen@chromium.org>
Diffstat (limited to 'driver')
| -rw-r--r-- | driver/battery/mm8013.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/driver/battery/mm8013.c b/driver/battery/mm8013.c index 6013146be9..cd1a2a2083 100644 --- a/driver/battery/mm8013.c +++ b/driver/battery/mm8013.c @@ -212,12 +212,12 @@ void battery_get_params(struct batt_params *batt) if (battery_full_charge_capacity(&batt_new.full_capacity)) batt_new.flags |= BATT_FLAG_BAD_FULL_CAPACITY; - if (battery_flag(&flag) && (flag & MM8013_FLAG_CHG)) - batt_new.flags |= BATT_FLAG_WANT_CHARGE; - if (battery_status(&batt_new.status)) batt_new.flags |= BATT_FLAG_BAD_STATUS; + if (!battery_flag(&flag) && (flag & MM8013_FLAG_CHG)) + batt_new.flags |= BATT_FLAG_WANT_CHARGE; + batt_out: /* Update visible battery parameters */ memcpy(batt, &batt_new, sizeof(*batt)); |