From 3505d8972c647a708da3e2b29ba69a59094aa05d Mon Sep 17 00:00:00 2001 From: Bill Richardson Date: Tue, 8 Apr 2014 17:40:07 -0700 Subject: [PATCH] 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 Reviewed-on: https://chromium-review.googlesource.com/193735 Reviewed-by: Alec Berg --- common/charge_state_v2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 &&