From 2187b4d9acbc1fa9a14ec78e8db4ad0c595c67b0 Mon Sep 17 00:00:00 2001 From: Bill Richardson Date: Mon, 10 Jun 2013 12:55:06 -0700 Subject: Revert "Add some debugging messages for unresponsive batteries" This reverts commit c848e3216179ad0fb53d937da20237d4fe0266c7. Too noisy. It was supposed to only show it once, but it spews constantly instead. Debug it later - just remove it for now. BUG=chrome-os-partner:20057 BRANCH=none TEST=none Change-Id: Ie6dfd4ccb84a612bb72a0d2b758361a13644e4d9 Signed-off-by: Bill Richardson Reviewed-on: https://gerrit.chromium.org/gerrit/58111 Reviewed-by: Duncan Laurie --- common/charge_state.c | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/common/charge_state.c b/common/charge_state.c index 6cdcc6353e..1536c2b1b7 100644 --- a/common/charge_state.c +++ b/common/charge_state.c @@ -176,7 +176,6 @@ 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)) { @@ -186,16 +185,12 @@ 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; } @@ -203,10 +198,8 @@ static int state_common(struct power_state_context *ctx) } /* Set error if battery is still unresponsive */ - if (rv) { - CPRINTF("[%T Battery still not responding]\n"); + if (rv) curr->error |= F_BATTERY_TEMPERATURE; - } } else { ctx->battery_present = 1; } -- cgit v1.2.1