[PATCH] max17042_battery: Support CHARGE_COUNTER power supply attribute

[PATCH] max17042_battery: Support CHARGE_COUNTER power supply attribute

[Ramakrishna Pallala]

This patch adds the support for CHARGE_COUNTER power supply
attribute to max17042/47 driver.

Note:QH(Charge Counter) register is not documented in max17042 the Spec.

Signed-off-by: Ramakrishna Pallala <ramakrishna.pallala@intel.com>
---
 drivers/power/max17042_battery.c |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/drivers/power/max17042_battery.c b/drivers/power/max17042_battery.c
index 140788b..74abc6c 100644
--- a/drivers/power/max17042_battery.c
+++ b/drivers/power/max17042_battery.c
@@ -113,6 +113,7 @@ static enum power_supply_property max17042_battery_props[] = {
 	POWER_SUPPLY_PROP_VOLTAGE_OCV,
 	POWER_SUPPLY_PROP_CAPACITY,
 	POWER_SUPPLY_PROP_CHARGE_FULL,
+	POWER_SUPPLY_PROP_CHARGE_COUNTER,
 	POWER_SUPPLY_PROP_TEMP,
 	POWER_SUPPLY_PROP_CURRENT_NOW,
 	POWER_SUPPLY_PROP_CURRENT_AVG,
@@ -201,6 +202,13 @@ static int max17042_get_property(struct power_supply *psy,
 
 		val->intval = ret * 1000 / 2;
 		break;
+	case POWER_SUPPLY_PROP_CHARGE_COUNTER:
+		ret = max17042_read_reg(chip->client, MAX17042_QH);
+		if (ret < 0)
+			return ret;
+
+		val->intval = ret * 1000 / 2;
+		break;
 	case POWER_SUPPLY_PROP_TEMP:
 		ret = max17042_read_reg(chip->client, MAX17042_TEMP);
 		if (ret < 0)
-- 
1.7.0.4

RE: [PATCH] max17042_battery: Support CHARGE_COUNTER power supply attribute

[Pallala, Ramakrishna]

Hi Anton,

> Sent: Monday, May 07, 2012 10:26 AM
> To: linux-kernel@vger.kernel.org
> Cc: Anton Vorontsov; Anton Vorontsov; Pallala, Ramakrishna
> Subject: [PATCH] max17042_battery: Support CHARGE_COUNTER power supply
> attribute
> 
> This patch adds the support for CHARGE_COUNTER power supply attribute to
> max17042/47 driver.
> 
> Note:QH(Charge Counter) register is not documented in max17042 the Spec.
> 
> Signed-off-by: Ramakrishna Pallala <ramakrishna.pallala@intel.com>
> ---
>  drivers/power/max17042_battery.c |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
> 

I am waiting for the comments or ACK from you on this patch.

Thanks,
Ram

Re: [PATCH] max17042_battery: Support CHARGE_COUNTER power supply attribute

[Anton Vorontsov]

On Mon, May 07, 2012 at 10:25:58AM +0530, Ramakrishna Pallala wrote:
> This patch adds the support for CHARGE_COUNTER power supply
> attribute to max17042/47 driver.
> 
> Note:QH(Charge Counter) register is not documented in max17042 the Spec.
> 
> Signed-off-by: Ramakrishna Pallala <ramakrishna.pallala@intel.com>
> ---

Applied, thank you!

-- 
Anton Vorontsov
Email: cbouatmailru@gmail.com