summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBill Richardson <wfrichar@chromium.org>2013-06-06 10:15:49 -0700
committerChromeBot <chrome-bot@google.com>2013-06-07 10:52:12 -0700
commitc848e3216179ad0fb53d937da20237d4fe0266c7 (patch)
tree0e15208ed6332d4090417cb4dd4145d691615d45
parent124b2f1492b970ebaf34c14920bdcdcc79be536e (diff)
downloadchrome-ec-c848e3216179ad0fb53d937da20237d4fe0266c7.tar.gz
Add some debugging messages for unresponsive batteries
BUG=none BRANCH=none TEST=none Change-Id: Id56c2b180c670115819dd29c85ecc3d0e96bd26f Signed-off-by: Bill Richardson <wfrichar@chromium.org> Reviewed-on: https://gerrit.chromium.org/gerrit/57812 Reviewed-by: Randall Spangler <rspangler@chromium.org>
-rw-r--r--common/charge_state.c9
1 files changed, 8 insertions, 1 deletions
diff --git a/common/charge_state.c b/common/charge_state.c
index 1536c2b1b7..6cdcc6353e 100644
--- a/common/charge_state.c
+++ b/common/charge_state.c
@@ -176,6 +176,7 @@ static int state_common(struct power_state_context *ctx)
rv = battery_temperature(&batt->temperature);
if (rv) {
+ CPRINTF("[%T Can't communicate with battery!]\n");
/* Check low battery condition and retry */
if (curr->ac && ctx->battery_present == 1 &&
!(curr->error & F_CHARGER_MASK)) {
@@ -185,12 +186,16 @@ static int state_common(struct power_state_context *ctx)
* battery pack with minimum current and maximum
* voltage for 30 seconds.
*/
+ CPRINTF("[%T Trickle charge battery at %dmV, %dmA]\n",
+ ctx->battery->voltage_max,
+ ctx->charger->current_min);
charger_set_voltage(ctx->battery->voltage_max);
charger_set_current(ctx->charger->current_min);
for (d = 0; d < 30; d++) {
sleep(1);
rv = battery_temperature(&batt->temperature);
if (rv == 0) {
+ CPRINTF("[%T Battery is responding]\n");
ctx->battery_present = 1;
break;
}
@@ -198,8 +203,10 @@ static int state_common(struct power_state_context *ctx)
}
/* Set error if battery is still unresponsive */
- if (rv)
+ if (rv) {
+ CPRINTF("[%T Battery still not responding]\n");
curr->error |= F_BATTERY_TEMPERATURE;
+ }
} else {
ctx->battery_present = 1;
}