From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967677AbdADNWd (ORCPT ); Wed, 4 Jan 2017 08:22:33 -0500 Received: from mail-oi0-f67.google.com ([209.85.218.67]:36562 "EHLO mail-oi0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935378AbdADNVT (ORCPT ); Wed, 4 Jan 2017 08:21:19 -0500 Date: Wed, 4 Jan 2017 07:21:07 -0600 From: Rob Herring To: Quentin Schulz Cc: jic23@kernel.org, knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net, mark.rutland@arm.com, wens@csie.org, sre@kernel.org, linux@armlinux.org.uk, maxime.ripard@free-electrons.com, lee.jones@linaro.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, thomas.petazzoni@free-electrons.com, icenowy@aosc.xyz, bonbons@linux-vserver.org Subject: Re: [PATCH 14/22] dt-bindings: power: supply: add AXP20X/AXP22X battery DT binding Message-ID: <20170104132107.tciklylgqvhftb6f@rob-hp-laptop> References: <20170102163723.7939-1-quentin.schulz@free-electrons.com> <20170102163723.7939-15-quentin.schulz@free-electrons.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170102163723.7939-15-quentin.schulz@free-electrons.com> User-Agent: Mutt/1.6.2-neo (2016-08-21) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jan 02, 2017 at 05:37:14PM +0100, Quentin Schulz wrote: > The X-Powers AXP20X and AXP22X PMICs can have a battery as power supply. > > This patch adds the DT binding documentation for the battery power > supply which gets various data from the PMIC, such as the battery status > (charging, discharging, full, dead), current max limit, current current, > battery capacity (in percentage), voltage max and min limits, current > voltage and battery capacity (in Ah). > > Signed-off-by: Quentin Schulz > --- > .../bindings/power/supply/axp20x_battery.txt | 27 ++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > create mode 100644 Documentation/devicetree/bindings/power/supply/axp20x_battery.txt > > diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt > new file mode 100644 > index 0000000..5489d0d > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt > @@ -0,0 +1,27 @@ > +AXP20x and AXP22x battery power supply > + > +Required Properties: > + - compatible, one of: > + "x-powers,axp209-battery-power-supply" > + "x-powers,axp221-battery-power-supply" > + - io-channels: phandles to battery voltage, charge and discharge > + currents ADC channels > + - io-channel-names = "batt_v", "batt_chrg_i", "batt_dischrg_i"; > + > +This node is a subnode of the axp20x/axp22x PMIC. > + > +The AXP20X and AXP22X can read the battery voltage, charge and discharge > +currents of the battery by reading ADC channels from the AXP20X/AXP22X > +ADC. > + > +Example: > + > +&axp209 { > + battery_power_supply: battery_power_supply { Humm, I guess you power-supply is not sufficient, so 'battery-power-supply' and similar for ac. > + compatible = "x-powers,axp209-battery-power-supply"; > + io-channels = <&axp209_adc 7>, <&axp209_adc 8>, > + <&axp209_adc 9>; > + io-channel-names = "batt_v", "batt_chrg_i", > + "batt_dischrg_i"; > + } > +}; > -- > 2.9.3 >