linux-kernel - dt-bindings: power: supply: bq27xxx: Add monitored-battery documentation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <E66F5031-96E6-49B7-8C22-AD92A5D0CA00@goldelico.com>
Date:   Tue, 29 Aug 2017 11:43:43 +0200
From:   "H. Nikolaus Schaller" <hns@...delico.com>
To:     Liam Breck <kernel@...workimprov.net>,
        Sebastian Reichel <sre@...nel.org>
Cc:     LKML <linux-kernel@...r.kernel.org>,
        Discussions about the Letux Kernel 
        <letux-kernel@...nphoenux.org>, kernel@...a-handheld.com
Subject: dt-bindings: power: supply: bq27xxx: Add monitored-battery documentation

Hi,
I am trying to get this working on our bq27421.

But the only message I get is:

[    6.086407] bq27xxx-battery 1-0055: data memory update not supported for chip

A little research shows that this message comes from

	http://elixir.free-electrons.com/linux/v4.13-rc7/source/drivers/power/supply/bq27xxx_battery.c#L1279

So di->dm_regs is NULL.

But doing an fgrep for dm_regs shows no line of code where the pointer is set to a non-null value:

master hns$ fgrep -R dm_regs *
drivers/power/supply/bq27xxx_battery.c:	.class = (di)->dm_regs[i].subclass_id, \
drivers/power/supply/bq27xxx_battery.c:	.block = (di)->dm_regs[i].offset / BQ27XXX_DM_SZ, \
drivers/power/supply/bq27xxx_battery.c:	struct bq27xxx_dm_reg *reg = &di->dm_regs[reg_id];
drivers/power/supply/bq27xxx_battery.c:	if (!di->dm_regs) {
drivers/power/supply/bq27xxx_battery.c:	max = di->dm_regs[BQ27XXX_DM_DESIGN_ENERGY].max;
drivers/power/supply/bq27xxx_battery.c:	max = di->dm_regs[BQ27XXX_DM_DESIGN_CAPACITY].max;
drivers/power/supply/bq27xxx_battery.c:	min = di->dm_regs[BQ27XXX_DM_TERMINATE_VOLTAGE].min;
drivers/power/supply/bq27xxx_battery.c:	max = di->dm_regs[BQ27XXX_DM_TERMINATE_VOLTAGE].max;
Binary file drivers/power/supply/bq27xxx_battery.ko matches
Binary file drivers/power/supply/bq27xxx_battery.o matches
Binary file drivers/power/supply/bq27xxx_battery_i2c.ko matches
Binary file drivers/power/supply/bq27xxx_battery_i2c.o matches
include/linux/power/bq27xxx_battery.h:	struct bq27xxx_dm_reg *dm_regs;
master hns$ 

What am I doing wrong here?

BR and thanks,
Nikolaus