diff options
| author | Vic Yang <victoryang@chromium.org> | 2012-10-17 14:36:42 +0800 |
|---|---|---|
| committer | Gerrit <chrome-bot@google.com> | 2012-10-17 00:54:30 -0700 |
| commit | a2c0103908876afd1999f607ae5a285958c2b3a0 (patch) | |
| tree | 614b69cf5b4ddddc211142dd7037e393010c3a57 | |
| parent | a6ec0e16d8b27b94325b62be267d221d625ed596 (diff) | |
| download | chrome-ec-a2c0103908876afd1999f607ae5a285958c2b3a0.tar.gz | |
Fix delay loop for battery detecting
The charging voltage and current are not necessary zero. Remove the
conditions.
BUG=chrome-os-partner:14094
TEST=Remove the battery with AC connected. LED turns red after 30
seconds.
Plug in AC with battery disconnected. LED turns red after 30 seconds.
BRANCH=link
Change-Id: I754383b05af6de0ceeb706612f1cb534254e47e3
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/35807
Reviewed-by: Rong Chang <rongchang@chromium.org>
Tested-by: Rong Chang <rongchang@chromium.org>
| -rw-r--r-- | common/charge_state.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/common/charge_state.c b/common/charge_state.c index 8b4c6219f8..3aaad3b153 100644 --- a/common/charge_state.c +++ b/common/charge_state.c @@ -179,9 +179,7 @@ static int state_common(struct power_state_context *ctx) if (rv) { /* Check low battery condition and retry */ if (curr->ac && ctx->battery_present == 1 && - !(curr->error & F_CHARGER_MASK) && - (curr->charging_voltage == 0 || - curr->charging_current == 0)) { + !(curr->error & F_CHARGER_MASK)) { ctx->battery_present = 0; /* * Try to revive ultra low voltage pack. |