scottk/OpenCellular
1
0
Fork 0
mirror of https://github.com/Telecominfraproject/OpenCellular.git synced 2025-12-27 18:25:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

link: Don't force battery to request current

Browse Source 
Previously, the battery charging algorithm would give the battery
current even when it was requesting 0 mA.

BUG=chrome-os-partner:15573
BRANCH=link
TEST=charge battery and leave AC plugged in

Battery will eventually drop to 99% charged.  At that point, it should
request 0mA, not 84mA or 85mA.  (When it drops farther, it will really
start requesting charge.)

Change-Id: I464a7c8e857c928dbb9f9595f9c947d565b5f5c6
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/36526
Reviewed-by: Rong Chang <rongchang@chromium.org>
This commit is contained in:
Randall Spangler
2012-10-24 18:42:18 -07:00
parent dc53e4db7f
commit 61ebbbbbd7
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
common/battery_link.c
View File

@@ -123,12 +123,8 @@ void battery_vendor_params(struct batt_params *batt)
limit_value(desired_current, current_limit[temp_range][volt_range]);
#ifndef CONFIG_SLOW_PRECHARGE
/* Always request some current for trickle charging and pre-charging */
/*
* TODO: (crosbug.com/p/15573) Shouldn't we only do this if desired
* current is non-zero?
*/
if (*desired_current < info.precharge_current)
/* If battery wants current, give it at least the precharge current */
if (*desired_current > 0 && *desired_current < info.precharge_current)
*desired_current = info.precharge_current;
#endif /* CONFIG_SLOW_PRECHARGE */