summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChanwoo Choi <cw00.choi@samsung.com>2012-11-22 16:53:51 +0900
committerAnton Vorontsov <anton.vorontsov@linaro.org>2012-12-16 22:45:58 -0800
commitf36b9ddbab408f5f5ed9105d857189b84337af48 (patch)
tree3bb1a850b2c66f2702d4f438fb7d5915f7180738
parent0fa11dbc234268c4160b81f67220308dea3c156a (diff)
downloadlinux-exynos-f36b9ddbab408f5f5ed9105d857189b84337af48.tar.gz
linux-exynos-f36b9ddbab408f5f5ed9105d857189b84337af48.tar.bz2
linux-exynos-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>
-rw-r--r--drivers/power/charger-manager.c5
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);