* [PATCH] power: supply: bq27xxx: Add support for BQ27411
@ 2018-07-22 13:33 Liu Xiang
2018-09-16 11:35 ` Sebastian Reichel
0 siblings, 1 reply; 4+ messages in thread
From: Liu Xiang @ 2018-07-22 13:33 UTC (permalink / raw)
To: sre; +Cc: pali.rohar, afd, linux-pm, linux-kernel, liuxiang_1999, Liu Xiang
According to the datasheet, bq27411 is similar to bq27421.
Signed-off-by: Liu Xiang <liu.xiang6@zte.com.cn>
---
Documentation/devicetree/bindings/power/supply/bq27xxx.txt | 1 +
drivers/power/supply/bq27xxx_battery.c | 9 +++++++++
drivers/power/supply/bq27xxx_battery_i2c.c | 2 ++
include/linux/power/bq27xxx_battery.h | 1 +
4 files changed, 13 insertions(+)
diff --git a/Documentation/devicetree/bindings/power/supply/bq27xxx.txt b/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
index 37994fd..4fa8e08 100644
--- a/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
+++ b/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
@@ -23,6 +23,7 @@ Required properties:
* "ti,bq27546" - BQ27546
* "ti,bq27742" - BQ27742
* "ti,bq27545" - BQ27545
+ * "ti,bq27411" - BQ27411
* "ti,bq27421" - BQ27421
* "ti,bq27425" - BQ27425
* "ti,bq27426" - BQ27426
diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
index f022e1b..6dbbe95 100644
--- a/drivers/power/supply/bq27xxx_battery.c
+++ b/drivers/power/supply/bq27xxx_battery.c
@@ -432,6 +432,7 @@ enum bq27xxx_reg_index {
[BQ27XXX_REG_AP] = 0x18,
BQ27XXX_DM_REG_ROWS,
};
+#define bq27411_regs bq27421_regs
#define bq27425_regs bq27421_regs
#define bq27426_regs bq27421_regs
#define bq27441_regs bq27421_regs
@@ -665,6 +666,7 @@ enum bq27xxx_reg_index {
POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
POWER_SUPPLY_PROP_MANUFACTURER,
};
+#define bq27411_props bq27421_props
#define bq27425_props bq27421_props
#define bq27426_props bq27421_props
#define bq27441_props bq27421_props
@@ -725,6 +727,12 @@ enum bq27xxx_dm_reg_id {
#define bq27545_dm_regs 0
#endif
+static struct bq27xxx_dm_reg bq27411_dm_regs[] = {
+ [BQ27XXX_DM_DESIGN_CAPACITY] = { 82, 10, 2, 0, 32767 },
+ [BQ27XXX_DM_DESIGN_ENERGY] = { 82, 12, 2, 0, 32767 },
+ [BQ27XXX_DM_TERMINATE_VOLTAGE] = { 82, 16, 2, 2800, 3700 },
+};
+
static struct bq27xxx_dm_reg bq27421_dm_regs[] = {
[BQ27XXX_DM_DESIGN_CAPACITY] = { 82, 10, 2, 0, 8000 },
[BQ27XXX_DM_DESIGN_ENERGY] = { 82, 12, 2, 0, 32767 },
@@ -802,6 +810,7 @@ enum bq27xxx_dm_reg_id {
[BQ27546] = BQ27XXX_DATA(bq27546, 0 , BQ27XXX_O_OTDC),
[BQ27742] = BQ27XXX_DATA(bq27742, 0 , BQ27XXX_O_OTDC),
[BQ27545] = BQ27XXX_DATA(bq27545, 0x04143672, BQ27XXX_O_OTDC),
+ [BQ27411] = BQ27XXX_DATA(bq27411, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
[BQ27421] = BQ27XXX_DATA(bq27421, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
[BQ27425] = BQ27XXX_DATA(bq27425, 0x04143672, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP),
[BQ27426] = BQ27XXX_DATA(bq27426, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
diff --git a/drivers/power/supply/bq27xxx_battery_i2c.c b/drivers/power/supply/bq27xxx_battery_i2c.c
index 4006912..2677c38 100644
--- a/drivers/power/supply/bq27xxx_battery_i2c.c
+++ b/drivers/power/supply/bq27xxx_battery_i2c.c
@@ -247,6 +247,7 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
{ "bq27546", BQ27546 },
{ "bq27742", BQ27742 },
{ "bq27545", BQ27545 },
+ { "bq27411", BQ27411 },
{ "bq27421", BQ27421 },
{ "bq27425", BQ27425 },
{ "bq27426", BQ27426 },
@@ -279,6 +280,7 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
{ .compatible = "ti,bq27546" },
{ .compatible = "ti,bq27742" },
{ .compatible = "ti,bq27545" },
+ { .compatible = "ti,bq27411" },
{ .compatible = "ti,bq27421" },
{ .compatible = "ti,bq27425" },
{ .compatible = "ti,bq27426" },
diff --git a/include/linux/power/bq27xxx_battery.h b/include/linux/power/bq27xxx_battery.h
index d6355f4..507c5e2 100644
--- a/include/linux/power/bq27xxx_battery.h
+++ b/include/linux/power/bq27xxx_battery.h
@@ -24,6 +24,7 @@ enum bq27xxx_chip {
BQ27546,
BQ27742,
BQ27545, /* bq27545 */
+ BQ27411,
BQ27421, /* bq27421, bq27441, bq27621 */
BQ27425,
BQ27426,
--
1.9.1
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] power: supply: bq27xxx: Add support for BQ27411
2018-07-22 13:33 [PATCH] power: supply: bq27xxx: Add support for BQ27411 Liu Xiang
@ 2018-09-16 11:35 ` Sebastian Reichel
2018-09-17 20:37 ` Andrew F. Davis
0 siblings, 1 reply; 4+ messages in thread
From: Sebastian Reichel @ 2018-09-16 11:35 UTC (permalink / raw)
To: Liu Xiang, afd; +Cc: pali.rohar, linux-pm, linux-kernel, liuxiang_1999
[-- Attachment #1: Type: text/plain, Size: 4511 bytes --]
Hi,
On Sun, Jul 22, 2018 at 09:33:56PM +0800, Liu Xiang wrote:
> According to the datasheet, bq27411 is similar to bq27421.
>
> Signed-off-by: Liu Xiang <liu.xiang6@zte.com.cn>
> ---
Looks good to me. Do you plan to test/review this Andrew, or
should I proceed with merging this patch?
-- Sebastian
> Documentation/devicetree/bindings/power/supply/bq27xxx.txt | 1 +
> drivers/power/supply/bq27xxx_battery.c | 9 +++++++++
> drivers/power/supply/bq27xxx_battery_i2c.c | 2 ++
> include/linux/power/bq27xxx_battery.h | 1 +
> 4 files changed, 13 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/power/supply/bq27xxx.txt b/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
> index 37994fd..4fa8e08 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
> +++ b/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
> @@ -23,6 +23,7 @@ Required properties:
> * "ti,bq27546" - BQ27546
> * "ti,bq27742" - BQ27742
> * "ti,bq27545" - BQ27545
> + * "ti,bq27411" - BQ27411
> * "ti,bq27421" - BQ27421
> * "ti,bq27425" - BQ27425
> * "ti,bq27426" - BQ27426
> diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
> index f022e1b..6dbbe95 100644
> --- a/drivers/power/supply/bq27xxx_battery.c
> +++ b/drivers/power/supply/bq27xxx_battery.c
> @@ -432,6 +432,7 @@ enum bq27xxx_reg_index {
> [BQ27XXX_REG_AP] = 0x18,
> BQ27XXX_DM_REG_ROWS,
> };
> +#define bq27411_regs bq27421_regs
> #define bq27425_regs bq27421_regs
> #define bq27426_regs bq27421_regs
> #define bq27441_regs bq27421_regs
> @@ -665,6 +666,7 @@ enum bq27xxx_reg_index {
> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
> POWER_SUPPLY_PROP_MANUFACTURER,
> };
> +#define bq27411_props bq27421_props
> #define bq27425_props bq27421_props
> #define bq27426_props bq27421_props
> #define bq27441_props bq27421_props
> @@ -725,6 +727,12 @@ enum bq27xxx_dm_reg_id {
> #define bq27545_dm_regs 0
> #endif
>
> +static struct bq27xxx_dm_reg bq27411_dm_regs[] = {
> + [BQ27XXX_DM_DESIGN_CAPACITY] = { 82, 10, 2, 0, 32767 },
> + [BQ27XXX_DM_DESIGN_ENERGY] = { 82, 12, 2, 0, 32767 },
> + [BQ27XXX_DM_TERMINATE_VOLTAGE] = { 82, 16, 2, 2800, 3700 },
> +};
> +
> static struct bq27xxx_dm_reg bq27421_dm_regs[] = {
> [BQ27XXX_DM_DESIGN_CAPACITY] = { 82, 10, 2, 0, 8000 },
> [BQ27XXX_DM_DESIGN_ENERGY] = { 82, 12, 2, 0, 32767 },
> @@ -802,6 +810,7 @@ enum bq27xxx_dm_reg_id {
> [BQ27546] = BQ27XXX_DATA(bq27546, 0 , BQ27XXX_O_OTDC),
> [BQ27742] = BQ27XXX_DATA(bq27742, 0 , BQ27XXX_O_OTDC),
> [BQ27545] = BQ27XXX_DATA(bq27545, 0x04143672, BQ27XXX_O_OTDC),
> + [BQ27411] = BQ27XXX_DATA(bq27411, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
> [BQ27421] = BQ27XXX_DATA(bq27421, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
> [BQ27425] = BQ27XXX_DATA(bq27425, 0x04143672, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP),
> [BQ27426] = BQ27XXX_DATA(bq27426, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
> diff --git a/drivers/power/supply/bq27xxx_battery_i2c.c b/drivers/power/supply/bq27xxx_battery_i2c.c
> index 4006912..2677c38 100644
> --- a/drivers/power/supply/bq27xxx_battery_i2c.c
> +++ b/drivers/power/supply/bq27xxx_battery_i2c.c
> @@ -247,6 +247,7 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
> { "bq27546", BQ27546 },
> { "bq27742", BQ27742 },
> { "bq27545", BQ27545 },
> + { "bq27411", BQ27411 },
> { "bq27421", BQ27421 },
> { "bq27425", BQ27425 },
> { "bq27426", BQ27426 },
> @@ -279,6 +280,7 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
> { .compatible = "ti,bq27546" },
> { .compatible = "ti,bq27742" },
> { .compatible = "ti,bq27545" },
> + { .compatible = "ti,bq27411" },
> { .compatible = "ti,bq27421" },
> { .compatible = "ti,bq27425" },
> { .compatible = "ti,bq27426" },
> diff --git a/include/linux/power/bq27xxx_battery.h b/include/linux/power/bq27xxx_battery.h
> index d6355f4..507c5e2 100644
> --- a/include/linux/power/bq27xxx_battery.h
> +++ b/include/linux/power/bq27xxx_battery.h
> @@ -24,6 +24,7 @@ enum bq27xxx_chip {
> BQ27546,
> BQ27742,
> BQ27545, /* bq27545 */
> + BQ27411,
> BQ27421, /* bq27421, bq27441, bq27621 */
> BQ27425,
> BQ27426,
> --
> 1.9.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] power: supply: bq27xxx: Add support for BQ27411
2018-09-16 11:35 ` Sebastian Reichel
@ 2018-09-17 20:37 ` Andrew F. Davis
2018-09-20 9:38 ` Sebastian Reichel
0 siblings, 1 reply; 4+ messages in thread
From: Andrew F. Davis @ 2018-09-17 20:37 UTC (permalink / raw)
To: Sebastian Reichel, Liu Xiang
Cc: pali.rohar, linux-pm, linux-kernel, liuxiang_1999
On 09/16/2018 06:35 AM, Sebastian Reichel wrote:
> Hi,
>
> On Sun, Jul 22, 2018 at 09:33:56PM +0800, Liu Xiang wrote:
>> According to the datasheet, bq27411 is similar to bq27421.
>>
>> Signed-off-by: Liu Xiang <liu.xiang6@zte.com.cn>
>> ---
>
> Looks good to me. Do you plan to test/review this Andrew, or
> should I proceed with merging this patch?
>
I do not have this particular hardware for testing, but the patch does
look correct,
Reviewed-by: Andrew F. Davis <afd@ti.com>
> -- Sebastian
>
>> Documentation/devicetree/bindings/power/supply/bq27xxx.txt | 1 +
>> drivers/power/supply/bq27xxx_battery.c | 9 +++++++++
>> drivers/power/supply/bq27xxx_battery_i2c.c | 2 ++
>> include/linux/power/bq27xxx_battery.h | 1 +
>> 4 files changed, 13 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/power/supply/bq27xxx.txt b/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
>> index 37994fd..4fa8e08 100644
>> --- a/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
>> +++ b/Documentation/devicetree/bindings/power/supply/bq27xxx.txt
>> @@ -23,6 +23,7 @@ Required properties:
>> * "ti,bq27546" - BQ27546
>> * "ti,bq27742" - BQ27742
>> * "ti,bq27545" - BQ27545
>> + * "ti,bq27411" - BQ27411
>> * "ti,bq27421" - BQ27421
>> * "ti,bq27425" - BQ27425
>> * "ti,bq27426" - BQ27426
>> diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
>> index f022e1b..6dbbe95 100644
>> --- a/drivers/power/supply/bq27xxx_battery.c
>> +++ b/drivers/power/supply/bq27xxx_battery.c
>> @@ -432,6 +432,7 @@ enum bq27xxx_reg_index {
>> [BQ27XXX_REG_AP] = 0x18,
>> BQ27XXX_DM_REG_ROWS,
>> };
>> +#define bq27411_regs bq27421_regs
>> #define bq27425_regs bq27421_regs
>> #define bq27426_regs bq27421_regs
>> #define bq27441_regs bq27421_regs
>> @@ -665,6 +666,7 @@ enum bq27xxx_reg_index {
>> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
>> POWER_SUPPLY_PROP_MANUFACTURER,
>> };
>> +#define bq27411_props bq27421_props
>> #define bq27425_props bq27421_props
>> #define bq27426_props bq27421_props
>> #define bq27441_props bq27421_props
>> @@ -725,6 +727,12 @@ enum bq27xxx_dm_reg_id {
>> #define bq27545_dm_regs 0
>> #endif
>>
>> +static struct bq27xxx_dm_reg bq27411_dm_regs[] = {
>> + [BQ27XXX_DM_DESIGN_CAPACITY] = { 82, 10, 2, 0, 32767 },
>> + [BQ27XXX_DM_DESIGN_ENERGY] = { 82, 12, 2, 0, 32767 },
>> + [BQ27XXX_DM_TERMINATE_VOLTAGE] = { 82, 16, 2, 2800, 3700 },
>> +};
>> +
>> static struct bq27xxx_dm_reg bq27421_dm_regs[] = {
>> [BQ27XXX_DM_DESIGN_CAPACITY] = { 82, 10, 2, 0, 8000 },
>> [BQ27XXX_DM_DESIGN_ENERGY] = { 82, 12, 2, 0, 32767 },
>> @@ -802,6 +810,7 @@ enum bq27xxx_dm_reg_id {
>> [BQ27546] = BQ27XXX_DATA(bq27546, 0 , BQ27XXX_O_OTDC),
>> [BQ27742] = BQ27XXX_DATA(bq27742, 0 , BQ27XXX_O_OTDC),
>> [BQ27545] = BQ27XXX_DATA(bq27545, 0x04143672, BQ27XXX_O_OTDC),
>> + [BQ27411] = BQ27XXX_DATA(bq27411, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
>> [BQ27421] = BQ27XXX_DATA(bq27421, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
>> [BQ27425] = BQ27XXX_DATA(bq27425, 0x04143672, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP),
>> [BQ27426] = BQ27XXX_DATA(bq27426, 0x80008000, BQ27XXX_O_UTOT | BQ27XXX_O_CFGUP | BQ27XXX_O_RAM),
>> diff --git a/drivers/power/supply/bq27xxx_battery_i2c.c b/drivers/power/supply/bq27xxx_battery_i2c.c
>> index 4006912..2677c38 100644
>> --- a/drivers/power/supply/bq27xxx_battery_i2c.c
>> +++ b/drivers/power/supply/bq27xxx_battery_i2c.c
>> @@ -247,6 +247,7 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
>> { "bq27546", BQ27546 },
>> { "bq27742", BQ27742 },
>> { "bq27545", BQ27545 },
>> + { "bq27411", BQ27411 },
>> { "bq27421", BQ27421 },
>> { "bq27425", BQ27425 },
>> { "bq27426", BQ27426 },
>> @@ -279,6 +280,7 @@ static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
>> { .compatible = "ti,bq27546" },
>> { .compatible = "ti,bq27742" },
>> { .compatible = "ti,bq27545" },
>> + { .compatible = "ti,bq27411" },
>> { .compatible = "ti,bq27421" },
>> { .compatible = "ti,bq27425" },
>> { .compatible = "ti,bq27426" },
>> diff --git a/include/linux/power/bq27xxx_battery.h b/include/linux/power/bq27xxx_battery.h
>> index d6355f4..507c5e2 100644
>> --- a/include/linux/power/bq27xxx_battery.h
>> +++ b/include/linux/power/bq27xxx_battery.h
>> @@ -24,6 +24,7 @@ enum bq27xxx_chip {
>> BQ27546,
>> BQ27742,
>> BQ27545, /* bq27545 */
>> + BQ27411,
>> BQ27421, /* bq27421, bq27441, bq27621 */
>> BQ27425,
>> BQ27426,
>> --
>> 1.9.1
>>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] power: supply: bq27xxx: Add support for BQ27411
2018-09-17 20:37 ` Andrew F. Davis
@ 2018-09-20 9:38 ` Sebastian Reichel
0 siblings, 0 replies; 4+ messages in thread
From: Sebastian Reichel @ 2018-09-20 9:38 UTC (permalink / raw)
To: Andrew F. Davis
Cc: Liu Xiang, pali.rohar, linux-pm, linux-kernel, liuxiang_1999
[-- Attachment #1: Type: text/plain, Size: 648 bytes --]
Hi,
On Mon, Sep 17, 2018 at 03:37:18PM -0500, Andrew F. Davis wrote:
> On 09/16/2018 06:35 AM, Sebastian Reichel wrote:
> > Hi,
> >
> > On Sun, Jul 22, 2018 at 09:33:56PM +0800, Liu Xiang wrote:
> >> According to the datasheet, bq27411 is similar to bq27421.
> >>
> >> Signed-off-by: Liu Xiang <liu.xiang6@zte.com.cn>
> >> ---
> >
> > Looks good to me. Do you plan to test/review this Andrew, or
> > should I proceed with merging this patch?
> >
>
>
> I do not have this particular hardware for testing, but the patch does
> look correct,
>
> Reviewed-by: Andrew F. Davis <afd@ti.com>
Thanks, queued.
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-09-20 9:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-22 13:33 [PATCH] power: supply: bq27xxx: Add support for BQ27411 Liu Xiang
2018-09-16 11:35 ` Sebastian Reichel
2018-09-17 20:37 ` Andrew F. Davis
2018-09-20 9:38 ` Sebastian Reichel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®