[PATCH] power: supply: bq27xxx: drop dead data memory update code
From: Henrik Grimler
Date: Mon Nov 02 2026 - 05:29:00 EST
bq27500, bq27545 and bq27441 had some work done to enable data memory
updates in commit 05045379b274 ("power: supply: bq27xxx: Enable data
memory update for certain chips"), but it seems like it was never
tested on the hardware, and the code has been in #if 0 .. #endif's
ever since.
It has been around 9 years since this dead code was added. Let's drop
it to give a cleaner driver, as no one has apparently needed the code.
Signed-off-by: Henrik Grimler <henrik.grimler@xxxxxxxx>
---
drivers/power/supply/bq27xxx_battery.c | 36 ++--------------------------------
1 file changed, 2 insertions(+), 34 deletions(-)
diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c
index 09272249b4f7..0238ed754e94 100644
--- a/drivers/power/supply/bq27xxx_battery.c
+++ b/drivers/power/supply/bq27xxx_battery.c
@@ -922,20 +922,11 @@ enum bq27xxx_dm_reg_id {
#define bq27000_dm_regs NULL
#define bq27010_dm_regs NULL
+#define bq27500_dm_regs NULL
#define bq2750x_dm_regs NULL
#define bq2751x_dm_regs NULL
#define bq2752x_dm_regs NULL
-#if 0 /* not yet tested */
-static struct bq27xxx_dm_reg bq27500_dm_regs[] = {
- [BQ27XXX_DM_DESIGN_CAPACITY] = { 48, 10, 2, 0, 65535 },
- [BQ27XXX_DM_DESIGN_ENERGY] = { }, /* missing on chip */
- [BQ27XXX_DM_TERMINATE_VOLTAGE] = { 80, 48, 2, 1000, 32767 },
-};
-#else
-#define bq27500_dm_regs NULL
-#endif
-
/* todo create data memory definitions from datasheets and test on chips */
#define bq27510g1_dm_regs NULL
#define bq27510g2_dm_regs NULL
@@ -949,19 +940,10 @@ static struct bq27xxx_dm_reg bq27500_dm_regs[] = {
#define bq27531_dm_regs NULL
#define bq27541_dm_regs NULL
#define bq27542_dm_regs NULL
+#define bq27545_dm_regs NULL
#define bq27546_dm_regs NULL
#define bq27742_dm_regs NULL
-#if 0 /* not yet tested */
-static struct bq27xxx_dm_reg bq27545_dm_regs[] = {
- [BQ27XXX_DM_DESIGN_CAPACITY] = { 48, 23, 2, 0, 32767 },
- [BQ27XXX_DM_DESIGN_ENERGY] = { 48, 25, 2, 0, 32767 },
- [BQ27XXX_DM_TERMINATE_VOLTAGE] = { 80, 67, 2, 2800, 3700 },
-};
-#else
-#define bq27545_dm_regs NULL
-#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 },
@@ -986,22 +968,8 @@ static struct bq27xxx_dm_reg bq27426_dm_regs[] = {
[BQ27XXX_DM_TERMINATE_VOLTAGE] = { 82, 10, 2, 2500, 3700 },
};
-#if 0 /* not yet tested */
-#define bq27441_dm_regs bq27421_dm_regs
-#else
#define bq27441_dm_regs NULL
-#endif
-
-#if 0 /* not yet tested */
-static struct bq27xxx_dm_reg bq27621_dm_regs[] = {
- [BQ27XXX_DM_DESIGN_CAPACITY] = { 82, 3, 2, 0, 8000 },
- [BQ27XXX_DM_DESIGN_ENERGY] = { 82, 5, 2, 0, 32767 },
- [BQ27XXX_DM_TERMINATE_VOLTAGE] = { 82, 9, 2, 2500, 3700 },
-};
-#else
#define bq27621_dm_regs NULL
-#endif
-
#define bq27z561_dm_regs NULL
#define bq27z746_dm_regs NULL
#define bq28z610_dm_regs NULL
---
base-commit: f0f400459bf6a037e70768de623fff3f8920b8bb
change-id: 20261026-bq27xxx-cleanup-2970cb8ab981
Best regards,
--
Henrik Grimler <henrik.grimler@xxxxxxxx>