commit 71816fde13316655dc72baa6e85ee02e73336d0e
author Abhijeet Dharmapurikar <adharmap@codeaurora.org> Fri Aug 12 17:14:10 2011 -0700
committer Bryan Huntsman <bryanh@codeaurora.org> Mon Oct 03 16:16:37 2011 -0700
tree 26924a324dc694d64ccd1e634072de80099eef89
parent ad899705e71d5de65db9247d6d26ad109bff4e6c

power: pm8921-charger: fix termination current

The driver was writing to SAFE current register instead of the
TERM current register.

Fix it.

Signed-off-by: Abhijeet Dharmapurikar <adharmap@codeaurora.org>

drivers/power/pm8921-charger.c

diff --git a/drivers/power/pm8921-charger.c b/drivers/power/pm8921-charger.c
index dbd8ba6..ec8e6b9 100644
--- a/drivers/power/pm8921-charger.c
+++ b/drivers/power/pm8921-charger.c
@@ -359,24 +359,23 @@
 						PM8921_CHG_I_MASK, temp);
 }
 
-#define PM8921_CHG_ITERM_MIN		50
-#define PM8921_CHG_ITERM_MAX		200
 #define PM8921_CHG_ITERM_MIN_MA		50
+#define PM8921_CHG_ITERM_MAX_MA		200
 #define PM8921_CHG_ITERM_STEP_MA	10
 #define PM8921_CHG_ITERM_MASK		0xF
 static int pm_chg_iterm_set(struct pm8921_chg_chip *chip, int chg_current)
 {
 	u8 temp;
 
-	if (chg_current < PM8921_CHG_ITERM_MIN
-			|| chg_current > PM8921_CHG_ITERM_MAX) {
+	if (chg_current < PM8921_CHG_ITERM_MIN_MA
+			|| chg_current > PM8921_CHG_ITERM_MAX_MA) {
 		pr_err("bad mA=%d asked to set\n", chg_current);
 		return -EINVAL;
 	}
 
 	temp = (chg_current - PM8921_CHG_ITERM_MIN_MA)
 				/ PM8921_CHG_ITERM_STEP_MA;
-	return pm_chg_masked_write(chip, CHG_IBAT_SAFE, PM8921_CHG_ITERM_MASK,
+	return pm_chg_masked_write(chip, CHG_ITERM, PM8921_CHG_ITERM_MASK,
 					 temp);
 }