From: Liam Breck <liam@networkimprov.net>
To: knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net,
robh+dt@kernel.org, mark.rutland@arm.com, wens@csie.org,
sre@kernel.org, linux@armlinux.org.uk,
maxime.ripard@free-electrons.com, lee.jones@linaro.org
Cc: thomas.petazzoni@free-electrons.com, devicetree@vger.kernel.org,
linux-pm@vger.kernel.org, linux-iio@vger.kernel.org,
linux-kernel@vger.kernel.org,
Quentin Schulz <quentin.schulz@free-electrons.com>,
bonbons@linux-vserver.org, icenowy@aosc.xyz,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 01/18] dt-bindings: power: battery: add constant-charge-current property
Date: Tue, 14 Feb 2017 16:46:46 -0800 [thread overview]
Message-ID: <20170215004646.23607-1-liam@networkimprov.net> (raw)
In-Reply-To: <20170214094112.27426-2-quentin.schulz@free-electrons.com>
On Tue, 14 Feb 2017 10:40:55 +0100 Quentin Schulz wrote:
> This adds the constant-charge-current property to the list of optional
> properties of the battery.
>
> The constant charge current is critical for batteries as they can't
> handle all charge currents.
>
> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>
> ---
>
> added in v3
>
> Documentation/devicetree/bindings/power/supply/battery.txt | 2 ++
Is constant-charge-current dependent on the battery (e.g. capacity, nominal voltage, etc) or the
system (charger chip, input current/voltage, etc)?
It belongs in Doc.../power/supply/battery.txt if it's a characteristic of the battery.
Note, this page asserts that constant-current charging applies to NiMH batteries:
http://power-topics.blogspot.com/2016/05/constant-voltage-constant-current.html
Related properties to be added to battery.txt near-future in a patchset for the BQ24190
charger are as follows. These are not currently in enum power_supply_property, so the actual names
are still to be decided.
precharge-current-microamp:
maximum charge current during precharge phase (typically 20% of battery capacity)
termination-current-microamp (or endcharge-current):
a charge cycle terminates when the battery voltage is above recharge threshold,
and the current is below this setting (typically 10% of battery capacity)
~.~
next prev parent reply other threads:[~2017-02-15 0:46 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-14 9:40 [PATCH v3 00/18] add support for AXP20X and AXP22X power supply drivers Quentin Schulz
2017-02-14 9:40 ` [PATCH v3 01/18] dt-bindings: power: battery: add constant-charge-current property Quentin Schulz
2017-02-15 0:46 ` Liam Breck [this message]
2017-02-15 8:53 ` Quentin Schulz
2017-02-15 20:18 ` Liam Breck
2017-02-21 4:55 ` Chen-Yu Tsai
2017-03-14 13:44 ` Quentin Schulz
2017-02-22 21:25 ` Rob Herring
2017-02-14 9:40 ` [PATCH v3 02/18] power: supply: power_supply_core: add constant-current-charge optional property Quentin Schulz
2017-02-14 9:40 ` [PATCH v3 03/18] mfd: axp20x: correct name of temperature data ADC registers Quentin Schulz
2017-02-14 9:40 ` [PATCH v3 04/18] iio: adc: add support for X-Powers AXP20X and AXP22X PMICs ADCs Quentin Schulz
2017-02-19 12:40 ` Jonathan Cameron
2017-02-21 18:03 ` Quentin Schulz
2017-02-25 16:53 ` Jonathan Cameron
2017-02-21 4:32 ` Chen-Yu Tsai
2017-02-14 9:40 ` [PATCH v3 05/18] mfd: axp20x: add ADC cells for AXP20X and AXP22X PMICs Quentin Schulz
2017-02-15 9:37 ` Lee Jones
2017-02-21 4:51 ` Chen-Yu Tsai
2017-02-14 9:41 ` [PATCH v3 06/18] mfd: axp20x: add AC power supply cells for " Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 07/18] ARM: dtsi: axp209: add AC power supply subnode Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 08/18] ARM: dtsi: axp22x: " Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 09/18] ARM: dts: sun8i: sina33: enable ACIN " Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 10/18] ARM: sun5i: chip: " Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 11/18] dt-bindings: power: supply: add AXP20X/AXP22X battery DT binding Quentin Schulz
2017-02-21 4:45 ` Chen-Yu Tsai
2017-02-21 18:05 ` Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 12/18] mfd: axp20x: add CHRG_CTRL1/2/3 to writeable regs for AXP20X/AXP22X Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 13/18] power: supply: add battery driver for AXP20X and AXP22X PMICs Quentin Schulz
2017-02-21 4:44 ` Chen-Yu Tsai
2017-02-21 18:19 ` Quentin Schulz
2017-02-22 6:22 ` Chen-Yu Tsai
2017-02-14 9:41 ` [PATCH v3 14/18] mfd: axp20x: add MFD cells for AXP20X and AXP22X battery driver Quentin Schulz
2017-02-21 4:46 ` Chen-Yu Tsai
2017-02-14 9:41 ` [PATCH v3 15/18] ARM: dtsi: axp209: add battery power supply subnode Quentin Schulz
2017-02-21 4:47 ` Chen-Yu Tsai
2017-02-14 9:41 ` [PATCH v3 16/18] ARM: dtsi: axp22x: " Quentin Schulz
2017-02-21 4:47 ` Chen-Yu Tsai
2017-02-14 9:41 ` [PATCH v3 17/18] ARM: dts: sun8i: sina33: enable " Quentin Schulz
2017-02-21 4:50 ` Chen-Yu Tsai
2017-02-21 18:20 ` Quentin Schulz
2017-02-14 9:41 ` [PATCH v3 18/18] ARM: sun5i: chip: " Quentin Schulz
2017-02-21 4:50 ` Chen-Yu Tsai
2017-02-15 9:36 ` [PATCH v3 00/18] add support for AXP20X and AXP22X power supply drivers Lee Jones
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=20170215004646.23607-1-liam@networkimprov.net \
--to=liam@networkimprov.net \
--cc=bonbons@linux-vserver.org \
--cc=devicetree@vger.kernel.org \
--cc=icenowy@aosc.xyz \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=lee.jones@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=maxime.ripard@free-electrons.com \
--cc=pmeerw@pmeerw.net \
--cc=quentin.schulz@free-electrons.com \
--cc=robh+dt@kernel.org \
--cc=sre@kernel.org \
--cc=thomas.petazzoni@free-electrons.com \
--cc=wens@csie.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 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).