commit d3c435e37b1c1b2bd46c92177872a616c9709931
author Abhijeet Dharmapurikar <adharmap@codeaurora.org> Thu Aug 16 19:23:18 2012 -0700
committer Abhijeet Dharmapurikar <adharmap@codeaurora.org> Thu Aug 16 19:28:30 2012 -0700
tree 33bbaa705072e59ce80350e162ef6024340b71c7
parent 37d554a44e05e5cf9c33ac6b2376b5e6ea941a17

power: pm8921-bms: fix 0% soc reported when started with a charger

When the device is started with a charger inserted, it reports 0% soc.
This is because a variable "soc_at_cv" is not initialized correctly in
the probe function.

Fix this by initializing it appropriately to -EINVAL.

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

drivers/power/pm8921-bms.c

diff --git a/drivers/power/pm8921-bms.c b/drivers/power/pm8921-bms.c
index bd6af0c..5e1b77a 100644
--- a/drivers/power/pm8921-bms.c
+++ b/drivers/power/pm8921-bms.c
@@ -1615,7 +1615,8 @@
 	 * keep reporting the prev chg soc
 	 */
 	if (vbat_uv <= chip->max_voltage_uv) {
-		pr_debug("CC CHG SOC %d\n", chip->prev_chg_soc);
+		pr_debug("vbat %d < max = %d CC CHG SOC %d\n",
+			vbat_uv, chip->max_voltage_uv, chip->prev_chg_soc);
 		return chip->prev_chg_soc;
 	}
 
@@ -1640,7 +1641,7 @@
 				chip->prev_chg_soc);
 	}
 
-	pr_debug("CHG SOC %d\n", chip->prev_chg_soc);
+	pr_debug("Reporting CHG SOC %d\n", chip->prev_chg_soc);
 	return chip->prev_chg_soc;
 }
 
@@ -3170,6 +3171,7 @@
 		chip->adjust_soc_low_threshold = 45;
 
 	chip->prev_pc_unusable = -EINVAL;
+	chip->soc_at_cv = -EINVAL;
 
 	chip->ignore_shutdown_soc = pdata->ignore_shutdown_soc;
 	rc = set_battery_data(chip);