From: Ramakrishna Pallala <ramakrishna.pallala@intel.com>
Date: Fri, 22 Jun 2012 11:14:50 +0000 (+0530)
Subject: MFD/Battery: Fix battery status setting logic for interrupt case
X-Git-Tag: 2.1b_release~558
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=55a1a8e8d0d0f1cea22f0d5e23bbe75dfcbf7056;p=kernel%2Fkernel-mfld-blackbay.git

MFD/Battery: Fix battery status setting logic for interrupt case

BZ: 38168

Right we are setting the battery status to Not Chargring upon certain
interrutps without properly(really) checking the charger fault condition.

This patch will fix this issue by making sure that we check charger
fault case AND interrupt bits.

Change-Id: Ia41c5820dd616ca295a0421f8c3894828a173892
Signed-off-by: Ramakrishna Pallala <ramakrishna.pallala@intel.com>
Reviewed-on: http://android.intel.com:8080/53736
Reviewed-by: Tc, Jenny <jenny.tc@intel.com>
Reviewed-by: Mukherjee, DevapriyaX <devapriyax.mukherjee@intel.com>
Reviewed-by: Jena, TapanX <tapanx.jena@intel.com>
Tested-by: Kallappa Manjanna, MadhukumarX <madhukumarx.kallappa.manjanna@intel.com>
Reviewed-by: buildbot <buildbot@intel.com>
Tested-by: buildbot <buildbot@intel.com>
---

diff --git a/drivers/power/intel_mdf_battery.c b/drivers/power/intel_mdf_battery.c
index 5b8033a..8440d24 100644
--- a/drivers/power/intel_mdf_battery.c
+++ b/drivers/power/intel_mdf_battery.c
@@ -2301,8 +2301,8 @@ static irqreturn_t msic_battery_thread_handler(int id, void *dev)
 	}
 
 	/* Check charger Status bits */
-	if ((data[0] & ~(MSIC_BATT_CHR_TIMEEXP_MASK)) ||
-		(data[1] & ~(MSIC_BATT_CHR_CHRCMPLT_MASK))
+	if (((data[0] & ~(MSIC_BATT_CHR_TIMEEXP_MASK)) ||
+		(data[1] & ~(MSIC_BATT_CHR_CHRCMPLT_MASK)))
 			&& is_charger_fault()) {
 		mutex_lock(&mbi->batt_lock);
 		mbi->batt_props.status = POWER_SUPPLY_STATUS_NOT_CHARGING;