From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Looijmans Subject: Re: [PATCH 1/2] power: supply: ltc2941-battery-gauge: Define LTC2942 registers Date: Mon, 12 Jun 2017 18:26:17 +0200 Message-ID: <8d97cccb-fef7-428d-b075-b0e59a590d80@topic.nl> References: <20170612150226.woxfdi4lharq5gtq@lenoch> <20170612150356.wvehmijba6xbdwvb@lenoch> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8BIT Return-path: Received: from mx6-05.smtp.antispamcloud.com ([95.211.2.196]:42991 "EHLO mx6-05.smtp.antispamcloud.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753065AbdFLQ4W (ORCPT ); Mon, 12 Jun 2017 12:56:22 -0400 In-Reply-To: <20170612150356.wvehmijba6xbdwvb@lenoch> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Ladislav Michl , linux-pm@vger.kernel.org Cc: Javier Martinez Canillas See below for my comment... On 12-6-2017 17:03, Ladislav Michl wrote: > Add LTC2942 registers definitions. No functional changes. > > Signed-off-by: Ladislav Michl > --- > drivers/power/supply/ltc2941-battery-gauge.c | 27 +++++++++++++-------------- > 1 file changed, 13 insertions(+), 14 deletions(-) > > diff --git a/drivers/power/supply/ltc2941-battery-gauge.c b/drivers/power/supply/ltc2941-battery-gauge.c > index 7efb908f4451..0a1b69bbca7f 100644 > --- a/drivers/power/supply/ltc2941-battery-gauge.c > +++ b/drivers/power/supply/ltc2941-battery-gauge.c > @@ -34,16 +34,14 @@ enum ltc294x_reg { > LTC294X_REG_CONTROL = 0x01, > LTC294X_REG_ACC_CHARGE_MSB = 0x02, > LTC294X_REG_ACC_CHARGE_LSB = 0x03, > - LTC294X_REG_THRESH_HIGH_MSB = 0x04, > - LTC294X_REG_THRESH_HIGH_LSB = 0x05, > - LTC294X_REG_THRESH_LOW_MSB = 0x06, > - LTC294X_REG_THRESH_LOW_LSB = 0x07, > - LTC294X_REG_VOLTAGE_MSB = 0x08, > - LTC294X_REG_VOLTAGE_LSB = 0x09, > - LTC294X_REG_CURRENT_MSB = 0x0E, > - LTC294X_REG_CURRENT_LSB = 0x0F, > - LTC294X_REG_TEMPERATURE_MSB = 0x14, > - LTC294X_REG_TEMPERATURE_LSB = 0x15, > + LTC294X_REG_VOLTAGE_MSB = 0x08, > + LTC294X_REG_VOLTAGE_LSB = 0x09, > + LTC2942_REG_TEMPERATURE_MSB = 0x0C, > + LTC2942_REG_TEMPERATURE_LSB = 0x0D, > + LTC2943_REG_CURRENT_MSB = 0x0E, > + LTC2943_REG_CURRENT_LSB = 0x0F, > + LTC2943_REG_TEMPERATURE_MSB = 0x14, > + LTC2943_REG_TEMPERATURE_LSB = 0x15, > }; I don't see the advantage of renaming these, I'd rather see the LTC294X naming throughout, making it easier to locate them in the enum as they all share the exact same prefix. But if you do want this renaming, I guess you should also rename the LTC2941 registers. I'm neutral to either naming, linux-pm maintainer will have the final word I guess. > > #define LTC2943_REG_CONTROL_MODE_MASK (BIT(7) | BIT(6)) > @@ -55,6 +53,7 @@ enum ltc294x_reg { > #define LTC294X_REG_CONTROL_ALCC_CONFIG_DISABLED 0 > > #define LTC2941_NUM_REGS 0x08 > +#define LTC2942_NUM_REGS 0x10 > #define LTC2943_NUM_REGS 0x18 > > struct ltc294x_info { > @@ -263,7 +262,7 @@ static int ltc294x_get_current(const struct ltc294x_info *info, int *val) > s32 value; > > ret = ltc294x_read_regs(info->client, > - LTC294X_REG_CURRENT_MSB, &datar[0], 2); > + LTC2943_REG_CURRENT_MSB, &datar[0], 2); > value = (datar[0] << 8) | datar[1]; > value -= 0x7FFF; > /* Value is in range -32k..+32k, r_sense is usually 10..50 mOhm, > @@ -280,7 +279,7 @@ static int ltc294x_get_temperature(const struct ltc294x_info *info, int *val) > u32 value; > > ret = ltc294x_read_regs(info->client, > - LTC294X_REG_TEMPERATURE_MSB, &datar[0], 2); > + LTC2943_REG_TEMPERATURE_MSB, &datar[0], 2); > value = (datar[0] << 8) | datar[1]; > /* Full-scale is 510 Kelvin, convert to centidegrees */ > *val = (((51000 * value) / 0xFFFF) - 27215); > @@ -424,10 +423,10 @@ static int ltc294x_i2c_probe(struct i2c_client *client, > info->client = client; > info->supply_desc.type = POWER_SUPPLY_TYPE_BATTERY; > info->supply_desc.properties = ltc294x_properties; > - if (info->num_regs >= LTC294X_REG_TEMPERATURE_LSB) > + if (info->num_regs >= LTC2943_REG_TEMPERATURE_LSB) > info->supply_desc.num_properties = > ARRAY_SIZE(ltc294x_properties); > - else if (info->num_regs >= LTC294X_REG_CURRENT_LSB) > + else if (info->num_regs >= LTC2943_REG_CURRENT_LSB) > info->supply_desc.num_properties = > ARRAY_SIZE(ltc294x_properties) - 1; > else if (info->num_regs >= LTC294X_REG_VOLTAGE_LSB) > -- Mike Looijmans Kind regards, Mike Looijmans System Expert TOPIC Products Materiaalweg 4, NL-5681 RJ Best Postbus 440, NL-5680 AK Best Telefoon: +31 (0) 499 33 69 79 E-mail: mike.looijmans@topicproducts.com Website: www.topicproducts.com Please consider the environment before printing this e-mail