From: Chanwoo Choi <cw00.choi@samsung.com>
Date: Thu, 22 Nov 2012 07:53:51 +0000 (+0900)
Subject: charger-manager: Fix bug when check dropped voltage after fullbatt event
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=f36b9ddbab408f5f5ed9105d857189b84337af48;p=linux-beck.git

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>
---

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);