From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757092Ab2AXR1m (ORCPT ); Tue, 24 Jan 2012 12:27:42 -0500 Received: from mail-vw0-f46.google.com ([209.85.212.46]:52047 "EHLO mail-vw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757022Ab2AXR1k (ORCPT ); Tue, 24 Jan 2012 12:27:40 -0500 From: dirk.brandewie@gmail.com To: linux-kernel@vger.kernel.org Cc: cbouatmailru@gmail.com, dg77.kim@samsung.com, kyungmin.park@samsung.com, myungjoo.ham@samsung.com, Jason.Wortham@maxim-ic.com, bruce.e.robertson@intel.com, Dirk Brandewie Subject: [PATCH 4/5] max17042: Fix value scaling for VCELL and avgVCELL Date: Tue, 24 Jan 2012 09:26:07 -0800 Message-Id: <1327425968-21195-5-git-send-email-dirk.brandewie@gmail.com> X-Mailer: git-send-email 1.7.7.5 In-Reply-To: <1327425968-21195-1-git-send-email-dirk.brandewie@gmail.com> References: <1327425968-21195-1-git-send-email-dirk.brandewie@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Bruce Robertson The bottom three bits of the register are don't care bits. The LSB value is 625 uV. Adjust the returned values appropriately Signed-off-by: Bruce Robertson Signed-off-by: Dirk Brandewie Acked-by: MyungJoo Ham --- drivers/power/max17042_battery.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/power/max17042_battery.c b/drivers/power/max17042_battery.c index e0a4430..6e96b58 100644 --- a/drivers/power/max17042_battery.c +++ b/drivers/power/max17042_battery.c @@ -148,11 +148,15 @@ static int max17042_get_property(struct power_supply *psy, break; case POWER_SUPPLY_PROP_VOLTAGE_NOW: val->intval = max17042_read_reg(chip->client, - MAX17042_VCELL) * 83; /* 1000 / 12 = 83 */ + MAX17042_VCELL); + val->intval >>= 3; + val->intval *= 625; /* Units of LSB = 625 uV */ break; case POWER_SUPPLY_PROP_VOLTAGE_AVG: val->intval = max17042_read_reg(chip->client, - MAX17042_AvgVCELL) * 83; + MAX17042_AvgVCELL); + val->intval >>= 3; + val->intval *= 625; /* Units of LSB = 625 uV */ break; case POWER_SUPPLY_PROP_CAPACITY: val->intval = max17042_read_reg(chip->client, -- 1.7.7.5