From: MyungJoo Ham Date: Wed, 14 Mar 2012 02:00:01 +0000 (+0100) Subject: max17042_battery: Fix CHARGE_FULL representation. X-Git-Tag: v3.4-rc1~20^2~3 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6e0e60cd0d688d0d1af85ef2abb8e363595af988;p=platform%2Fupstream%2Fkernel-adaptation-pc.git max17042_battery: Fix CHARGE_FULL representation. CHARGE_FULL should represent the full capacity of the battery in uAh. The 0x10 (FullCAP) register shows the compensated full capacity in mAh * 2; e.g., reg(0x10) = 0xBB8 means that it is 1500mAh. Signed-off-by: MyungJoo Ham Signed-off-by: Anton Vorontsov --- diff --git a/drivers/power/max17042_battery.c b/drivers/power/max17042_battery.c index 5474e76..d576912 100644 --- a/drivers/power/max17042_battery.c +++ b/drivers/power/max17042_battery.c @@ -179,14 +179,11 @@ static int max17042_get_property(struct power_supply *psy, val->intval = ret >> 8; break; case POWER_SUPPLY_PROP_CHARGE_FULL: - ret = max17042_read_reg(chip->client, MAX17042_RepSOC); + ret = max17042_read_reg(chip->client, MAX17042_FullCAP); if (ret < 0) return ret; - if ((ret >> 8) >= MAX17042_BATTERY_FULL) - val->intval = 1; - else if (ret >= 0) - val->intval = 0; + val->intval = ret * 1000 / 2; break; case POWER_SUPPLY_PROP_TEMP: ret = max17042_read_reg(chip->client, MAX17042_TEMP);