Re: [PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
Hi, On Wed, Jan 11, 2017 at 09:25:01AM +1100, Chris Lapa wrote: > On 10/1/17 7:58 pm, Pali Rohár wrote: > > On Tuesday 10 January 2017 16:25:29 Chris Lapa wrote: > > > From: Chris Lapa> > > > > > The BQ275XX definition exists only to satisfy backwards compatibility. > > > > > > tested: yes > > > > Instead "tested: yes" we use: "Tested-by: name " line. > > > > > Signed-off-by: Chris Lapa > > > Acked-by: Pali Rohár > > > Reviewed-by: Andrew F. Davis > > > > Doh, I went through the log and thought I saw 'tested: yes' being used > previously. Want me to resend? If a respin is needed, please fix it. Also the patches still use "supplies" instead of "supply" in the patch subject. I would have fixed this while applying, but this (PATCH 1/10) actually looks fishy to me: > - { "bq27500", BQ27500 }, > - { "bq27510", BQ27510 }, > - { "bq27520", BQ27510 }, > + { "bq27500", BQ275XX }, > + { "bq27510", BQ275XX }, > + { "bq27520", BQ275XX }, Previously bq27500 and bq27510/bq27520 had different type ids, while after the patch both use the same. The patch description does not mention why this is ok and it actually looks incorrect. I guess we need to introduce BQ2750X and BQ2751X for backwards compatibility instead? -- Sebastian signature.asc Description: PGP signature
Re: [PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
Hi, On Wed, Jan 11, 2017 at 09:25:01AM +1100, Chris Lapa wrote: > On 10/1/17 7:58 pm, Pali Rohár wrote: > > On Tuesday 10 January 2017 16:25:29 Chris Lapa wrote: > > > From: Chris Lapa > > > > > > The BQ275XX definition exists only to satisfy backwards compatibility. > > > > > > tested: yes > > > > Instead "tested: yes" we use: "Tested-by: name " line. > > > > > Signed-off-by: Chris Lapa > > > Acked-by: Pali Rohár > > > Reviewed-by: Andrew F. Davis > > > > Doh, I went through the log and thought I saw 'tested: yes' being used > previously. Want me to resend? If a respin is needed, please fix it. Also the patches still use "supplies" instead of "supply" in the patch subject. I would have fixed this while applying, but this (PATCH 1/10) actually looks fishy to me: > - { "bq27500", BQ27500 }, > - { "bq27510", BQ27510 }, > - { "bq27520", BQ27510 }, > + { "bq27500", BQ275XX }, > + { "bq27510", BQ275XX }, > + { "bq27520", BQ275XX }, Previously bq27500 and bq27510/bq27520 had different type ids, while after the patch both use the same. The patch description does not mention why this is ok and it actually looks incorrect. I guess we need to introduce BQ2750X and BQ2751X for backwards compatibility instead? -- Sebastian signature.asc Description: PGP signature
Re: [PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
On 10/1/17 7:58 pm, Pali Rohár wrote: On Tuesday 10 January 2017 16:25:29 Chris Lapa wrote: From: Chris LapaThe BQ275XX definition exists only to satisfy backwards compatibility. tested: yes Instead "tested: yes" we use: "Tested-by: name " line. Signed-off-by: Chris Lapa Acked-by: Pali Rohár Reviewed-by: Andrew F. Davis Doh, I went through the log and thought I saw 'tested: yes' being used previously. Want me to resend? Thanks, Chris Lapa
Re: [PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
On 10/1/17 7:58 pm, Pali Rohár wrote: On Tuesday 10 January 2017 16:25:29 Chris Lapa wrote: From: Chris Lapa The BQ275XX definition exists only to satisfy backwards compatibility. tested: yes Instead "tested: yes" we use: "Tested-by: name " line. Signed-off-by: Chris Lapa Acked-by: Pali Rohár Reviewed-by: Andrew F. Davis Doh, I went through the log and thought I saw 'tested: yes' being used previously. Want me to resend? Thanks, Chris Lapa
Re: [PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
On Tuesday 10 January 2017 16:25:29 Chris Lapa wrote: > From: Chris Lapa> > The BQ275XX definition exists only to satisfy backwards compatibility. > > tested: yes Instead "tested: yes" we use: "Tested-by: name " line. > Signed-off-by: Chris Lapa > Acked-by: Pali Rohár > Reviewed-by: Andrew F. Davis -- Pali Rohár pali.ro...@gmail.com
Re: [PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
On Tuesday 10 January 2017 16:25:29 Chris Lapa wrote: > From: Chris Lapa > > The BQ275XX definition exists only to satisfy backwards compatibility. > > tested: yes Instead "tested: yes" we use: "Tested-by: name " line. > Signed-off-by: Chris Lapa > Acked-by: Pali Rohár > Reviewed-by: Andrew F. Davis -- Pali Rohár pali.ro...@gmail.com
[PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
From: Chris LapaThe BQ275XX definition exists only to satisfy backwards compatibility. tested: yes Signed-off-by: Chris Lapa Acked-by: Pali Rohár Reviewed-by: Andrew F. Davis --- drivers/power/supply/bq27xxx_battery.c | 10 -- drivers/power/supply/bq27xxx_battery_i2c.c | 6 +++--- include/linux/power/bq27xxx_battery.h | 3 +-- 3 files changed, 8 insertions(+), 11 deletions(-) diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c index 7272d1e..a86417c 100644 --- a/drivers/power/supply/bq27xxx_battery.c +++ b/drivers/power/supply/bq27xxx_battery.c @@ -145,7 +145,7 @@ static u8 bq27xxx_regs[][BQ27XXX_REG_MAX] = { [BQ27XXX_REG_DCAP] = 0x76, [BQ27XXX_REG_AP] = INVALID_REG_ADDR, }, - [BQ27500] = { + [BQ275XX] = { [BQ27XXX_REG_CTRL] = 0x00, [BQ27XXX_REG_TEMP] = 0x06, [BQ27XXX_REG_INT_TEMP] = 0x28, @@ -303,7 +303,7 @@ static enum power_supply_property bq27010_battery_props[] = { POWER_SUPPLY_PROP_MANUFACTURER, }; -static enum power_supply_property bq27500_battery_props[] = { +static enum power_supply_property bq275xx_battery_props[] = { POWER_SUPPLY_PROP_STATUS, POWER_SUPPLY_PROP_PRESENT, POWER_SUPPLY_PROP_VOLTAGE_NOW, @@ -421,8 +421,7 @@ static struct { } bq27xxx_battery_props[] = { BQ27XXX_PROP(BQ27000, bq27000_battery_props), BQ27XXX_PROP(BQ27010, bq27010_battery_props), - BQ27XXX_PROP(BQ27500, bq27500_battery_props), - BQ27XXX_PROP(BQ27510, bq27510_battery_props), + BQ27XXX_PROP(BQ275XX, bq275xx_battery_props), BQ27XXX_PROP(BQ27530, bq27530_battery_props), BQ27XXX_PROP(BQ27541, bq27541_battery_props), BQ27XXX_PROP(BQ27545, bq27545_battery_props), @@ -675,8 +674,7 @@ static int bq27xxx_battery_read_pwr_avg(struct bq27xxx_device_info *di) static bool bq27xxx_battery_overtemp(struct bq27xxx_device_info *di, u16 flags) { switch (di->chip) { - case BQ27500: - case BQ27510: + case BQ275XX: case BQ27541: case BQ27545: return flags & (BQ27XXX_FLAG_OTC | BQ27XXX_FLAG_OTD); diff --git a/drivers/power/supply/bq27xxx_battery_i2c.c b/drivers/power/supply/bq27xxx_battery_i2c.c index 5c5c3a6..762d96e 100644 --- a/drivers/power/supply/bq27xxx_battery_i2c.c +++ b/drivers/power/supply/bq27xxx_battery_i2c.c @@ -148,9 +148,9 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client) static const struct i2c_device_id bq27xxx_i2c_id_table[] = { { "bq27200", BQ27000 }, { "bq27210", BQ27010 }, - { "bq27500", BQ27500 }, - { "bq27510", BQ27510 }, - { "bq27520", BQ27510 }, + { "bq27500", BQ275XX }, + { "bq27510", BQ275XX }, + { "bq27520", BQ275XX }, { "bq27530", BQ27530 }, { "bq27531", BQ27530 }, { "bq27541", BQ27541 }, diff --git a/include/linux/power/bq27xxx_battery.h b/include/linux/power/bq27xxx_battery.h index bed9557..c452b94 100644 --- a/include/linux/power/bq27xxx_battery.h +++ b/include/linux/power/bq27xxx_battery.h @@ -4,8 +4,7 @@ enum bq27xxx_chip { BQ27000 = 1, /* bq27000, bq27200 */ BQ27010, /* bq27010, bq27210 */ - BQ27500, /* bq27500 */ - BQ27510, /* bq27510, bq27520 */ + BQ275XX, /* bq27500, bq27510, bq27520 deprecated alias */ BQ27530, /* bq27530, bq27531 */ BQ27541, /* bq27541, bq27542, bq27546, bq27742 */ BQ27545, /* bq27545 */ -- 2.1.4
[PATCH v4 01/10] power: supplies: bq27xxx: rename BQ27500 allow for deprecation in future.
From: Chris Lapa The BQ275XX definition exists only to satisfy backwards compatibility. tested: yes Signed-off-by: Chris Lapa Acked-by: Pali Rohár Reviewed-by: Andrew F. Davis --- drivers/power/supply/bq27xxx_battery.c | 10 -- drivers/power/supply/bq27xxx_battery_i2c.c | 6 +++--- include/linux/power/bq27xxx_battery.h | 3 +-- 3 files changed, 8 insertions(+), 11 deletions(-) diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c index 7272d1e..a86417c 100644 --- a/drivers/power/supply/bq27xxx_battery.c +++ b/drivers/power/supply/bq27xxx_battery.c @@ -145,7 +145,7 @@ static u8 bq27xxx_regs[][BQ27XXX_REG_MAX] = { [BQ27XXX_REG_DCAP] = 0x76, [BQ27XXX_REG_AP] = INVALID_REG_ADDR, }, - [BQ27500] = { + [BQ275XX] = { [BQ27XXX_REG_CTRL] = 0x00, [BQ27XXX_REG_TEMP] = 0x06, [BQ27XXX_REG_INT_TEMP] = 0x28, @@ -303,7 +303,7 @@ static enum power_supply_property bq27010_battery_props[] = { POWER_SUPPLY_PROP_MANUFACTURER, }; -static enum power_supply_property bq27500_battery_props[] = { +static enum power_supply_property bq275xx_battery_props[] = { POWER_SUPPLY_PROP_STATUS, POWER_SUPPLY_PROP_PRESENT, POWER_SUPPLY_PROP_VOLTAGE_NOW, @@ -421,8 +421,7 @@ static struct { } bq27xxx_battery_props[] = { BQ27XXX_PROP(BQ27000, bq27000_battery_props), BQ27XXX_PROP(BQ27010, bq27010_battery_props), - BQ27XXX_PROP(BQ27500, bq27500_battery_props), - BQ27XXX_PROP(BQ27510, bq27510_battery_props), + BQ27XXX_PROP(BQ275XX, bq275xx_battery_props), BQ27XXX_PROP(BQ27530, bq27530_battery_props), BQ27XXX_PROP(BQ27541, bq27541_battery_props), BQ27XXX_PROP(BQ27545, bq27545_battery_props), @@ -675,8 +674,7 @@ static int bq27xxx_battery_read_pwr_avg(struct bq27xxx_device_info *di) static bool bq27xxx_battery_overtemp(struct bq27xxx_device_info *di, u16 flags) { switch (di->chip) { - case BQ27500: - case BQ27510: + case BQ275XX: case BQ27541: case BQ27545: return flags & (BQ27XXX_FLAG_OTC | BQ27XXX_FLAG_OTD); diff --git a/drivers/power/supply/bq27xxx_battery_i2c.c b/drivers/power/supply/bq27xxx_battery_i2c.c index 5c5c3a6..762d96e 100644 --- a/drivers/power/supply/bq27xxx_battery_i2c.c +++ b/drivers/power/supply/bq27xxx_battery_i2c.c @@ -148,9 +148,9 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client) static const struct i2c_device_id bq27xxx_i2c_id_table[] = { { "bq27200", BQ27000 }, { "bq27210", BQ27010 }, - { "bq27500", BQ27500 }, - { "bq27510", BQ27510 }, - { "bq27520", BQ27510 }, + { "bq27500", BQ275XX }, + { "bq27510", BQ275XX }, + { "bq27520", BQ275XX }, { "bq27530", BQ27530 }, { "bq27531", BQ27530 }, { "bq27541", BQ27541 }, diff --git a/include/linux/power/bq27xxx_battery.h b/include/linux/power/bq27xxx_battery.h index bed9557..c452b94 100644 --- a/include/linux/power/bq27xxx_battery.h +++ b/include/linux/power/bq27xxx_battery.h @@ -4,8 +4,7 @@ enum bq27xxx_chip { BQ27000 = 1, /* bq27000, bq27200 */ BQ27010, /* bq27010, bq27210 */ - BQ27500, /* bq27500 */ - BQ27510, /* bq27510, bq27520 */ + BQ275XX, /* bq27500, bq27510, bq27520 deprecated alias */ BQ27530, /* bq27530, bq27531 */ BQ27541, /* bq27541, bq27542, bq27546, bq27742 */ BQ27545, /* bq27545 */ -- 2.1.4