diff options
| author |   Chanwoo Choi <cw00.choi@samsung.com> | 2012-11-22 16:53:51 +0900 |
|---|---|---|
| committer |   Anton Vorontsov <anton.vorontsov@linaro.org> | 2012-12-16 22:45:58 -0800 |
| commit | f36b9ddbab408f5f5ed9105d857189b84337af48 (patch) | |
| tree | 3bb1a850b2c66f2702d4f438fb7d5915f7180738 /drivers/power | |
| parent | charger-manager: Fix bug related to checking fully charged state of battery (diff) | |
| download | linux-dev-f36b9ddbab408f5f5ed9105d857189b84337af48.tar.xz linux-dev-f36b9ddbab408f5f5ed9105d857189b84337af48.zip | |
charger-manager: Fix bug when check dropped voltage after fullbatt event
This patch check difference value between current voltage of battery and
desc->fullbatt_uV whether positve or negative number. If difference value
is negative number when current voltage of battery is larger than
desc->fullbatt_uV, charger-manager return immediately because battery is
fully charged.
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: Myungjoo Ham <myungjoo.ham@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
Diffstat (limited to 'drivers/power')
| -rw-r--r-- | drivers/power/charger-manager.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/power/charger-manager.c b/drivers/power/charger-manager.c index 633e41ca49ac..6ba047f5ac2c 100644 --- a/drivers/power/charger-manager.c +++ b/drivers/power/charger-manager.c @@ -482,8 +482,9 @@ static void fullbatt_vchk(struct work_struct *work) return; } - diff = desc->fullbatt_uV; - diff -= batt_uV; + diff = desc->fullbatt_uV - batt_uV; + if (diff < 0) + return; dev_info(cm->dev, "VBATT dropped %duV after full-batt.\n", diff); |