* [PATCH v2] power: supply: bq27xxx: Supporrt CHARGE_NOW for bq27z561/bq28z610/bq34z100
@ 2020-12-22 11:07 Hermes Zhang
2020-12-22 11:53 ` Pali Rohár
0 siblings, 1 reply; 4+ messages in thread
From: Hermes Zhang @ 2020-12-22 11:07 UTC (permalink / raw)
To: Pali Rohár, Dan Murphy, Sebastian Reichel
Cc: kernel, Hermes Zhang, linux-pm, linux-kernel
From: Hermes Zhang <chenhuiz@axis.com>
The CHARGE_NOW is map to REG_NAC for all the gauge chips beofre. But for
some chips (e.g. bq27z561) which doesn't have the REG_NAC, we use REG_RC
(remaining capacity) for CHARGE_NOW.
Signed-off-by: Hermes Zhang <chenhuiz@axis.com>
---
Notes:
Set correct REG_RC for all the chips if have
keep INVALID_REG_ADDR for bq27521, as we could not find
the datasheet and seems no one use it now.
drivers/power/supply/bq27xxx_battery.c | 35 +++++++++++++++++++++++++-
1 file changed, 34 insertions(+), 1 deletion(-)
diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
index 315e0909e6a4..774aa376653e 100644
--- a/drivers/power/supply/bq27xxx_battery.c
+++ b/drivers/power/supply/bq27xxx_battery.c
@@ -110,6 +110,7 @@ enum bq27xxx_reg_index {
BQ27XXX_REG_TTES, /* Time-to-Empty Standby */
BQ27XXX_REG_TTECP, /* Time-to-Empty at Constant Power */
BQ27XXX_REG_NAC, /* Nominal Available Capacity */
+ BQ27XXX_REG_RC, /* Remaining Capacity */
BQ27XXX_REG_FCC, /* Full Charge Capacity */
BQ27XXX_REG_CYCT, /* Cycle Count */
BQ27XXX_REG_AE, /* Available Energy */
@@ -145,6 +146,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1c,
[BQ27XXX_REG_TTECP] = 0x26,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = 0x22,
@@ -169,6 +171,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1c,
[BQ27XXX_REG_TTECP] = 0x26,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -193,6 +196,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1a,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -215,6 +219,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1c,
[BQ27XXX_REG_TTECP] = 0x26,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = 0x22,
@@ -237,6 +242,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1a,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x1e,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -257,6 +263,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1c,
[BQ27XXX_REG_TTECP] = 0x26,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
[BQ27XXX_REG_AE] = 0x22,
@@ -277,6 +284,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1c,
[BQ27XXX_REG_TTECP] = 0x26,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = 0x22,
@@ -297,6 +305,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1c,
[BQ27XXX_REG_TTECP] = 0x26,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = 0x22,
@@ -317,6 +326,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1c,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x1e,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -337,6 +347,7 @@ static u8
[BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
+ [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
[BQ27XXX_REG_FCC] = INVALID_REG_ADDR,
[BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -361,6 +372,7 @@ static u8
[BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -382,6 +394,7 @@ static u8
[BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -405,6 +418,7 @@ static u8
[BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = 0x0c,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -425,6 +439,7 @@ static u8
[BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = 0x08,
+ [BQ27XXX_REG_RC] = 0x0c,
[BQ27XXX_REG_FCC] = 0x0e,
[BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
[BQ27XXX_REG_AE] = INVALID_REG_ADDR,
@@ -450,6 +465,7 @@ static u8
[BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = 0x22,
@@ -470,6 +486,7 @@ static u8
[BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
+ [BQ27XXX_REG_RC] = 0x10,
[BQ27XXX_REG_FCC] = 0x12,
[BQ27XXX_REG_CYCT] = 0x2a,
[BQ27XXX_REG_AE] = 0x22,
@@ -490,6 +507,7 @@ static u8
[BQ27XXX_REG_TTES] = 0x1e,
[BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
[BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
+ [BQ27XXX_REG_RC] = 0x04,
[BQ27XXX_REG_FCC] = 0x06,
[BQ27XXX_REG_CYCT] = 0x2c,
[BQ27XXX_REG_AE] = 0x24,
@@ -745,6 +763,7 @@ static enum power_supply_property bq27z561_props[] = {
POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
POWER_SUPPLY_PROP_TECHNOLOGY,
POWER_SUPPLY_PROP_CHARGE_FULL,
+ POWER_SUPPLY_PROP_CHARGE_NOW,
POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
POWER_SUPPLY_PROP_CYCLE_COUNT,
POWER_SUPPLY_PROP_POWER_AVG,
@@ -764,6 +783,7 @@ static enum power_supply_property bq28z610_props[] = {
POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
POWER_SUPPLY_PROP_TECHNOLOGY,
POWER_SUPPLY_PROP_CHARGE_FULL,
+ POWER_SUPPLY_PROP_CHARGE_NOW,
POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
POWER_SUPPLY_PROP_CYCLE_COUNT,
POWER_SUPPLY_PROP_POWER_AVG,
@@ -784,6 +804,7 @@ static enum power_supply_property bq34z100_props[] = {
POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
POWER_SUPPLY_PROP_TECHNOLOGY,
POWER_SUPPLY_PROP_CHARGE_FULL,
+ POWER_SUPPLY_PROP_CHARGE_NOW,
POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
POWER_SUPPLY_PROP_CYCLE_COUNT,
POWER_SUPPLY_PROP_ENERGY_NOW,
@@ -1518,6 +1539,15 @@ static inline int bq27xxx_battery_read_nac(struct bq27xxx_device_info *di)
return bq27xxx_battery_read_charge(di, BQ27XXX_REG_NAC);
}
+/*
+ * Return the battery Remaining Capacity in µAh
+ * Or < 0 if something fails.
+ */
+static inline int bq27xxx_battery_read_rc(struct bq27xxx_device_info *di)
+{
+ return bq27xxx_battery_read_charge(di, BQ27XXX_REG_RC);
+}
+
/*
* Return the battery Full Charge Capacity in µAh
* Or < 0 if something fails.
@@ -1965,7 +1995,10 @@ static int bq27xxx_battery_get_property(struct power_supply *psy,
val->intval = POWER_SUPPLY_TECHNOLOGY_LION;
break;
case POWER_SUPPLY_PROP_CHARGE_NOW:
- ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
+ if (di->regs[BQ27XXX_REG_NAC] != INVALID_REG_ADDR)
+ ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
+ else
+ ret = bq27xxx_simple_value(bq27xxx_battery_read_rc(di), val);
break;
case POWER_SUPPLY_PROP_CHARGE_FULL:
ret = bq27xxx_simple_value(di->cache.charge_full, val);
--
2.20.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH v2] power: supply: bq27xxx: Supporrt CHARGE_NOW for bq27z561/bq28z610/bq34z100
2020-12-22 11:07 [PATCH v2] power: supply: bq27xxx: Supporrt CHARGE_NOW for bq27z561/bq28z610/bq34z100 Hermes Zhang
@ 2020-12-22 11:53 ` Pali Rohár
2020-12-23 0:40 ` Hermes Zhang
0 siblings, 1 reply; 4+ messages in thread
From: Pali Rohár @ 2020-12-22 11:53 UTC (permalink / raw)
To: Hermes Zhang, Pavel Machek
Cc: Dan Murphy, Sebastian Reichel, kernel, Hermes Zhang, linux-pm,
linux-kernel
On Tuesday 22 December 2020 19:07:20 Hermes Zhang wrote:
> From: Hermes Zhang <chenhuiz@axis.com>
>
> The CHARGE_NOW is map to REG_NAC for all the gauge chips beofre. But for
> some chips (e.g. bq27z561) which doesn't have the REG_NAC, we use REG_RC
> (remaining capacity) for CHARGE_NOW.
>
> Signed-off-by: Hermes Zhang <chenhuiz@axis.com>
> ---
>
> Notes:
> Set correct REG_RC for all the chips if have
>
> keep INVALID_REG_ADDR for bq27521, as we could not find
> the datasheet and seems no one use it now.
This chip is used in Nokia N950 and Nokia N9. Pavel implemented kernel
support, adding to loop.
Public information about it are at:
https://elinux.org/N950#sn27521_register_map
> drivers/power/supply/bq27xxx_battery.c | 35 +++++++++++++++++++++++++-
> 1 file changed, 34 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
> index 315e0909e6a4..774aa376653e 100644
> --- a/drivers/power/supply/bq27xxx_battery.c
> +++ b/drivers/power/supply/bq27xxx_battery.c
> @@ -110,6 +110,7 @@ enum bq27xxx_reg_index {
> BQ27XXX_REG_TTES, /* Time-to-Empty Standby */
> BQ27XXX_REG_TTECP, /* Time-to-Empty at Constant Power */
> BQ27XXX_REG_NAC, /* Nominal Available Capacity */
> + BQ27XXX_REG_RC, /* Remaining Capacity */
> BQ27XXX_REG_FCC, /* Full Charge Capacity */
> BQ27XXX_REG_CYCT, /* Cycle Count */
> BQ27XXX_REG_AE, /* Available Energy */
> @@ -145,6 +146,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1c,
> [BQ27XXX_REG_TTECP] = 0x26,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = 0x22,
> @@ -169,6 +171,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1c,
> [BQ27XXX_REG_TTECP] = 0x26,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -193,6 +196,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1a,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -215,6 +219,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1c,
> [BQ27XXX_REG_TTECP] = 0x26,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = 0x22,
> @@ -237,6 +242,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1a,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x1e,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -257,6 +263,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1c,
> [BQ27XXX_REG_TTECP] = 0x26,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
> [BQ27XXX_REG_AE] = 0x22,
> @@ -277,6 +284,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1c,
> [BQ27XXX_REG_TTECP] = 0x26,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = 0x22,
> @@ -297,6 +305,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1c,
> [BQ27XXX_REG_TTECP] = 0x26,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = 0x22,
> @@ -317,6 +326,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1c,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x1e,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -337,6 +347,7 @@ static u8
> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
> [BQ27XXX_REG_FCC] = INVALID_REG_ADDR,
> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -361,6 +372,7 @@ static u8
> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -382,6 +394,7 @@ static u8
> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -405,6 +418,7 @@ static u8
> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = 0x0c,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -425,6 +439,7 @@ static u8
> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = 0x08,
> + [BQ27XXX_REG_RC] = 0x0c,
> [BQ27XXX_REG_FCC] = 0x0e,
> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> @@ -450,6 +465,7 @@ static u8
> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = 0x22,
> @@ -470,6 +486,7 @@ static u8
> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> + [BQ27XXX_REG_RC] = 0x10,
> [BQ27XXX_REG_FCC] = 0x12,
> [BQ27XXX_REG_CYCT] = 0x2a,
> [BQ27XXX_REG_AE] = 0x22,
> @@ -490,6 +507,7 @@ static u8
> [BQ27XXX_REG_TTES] = 0x1e,
> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> + [BQ27XXX_REG_RC] = 0x04,
> [BQ27XXX_REG_FCC] = 0x06,
> [BQ27XXX_REG_CYCT] = 0x2c,
> [BQ27XXX_REG_AE] = 0x24,
> @@ -745,6 +763,7 @@ static enum power_supply_property bq27z561_props[] = {
> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
> POWER_SUPPLY_PROP_TECHNOLOGY,
> POWER_SUPPLY_PROP_CHARGE_FULL,
> + POWER_SUPPLY_PROP_CHARGE_NOW,
> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
> POWER_SUPPLY_PROP_CYCLE_COUNT,
> POWER_SUPPLY_PROP_POWER_AVG,
> @@ -764,6 +783,7 @@ static enum power_supply_property bq28z610_props[] = {
> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
> POWER_SUPPLY_PROP_TECHNOLOGY,
> POWER_SUPPLY_PROP_CHARGE_FULL,
> + POWER_SUPPLY_PROP_CHARGE_NOW,
> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
> POWER_SUPPLY_PROP_CYCLE_COUNT,
> POWER_SUPPLY_PROP_POWER_AVG,
> @@ -784,6 +804,7 @@ static enum power_supply_property bq34z100_props[] = {
> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
> POWER_SUPPLY_PROP_TECHNOLOGY,
> POWER_SUPPLY_PROP_CHARGE_FULL,
> + POWER_SUPPLY_PROP_CHARGE_NOW,
> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
> POWER_SUPPLY_PROP_CYCLE_COUNT,
> POWER_SUPPLY_PROP_ENERGY_NOW,
> @@ -1518,6 +1539,15 @@ static inline int bq27xxx_battery_read_nac(struct bq27xxx_device_info *di)
> return bq27xxx_battery_read_charge(di, BQ27XXX_REG_NAC);
> }
>
> +/*
> + * Return the battery Remaining Capacity in µAh
> + * Or < 0 if something fails.
> + */
> +static inline int bq27xxx_battery_read_rc(struct bq27xxx_device_info *di)
> +{
> + return bq27xxx_battery_read_charge(di, BQ27XXX_REG_RC);
> +}
> +
> /*
> * Return the battery Full Charge Capacity in µAh
> * Or < 0 if something fails.
> @@ -1965,7 +1995,10 @@ static int bq27xxx_battery_get_property(struct power_supply *psy,
> val->intval = POWER_SUPPLY_TECHNOLOGY_LION;
> break;
> case POWER_SUPPLY_PROP_CHARGE_NOW:
> - ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
> + if (di->regs[BQ27XXX_REG_NAC] != INVALID_REG_ADDR)
> + ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
> + else
> + ret = bq27xxx_simple_value(bq27xxx_battery_read_rc(di), val);
> break;
> case POWER_SUPPLY_PROP_CHARGE_FULL:
> ret = bq27xxx_simple_value(di->cache.charge_full, val);
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] power: supply: bq27xxx: Supporrt CHARGE_NOW for bq27z561/bq28z610/bq34z100
2020-12-22 11:53 ` Pali Rohár
@ 2020-12-23 0:40 ` Hermes Zhang
2021-01-16 11:05 ` Sebastian Reichel
0 siblings, 1 reply; 4+ messages in thread
From: Hermes Zhang @ 2020-12-23 0:40 UTC (permalink / raw)
To: Pali Rohár, Pavel Machek
Cc: Dan Murphy, Sebastian Reichel, kernel, Hermes Zhang, linux-pm,
linux-kernel
On 12/22/20 7:53 PM, Pali Rohár wrote:
> On Tuesday 22 December 2020 19:07:20 Hermes Zhang wrote:
>> From: Hermes Zhang <chenhuiz@axis.com>
>>
>> The CHARGE_NOW is map to REG_NAC for all the gauge chips beofre. But for
>> some chips (e.g. bq27z561) which doesn't have the REG_NAC, we use REG_RC
>> (remaining capacity) for CHARGE_NOW.
>>
>> Signed-off-by: Hermes Zhang <chenhuiz@axis.com>
>> ---
>>
>> Notes:
>> Set correct REG_RC for all the chips if have
>>
>> keep INVALID_REG_ADDR for bq27521, as we could not find
>> the datasheet and seems no one use it now.
> This chip is used in Nokia N950 and Nokia N9. Pavel implemented kernel
> support, adding to loop.
>
> Public information about it are at:
> https://elinux.org/N950#sn27521_register_map
Thanks for the info. From the link it seem bq27521 have neither NAC and
RC register.
Best Regards,
Hermes
>
>> drivers/power/supply/bq27xxx_battery.c | 35 +++++++++++++++++++++++++-
>> 1 file changed, 34 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
>> index 315e0909e6a4..774aa376653e 100644
>> --- a/drivers/power/supply/bq27xxx_battery.c
>> +++ b/drivers/power/supply/bq27xxx_battery.c
>> @@ -110,6 +110,7 @@ enum bq27xxx_reg_index {
>> BQ27XXX_REG_TTES, /* Time-to-Empty Standby */
>> BQ27XXX_REG_TTECP, /* Time-to-Empty at Constant Power */
>> BQ27XXX_REG_NAC, /* Nominal Available Capacity */
>> + BQ27XXX_REG_RC, /* Remaining Capacity */
>> BQ27XXX_REG_FCC, /* Full Charge Capacity */
>> BQ27XXX_REG_CYCT, /* Cycle Count */
>> BQ27XXX_REG_AE, /* Available Energy */
>> @@ -145,6 +146,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1c,
>> [BQ27XXX_REG_TTECP] = 0x26,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = 0x22,
>> @@ -169,6 +171,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1c,
>> [BQ27XXX_REG_TTECP] = 0x26,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -193,6 +196,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1a,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -215,6 +219,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1c,
>> [BQ27XXX_REG_TTECP] = 0x26,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = 0x22,
>> @@ -237,6 +242,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1a,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x1e,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -257,6 +263,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1c,
>> [BQ27XXX_REG_TTECP] = 0x26,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_AE] = 0x22,
>> @@ -277,6 +284,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1c,
>> [BQ27XXX_REG_TTECP] = 0x26,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = 0x22,
>> @@ -297,6 +305,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1c,
>> [BQ27XXX_REG_TTECP] = 0x26,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = 0x22,
>> @@ -317,6 +326,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1c,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x1e,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -337,6 +347,7 @@ static u8
>> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
>> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_FCC] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -361,6 +372,7 @@ static u8
>> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -382,6 +394,7 @@ static u8
>> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -405,6 +418,7 @@ static u8
>> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = 0x0c,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -425,6 +439,7 @@ static u8
>> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = 0x08,
>> + [BQ27XXX_REG_RC] = 0x0c,
>> [BQ27XXX_REG_FCC] = 0x0e,
>> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
>> @@ -450,6 +465,7 @@ static u8
>> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = 0x22,
>> @@ -470,6 +486,7 @@ static u8
>> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
>> + [BQ27XXX_REG_RC] = 0x10,
>> [BQ27XXX_REG_FCC] = 0x12,
>> [BQ27XXX_REG_CYCT] = 0x2a,
>> [BQ27XXX_REG_AE] = 0x22,
>> @@ -490,6 +507,7 @@ static u8
>> [BQ27XXX_REG_TTES] = 0x1e,
>> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
>> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
>> + [BQ27XXX_REG_RC] = 0x04,
>> [BQ27XXX_REG_FCC] = 0x06,
>> [BQ27XXX_REG_CYCT] = 0x2c,
>> [BQ27XXX_REG_AE] = 0x24,
>> @@ -745,6 +763,7 @@ static enum power_supply_property bq27z561_props[] = {
>> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
>> POWER_SUPPLY_PROP_TECHNOLOGY,
>> POWER_SUPPLY_PROP_CHARGE_FULL,
>> + POWER_SUPPLY_PROP_CHARGE_NOW,
>> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
>> POWER_SUPPLY_PROP_CYCLE_COUNT,
>> POWER_SUPPLY_PROP_POWER_AVG,
>> @@ -764,6 +783,7 @@ static enum power_supply_property bq28z610_props[] = {
>> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
>> POWER_SUPPLY_PROP_TECHNOLOGY,
>> POWER_SUPPLY_PROP_CHARGE_FULL,
>> + POWER_SUPPLY_PROP_CHARGE_NOW,
>> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
>> POWER_SUPPLY_PROP_CYCLE_COUNT,
>> POWER_SUPPLY_PROP_POWER_AVG,
>> @@ -784,6 +804,7 @@ static enum power_supply_property bq34z100_props[] = {
>> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
>> POWER_SUPPLY_PROP_TECHNOLOGY,
>> POWER_SUPPLY_PROP_CHARGE_FULL,
>> + POWER_SUPPLY_PROP_CHARGE_NOW,
>> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
>> POWER_SUPPLY_PROP_CYCLE_COUNT,
>> POWER_SUPPLY_PROP_ENERGY_NOW,
>> @@ -1518,6 +1539,15 @@ static inline int bq27xxx_battery_read_nac(struct bq27xxx_device_info *di)
>> return bq27xxx_battery_read_charge(di, BQ27XXX_REG_NAC);
>> }
>>
>> +/*
>> + * Return the battery Remaining Capacity in µAh
>> + * Or < 0 if something fails.
>> + */
>> +static inline int bq27xxx_battery_read_rc(struct bq27xxx_device_info *di)
>> +{
>> + return bq27xxx_battery_read_charge(di, BQ27XXX_REG_RC);
>> +}
>> +
>> /*
>> * Return the battery Full Charge Capacity in µAh
>> * Or < 0 if something fails.
>> @@ -1965,7 +1995,10 @@ static int bq27xxx_battery_get_property(struct power_supply *psy,
>> val->intval = POWER_SUPPLY_TECHNOLOGY_LION;
>> break;
>> case POWER_SUPPLY_PROP_CHARGE_NOW:
>> - ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
>> + if (di->regs[BQ27XXX_REG_NAC] != INVALID_REG_ADDR)
>> + ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
>> + else
>> + ret = bq27xxx_simple_value(bq27xxx_battery_read_rc(di), val);
>> break;
>> case POWER_SUPPLY_PROP_CHARGE_FULL:
>> ret = bq27xxx_simple_value(di->cache.charge_full, val);
>> --
>> 2.20.1
>>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] power: supply: bq27xxx: Supporrt CHARGE_NOW for bq27z561/bq28z610/bq34z100
2020-12-23 0:40 ` Hermes Zhang
@ 2021-01-16 11:05 ` Sebastian Reichel
0 siblings, 0 replies; 4+ messages in thread
From: Sebastian Reichel @ 2021-01-16 11:05 UTC (permalink / raw)
To: Hermes Zhang
Cc: Pali Rohár, Pavel Machek, Dan Murphy, kernel, linux-pm,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 9746 bytes --]
Hi,
On Wed, Dec 23, 2020 at 12:40:09AM +0000, Hermes Zhang wrote:
> On 12/22/20 7:53 PM, Pali Rohár wrote:
> > On Tuesday 22 December 2020 19:07:20 Hermes Zhang wrote:
> >> From: Hermes Zhang <chenhuiz@axis.com>
> >>
> >> The CHARGE_NOW is map to REG_NAC for all the gauge chips beofre. But for
> >> some chips (e.g. bq27z561) which doesn't have the REG_NAC, we use REG_RC
> >> (remaining capacity) for CHARGE_NOW.
> >>
> >> Signed-off-by: Hermes Zhang <chenhuiz@axis.com>
> >> ---
> >>
> >> Notes:
> >> Set correct REG_RC for all the chips if have
> >>
> >> keep INVALID_REG_ADDR for bq27521, as we could not find
> >> the datasheet and seems no one use it now.
> > This chip is used in Nokia N950 and Nokia N9. Pavel implemented kernel
> > support, adding to loop.
> >
> > Public information about it are at:
> > https://elinux.org/N950#sn27521_register_map
>
> Thanks for the info. From the link it seem bq27521 have neither NAC and
> RC register.
Thanks, I queued the patch to power-supply's for-next branch. (I reworded the
commit message, which was was hard to understand and had a few typos).
-- Sebastian
> Best Regards,
>
> Hermes
>
> >
> >> drivers/power/supply/bq27xxx_battery.c | 35 +++++++++++++++++++++++++-
> >> 1 file changed, 34 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
> >> index 315e0909e6a4..774aa376653e 100644
> >> --- a/drivers/power/supply/bq27xxx_battery.c
> >> +++ b/drivers/power/supply/bq27xxx_battery.c
> >> @@ -110,6 +110,7 @@ enum bq27xxx_reg_index {
> >> BQ27XXX_REG_TTES, /* Time-to-Empty Standby */
> >> BQ27XXX_REG_TTECP, /* Time-to-Empty at Constant Power */
> >> BQ27XXX_REG_NAC, /* Nominal Available Capacity */
> >> + BQ27XXX_REG_RC, /* Remaining Capacity */
> >> BQ27XXX_REG_FCC, /* Full Charge Capacity */
> >> BQ27XXX_REG_CYCT, /* Cycle Count */
> >> BQ27XXX_REG_AE, /* Available Energy */
> >> @@ -145,6 +146,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1c,
> >> [BQ27XXX_REG_TTECP] = 0x26,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = 0x22,
> >> @@ -169,6 +171,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1c,
> >> [BQ27XXX_REG_TTECP] = 0x26,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -193,6 +196,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1a,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -215,6 +219,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1c,
> >> [BQ27XXX_REG_TTECP] = 0x26,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = 0x22,
> >> @@ -237,6 +242,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1a,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x1e,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -257,6 +263,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1c,
> >> [BQ27XXX_REG_TTECP] = 0x26,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_AE] = 0x22,
> >> @@ -277,6 +284,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1c,
> >> [BQ27XXX_REG_TTECP] = 0x26,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = 0x22,
> >> @@ -297,6 +305,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1c,
> >> [BQ27XXX_REG_TTECP] = 0x26,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = 0x22,
> >> @@ -317,6 +326,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1c,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x1e,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -337,6 +347,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> >> + [BQ27XXX_REG_RC] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_FCC] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -361,6 +372,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -382,6 +394,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -405,6 +418,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = 0x0c,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -425,6 +439,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = 0x08,
> >> + [BQ27XXX_REG_RC] = 0x0c,
> >> [BQ27XXX_REG_FCC] = 0x0e,
> >> [BQ27XXX_REG_CYCT] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_AE] = INVALID_REG_ADDR,
> >> @@ -450,6 +465,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = 0x22,
> >> @@ -470,6 +486,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> >> + [BQ27XXX_REG_RC] = 0x10,
> >> [BQ27XXX_REG_FCC] = 0x12,
> >> [BQ27XXX_REG_CYCT] = 0x2a,
> >> [BQ27XXX_REG_AE] = 0x22,
> >> @@ -490,6 +507,7 @@ static u8
> >> [BQ27XXX_REG_TTES] = 0x1e,
> >> [BQ27XXX_REG_TTECP] = INVALID_REG_ADDR,
> >> [BQ27XXX_REG_NAC] = INVALID_REG_ADDR,
> >> + [BQ27XXX_REG_RC] = 0x04,
> >> [BQ27XXX_REG_FCC] = 0x06,
> >> [BQ27XXX_REG_CYCT] = 0x2c,
> >> [BQ27XXX_REG_AE] = 0x24,
> >> @@ -745,6 +763,7 @@ static enum power_supply_property bq27z561_props[] = {
> >> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
> >> POWER_SUPPLY_PROP_TECHNOLOGY,
> >> POWER_SUPPLY_PROP_CHARGE_FULL,
> >> + POWER_SUPPLY_PROP_CHARGE_NOW,
> >> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
> >> POWER_SUPPLY_PROP_CYCLE_COUNT,
> >> POWER_SUPPLY_PROP_POWER_AVG,
> >> @@ -764,6 +783,7 @@ static enum power_supply_property bq28z610_props[] = {
> >> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
> >> POWER_SUPPLY_PROP_TECHNOLOGY,
> >> POWER_SUPPLY_PROP_CHARGE_FULL,
> >> + POWER_SUPPLY_PROP_CHARGE_NOW,
> >> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
> >> POWER_SUPPLY_PROP_CYCLE_COUNT,
> >> POWER_SUPPLY_PROP_POWER_AVG,
> >> @@ -784,6 +804,7 @@ static enum power_supply_property bq34z100_props[] = {
> >> POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
> >> POWER_SUPPLY_PROP_TECHNOLOGY,
> >> POWER_SUPPLY_PROP_CHARGE_FULL,
> >> + POWER_SUPPLY_PROP_CHARGE_NOW,
> >> POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN,
> >> POWER_SUPPLY_PROP_CYCLE_COUNT,
> >> POWER_SUPPLY_PROP_ENERGY_NOW,
> >> @@ -1518,6 +1539,15 @@ static inline int bq27xxx_battery_read_nac(struct bq27xxx_device_info *di)
> >> return bq27xxx_battery_read_charge(di, BQ27XXX_REG_NAC);
> >> }
> >>
> >> +/*
> >> + * Return the battery Remaining Capacity in µAh
> >> + * Or < 0 if something fails.
> >> + */
> >> +static inline int bq27xxx_battery_read_rc(struct bq27xxx_device_info *di)
> >> +{
> >> + return bq27xxx_battery_read_charge(di, BQ27XXX_REG_RC);
> >> +}
> >> +
> >> /*
> >> * Return the battery Full Charge Capacity in µAh
> >> * Or < 0 if something fails.
> >> @@ -1965,7 +1995,10 @@ static int bq27xxx_battery_get_property(struct power_supply *psy,
> >> val->intval = POWER_SUPPLY_TECHNOLOGY_LION;
> >> break;
> >> case POWER_SUPPLY_PROP_CHARGE_NOW:
> >> - ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
> >> + if (di->regs[BQ27XXX_REG_NAC] != INVALID_REG_ADDR)
> >> + ret = bq27xxx_simple_value(bq27xxx_battery_read_nac(di), val);
> >> + else
> >> + ret = bq27xxx_simple_value(bq27xxx_battery_read_rc(di), val);
> >> break;
> >> case POWER_SUPPLY_PROP_CHARGE_FULL:
> >> ret = bq27xxx_simple_value(di->cache.charge_full, val);
> >> --
> >> 2.20.1
> >>
>
[-- 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:[~2021-01-16 11:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-22 11:07 [PATCH v2] power: supply: bq27xxx: Supporrt CHARGE_NOW for bq27z561/bq28z610/bq34z100 Hermes Zhang
2020-12-22 11:53 ` Pali Rohár
2020-12-23 0:40 ` Hermes Zhang
2021-01-16 11:05 ` Sebastian Reichel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).