From: Liam Breck <liam@networkimprov.net>
To: Sebastian Reichel <sre@kernel.org>,
"Andrew F. Davis" <afd@ti.com>,
linux-pm@vger.kernel.org
Subject: [PATCH v13 0/11] devicetree simple-battery and client in bq27xxx_battery
Date: Wed, 3 May 2017 23:18:00 -0700 [thread overview]
Message-ID: <20170504061811.18107-1-liam@networkimprov.net> (raw)
Overview:
* new devicetree battery node specifies static battery data
* fuel gauge and charger nodes shall use monitored-battery=<&battery_node>
* new power_supply_get_battery_info() reads battery data from devicetree
* new struct power_supply_battery_info provides battery data to drivers
* drivers surface battery data in sysfs via related power_supply_prop_* fields
* bq27xxx driver calls the above and writes battery data to RAM/NVM for params
essential to correct operation: energy-full-design-microwatt-hours,
charge-full-design-microamp-hours, voltage-min-design-microvolt
Changes in v13:
Doc...bindings/power/supply/*
* add fields in battery.txt, see below
* caution about changing battery type in battery.txt
* note: Rob acked v11 battery.txt containing URL to Linux header
* more detail in bq27xxx.txt
power_supply_core:
* add battery_info fields precharge-current-microamp & charge-term-current-microamp
* new patch for power_supply_prop_precharge_current
bq27xxx_battery:
* add copyright notice
* for flash/NVM chips, emit warning instead of doing update
* config_battery_bq27xxx_dt_updates_nvm enables update of flash/NVM
* module param dt_monitored_battery_updates_nvm lets user disallow update
* new patch to flag dupes in bq27xxx_regs[]
* drop patch to clean up error reporting
* polishing from Andrew's feedback
* resolve checkpatch errors
* refactor patchset
* fix missing static keyword flagged by Colin
Changes in v12: (several partial series posts; see above)
Changes in v11:
power_supply_core:
* switch to compatible = "simple-battery"
* add docs to power_supply_class.txt
Documentation/devicetree/.../battery.txt:
* add description, drop refs to Linux, "simple-battery"
bq27xxx_battery:
* reset flash chips after DM update
* add bq27xxx_write/xfer()
* polishing from Andrew's feedback
* new patch to clean up error reporting
* new patch to consolidate duplicate register/property arrays
Changes in v10:
bq27xxx_battery:
* pass actual chip ID into _setup()
* add di->unseal_key & di->dm_regs; drop static arrays
* support bq27425, 441, 621
Changes in v9:
bq27xxx_battery:
* fix set_cfgupdate()
* support bq27500, 545, 421; defer others
* drop print_dm_blocks() patch
* minor polishing
Documentation/devicetree/.../battery.txt:
* describe rationale for enum power_supply_property names
Changes in v8:
bq27xxx_battery:
* wait on flag after set_cfgupdate & soft_reset
* drop print_config(), report status in update_dm_block()
* clarify error messages
* cleanup from Andrew's feedback; minor polishing
Changes in v7:
bq27xxx_battery:
* support chips where terminate_voltage & design_* live in separate blocks
* draft support for 421, 441, 621 chips
* new patch to log chip memory fields
* report bus I/O errors; return error code in bq27xxx_battery_i2c
* verify checksum in read_dm_block()
* use set_cfgupdate only if chip provides it, soft_reset on I/O error
* block_data_control=0 only in write_dm_block()
* note toxic code from TI bqtool in write_dm_block()
* lots of functionally neutral polishing
Documentation/devicetree/.../battery.txt:
* mention power_supply_get_battery_info()
Changes in v6:
* Documentation/devictree/... fixes
* bq27xxx_battery: clarify names
* bq27xxx_battery: verify that selected registers are supported
* bq27xxx_battery: allocate NVM buffer on stack
* bq27xxx_battery_i2c: fix return code of bulk_read
Changes in v5:
* incorporate feedback into Documentation/devicetree/.../battery.txt
* use power_supply_prop_* names in devicetree and power_supply_battery_info
* default fields to -EINVAL in power_supply_battery_info
* power_supply_get_battery_info() always looks for "monitored-battery"
* power_supply_get_battery_info() emits a warning if !psy->of_node
* squash patches for power_supply_battery_info
* bq27xxx_battery: check power_supply_battery_info values
* bq27xxx_battery: note missing power_supply_prop_* features
* bq27xxx_battery: new patch for access methods
Changes in v4:
* add "fixed-battery" compatible field to be be more consistant with devicetree
Changes in v3:
* split i2c changes into respective patches
* add documentation for battery information for fuel gauge
* rebased documentation patches on change on the list
* abstracted the battery configuration for the state machine
to an generic struct and platform data access function
Changes in v2:
* add documentation for uWh and uAh property units
* change devicetree entries to match new property units
Liam Breck (9):
dt-bindings: power: supply: Add battery.txt with simple-battery binding
power: supply: core: Add power_supply_battery_info and API
power: supply: core: Add power_supply_prop_precharge
dt-bindings: power: supply: bq27xxx: Add monitored-battery documentation
power: supply: bq27xxx_battery: Add chip data memory read/write support
power: supply: bq27xxx_battery: Add power_supply_battery_info support
power: supply: bq27xxx_battery: Enable data memory update for certain chips
power: supply: bq27xxx_battery: Flag identical register maps when in debug mode
power: supply: bq27xxx_battery: Remove duplicate register arrays
Matt Ranostay (2):
devicetree: property-units: Add uWh and uAh units
power: supply: bq27xxx_battery: Add bulk transfer bus methods
.../devicetree/bindings/power/supply/battery.txt | 53 ++
.../devicetree/bindings/power/supply/bq27xxx.txt | 31 +-
.../devicetree/bindings/property-units.txt | 2 +
Documentation/power/power_supply_class.txt | 31 +-
drivers/power/supply/Kconfig | 9 +
drivers/power/supply/bq27xxx_battery.c | 750 +++++++++++++++++----
drivers/power/supply/bq27xxx_battery_i2c.c | 98 ++-
drivers/power/supply/power_supply_core.c | 51 ++
drivers/power/supply/power_supply_sysfs.c | 1 +
include/linux/power/bq27xxx_battery.h | 29 +-
include/linux/power_supply.h | 23 +
11 files changed, 920 insertions(+), 158 deletions(-)
create mode 100644 Documentation/devicetree/bindings/power/supply/battery.txt
--
2.9.3
next reply other threads:[~2017-05-04 6:18 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-04 6:18 Liam Breck [this message]
2017-05-04 6:18 ` [PATCH v13 01/11] devicetree: property-units: Add uWh and uAh units Liam Breck
2017-05-04 6:18 ` Liam Breck
2017-05-04 6:18 ` [PATCH v13 02/11] dt-bindings: power: supply: Add battery.txt with simple-battery binding Liam Breck
2017-05-04 6:18 ` [PATCH v13 03/11] power: supply: core: Add power_supply_battery_info and API Liam Breck
2017-05-04 6:18 ` [PATCH v13 04/11] power: supply: core: Add power_supply_prop_precharge Liam Breck
[not found] ` <20170504061811.18107-1-liam-RYWXG+zxWwBdeoIcmNTgJF6hYfS7NtTn@public.gmane.org>
2017-05-04 6:18 ` [PATCH v13 05/11] dt-bindings: power: supply: bq27xxx: Add monitored-battery documentation Liam Breck
2017-05-04 6:18 ` [PATCH v13 06/11] power: supply: bq27xxx_battery: Add bulk transfer bus methods Liam Breck
2017-05-04 6:18 ` [PATCH v13 07/11] power: supply: bq27xxx_battery: Add chip data memory read/write support Liam Breck
2017-05-04 16:44 ` Andrew F. Davis
2017-05-04 18:07 ` Liam Breck
2017-05-05 12:45 ` Liam Breck
2017-05-05 15:40 ` Andrew F. Davis
2017-05-05 18:44 ` Liam Breck
2017-05-04 6:18 ` [PATCH v13 08/11] power: supply: bq27xxx_battery: Add power_supply_battery_info support Liam Breck
2017-05-04 16:52 ` Andrew F. Davis
2017-05-04 18:40 ` Liam Breck
2017-05-08 6:16 ` Liam Breck
2017-05-08 14:54 ` Andrew F. Davis
2017-05-08 18:39 ` Liam Breck
2017-05-08 18:42 ` Andrew F. Davis
2017-05-08 19:31 ` Liam Breck
2017-05-08 19:50 ` Andrew F. Davis
2017-05-08 20:34 ` Liam Breck
2017-05-09 16:06 ` Andrew F. Davis
2017-05-04 6:18 ` [PATCH v13 09/11] power: supply: bq27xxx_battery: Enable data memory update for certain chips Liam Breck
2017-05-04 17:00 ` Andrew F. Davis
2017-05-04 19:12 ` Liam Breck
2017-05-05 19:31 ` Liam Breck
2017-05-05 19:45 ` Andrew F. Davis
2017-05-05 20:14 ` Liam Breck
2017-05-05 20:44 ` Andrew F. Davis
2017-05-05 20:55 ` Liam Breck
2017-05-05 21:04 ` Andrew F. Davis
2017-05-05 21:27 ` Liam Breck
2017-05-05 21:29 ` Andrew F. Davis
2017-05-05 21:38 ` Liam Breck
2017-05-08 6:40 ` Liam Breck
2017-05-08 15:00 ` Andrew F. Davis
2017-05-08 19:02 ` Liam Breck
2017-05-08 19:09 ` Andrew F. Davis
2017-05-08 20:01 ` Liam Breck
2017-05-09 16:20 ` Andrew F. Davis
2017-05-10 9:16 ` Liam Breck
2017-05-04 6:18 ` [PATCH v13 10/11] power: supply: bq27xxx_battery: Flag identical register maps when in debug mode Liam Breck
2017-05-04 16:38 ` Andrew F. Davis
2017-05-08 6:18 ` Liam Breck
2017-05-08 15:01 ` Andrew F. Davis
2017-05-08 19:07 ` Liam Breck
2017-05-08 19:12 ` Andrew F. Davis
2017-05-08 20:08 ` Liam Breck
2017-05-04 6:18 ` [PATCH v13 11/11] power: supply: bq27xxx_battery: Remove duplicate register arrays Liam Breck
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170504061811.18107-1-liam@networkimprov.net \
--to=liam@networkimprov.net \
--cc=afd@ti.com \
--cc=linux-pm@vger.kernel.org \
--cc=sre@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.