diff options
| author | Bill Richardson <wfrichar@chromium.org> | 2014-04-08 17:40:07 -0700 |
|---|---|---|
| committer | Bill Richardson <wfrichar@chromium.org> | 2014-04-09 18:29:58 +0000 |
| commit | 3505d8972c647a708da3e2b29ba69a59094aa05d (patch) | |
| tree | 5acd241db529b292e2e894a07d63ea95a22456f5 | |
| parent | 4da95203866c871aa47c0bd779bdc110381360f0 (diff) | |
| download | chrome-ec-3505d8972c647a708da3e2b29ba69a59094aa05d.tar.gz | |
Small bugfix in low-battery notification (charge_state_v2.c)
I used '&&' instead of '&' for bitfield operations. Hooray for test coverage!
BUG=chrome-os-partner:20881
BRANCH=ToT
TEST=make buildall -j
Change-Id: I7e6343df4aead456d4984bf77667aa5cf852aa15
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/193735
Reviewed-by: Alec Berg <alecaberg@chromium.org>
| -rw-r--r-- | common/charge_state_v2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/charge_state_v2.c b/common/charge_state_v2.c index 6cd376bf16..6ca6222370 100644 --- a/common/charge_state_v2.c +++ b/common/charge_state_v2.c @@ -385,7 +385,7 @@ static void prevent_deep_discharge(void) static void notify_host_of_low_battery(void) { /* We can't tell what the current charge is. Assume it's okay. */ - if (curr.batt.flags && BATT_FLAG_BAD_STATE_OF_CHARGE) + if (curr.batt.flags & BATT_FLAG_BAD_STATE_OF_CHARGE) return; if (curr.batt.state_of_charge <= BATTERY_LEVEL_LOW && |