summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBill Richardson <wfrichar@chromium.org>2013-06-10 12:55:06 -0700
committerChromeBot <chrome-bot@google.com>2013-06-10 18:08:31 -0700
commit2187b4d9acbc1fa9a14ec78e8db4ad0c595c67b0 (patch)
tree25f8a3c9dd2532911db26b50589815ec2955d931
parent0b8f5a3a6ca9722e4c7bc931577b08616e31d72a (diff)
downloadchrome-ec-2187b4d9acbc1fa9a14ec78e8db4ad0c595c67b0.tar.gz
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 <wfrichar@chromium.org> Reviewed-on: https://gerrit.chromium.org/gerrit/58111 Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
-rw-r--r--common/charge_state.c9
1 files changed, 1 insertions, 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;
}