Commit f36b9ddb authored by Chanwoo Choi's avatar Chanwoo Choi Committed by Anton Vorontsov

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: default avatarChanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: default avatarMyungjoo Ham <myungjoo.ham@samsung.com>
Signed-off-by: default avatarKyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: default avatarAnton Vorontsov <anton.vorontsov@linaro.org>
parent 0fa11dbc
...@@ -482,8 +482,9 @@ static void fullbatt_vchk(struct work_struct *work) ...@@ -482,8 +482,9 @@ static void fullbatt_vchk(struct work_struct *work)
return; return;
} }
diff = desc->fullbatt_uV; diff = desc->fullbatt_uV - batt_uV;
diff -= batt_uV; if (diff < 0)
return;
dev_info(cm->dev, "VBATT dropped %duV after full-batt.\n", diff); dev_info(cm->dev, "VBATT dropped %duV after full-batt.\n", diff);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment