From: carlo@caione•org (Carlo Caione)
To: linux-arm-kernel@lists•infradead.org
Subject: [linux-sunxi] Re: [PATCH 2/7] mfd: AXP20x: Add bindings documentation
Date: Sat, 8 Mar 2014 12:38:43 +0100 [thread overview]
Message-ID: <20140308113843.GB14009@localhost.fastwebnet.it> (raw)
In-Reply-To: <20140307181300.GZ607@lukather>
On Fri, Mar 07, 2014 at 07:13:00PM +0100, Maxime Ripard wrote:
> On Sat, Mar 01, 2014 at 05:45:47PM +0100, Carlo Caione wrote:
> > Bindings documentation for the AXP20x driver. In this file also two
> > sub-nodes (PEK and regulators) are documented.
> >
> > Signed-off-by: Carlo Caione <carlo@caione•org>
> > ---
> > Documentation/devicetree/bindings/mfd/axp20x.txt | 93 ++++++++++++++++++++++++
> > 1 file changed, 93 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/mfd/axp20x.txt
> >
> > diff --git a/Documentation/devicetree/bindings/mfd/axp20x.txt b/Documentation/devicetree/bindings/mfd/axp20x.txt
> > new file mode 100644
> > index 0000000..ae3e3c4
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/mfd/axp20x.txt
> > @@ -0,0 +1,93 @@
> > +* axp20x device tree bindings
> > +
> > +The axp20x family current members :-
> > +axp202 (X-Powers)
> > +axp209 (X-Powers)
> > +
> > +Required properties:
> > +- compatible : Should be "x-powers,axp202" or "x-powers,axp209"
> > +- interrupt-controller : axp20x has its own internal IRQs
> > +- #interrupt-cells : Should be set to 1
> > +- interrupt-parent : The parent interrupt controller
> > +- interrupts : Interrupt specifiers for interrupt sources
> > +- reg : The I2C slave address for the AXP chip
> > +- axp,system-power-controller : Telling whether or not this pmic is
> > + controlling the system power
> > +
> > +Sub-nodes:
> > +* regulators : Contain the regulator nodes. The regulators are bound using
> > + their name as listed here: dcdc2, dcdc3, ldo1, ldo2, ldo3,
> > + ldo4, ldo5.
> > + The bindings details of individual regulator device can be found in:
> > + Documentation/devicetree/bindings/regulator/regulator.txt with the
> > + exception of:
> > +
> > + - dcdc-freq : defines the work frequency of DC-DC in KHz
> > + (range: 750-1875)
> > + - dcdc-workmode : 1 for PWM mode, 0 for AUTO mode
>
> You don't seem to always set this. You should mention that it is
> optional, and which default value it has.
Fix in v2.
> > +
> > +* axp20x-pek : Power Enable Key
> > + - compatible : should be "x-powers,axp20x-pek"
>
> Why is this needed for?
>
> Plus, please don't use any generic, or pattern matching compatibles,
> but rather precise ones, so that if it is needed, we can add any quirk
> we want.
It is not needed, in fact I deleted it from dts. It is a refuse from the
previous submission. I'll fix it.
> > +Example:
> > +
> > +axp: axp20x at 34 {
> > + reg = <0x34>;
> > + interrupt-parent = <&nmi_intc>;
> > + interrupts = <0 8>;
> > +
> > + axp,system-power-controller;
> > +
> > + compatible = "x-powers,axp209";
> > + interrupt-controller;
> > + #interrupt-cells = <1>;
> > +
> > + axp20x-pek {
> > + compatible = "x-powers,axp20x-pek";
> > + };
> > +
> > + regulators {
> > + dcdc-freq = "8";
> > +
> > + axp_dcdc2: dcdc2 {
> > + regulator-min-microvolt = <700000>;
> > + regulator-max-microvolt = <2275000>;
> > + dcdc-workmode = <0>;
> > + regulator-always-on;
> > + };
> > +
> > + axp_dcdc3: dcdc3 {
> > + regulator-min-microvolt = <700000>;
> > + regulator-max-microvolt = <3500000>;
> > + dcdc-workmode = <0>;
> > + regulator-always-on;
> > + };
> > +
> > + axp_ldo1: ldo1 {
> > + regulator-min-microvolt = <1300000>;
> > + regulator-max-microvolt = <1300000>;
> > + };
> > +
> > + axp_ldo2: ldo2 {
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-always-on;
> > + };
> > +
> > + axp_ldo3: ldo3 {
> > + regulator-min-microvolt = <700000>;
> > + regulator-max-microvolt = <3500000>;
> > + };
> > +
> > + axp_ldo4: ldo4 {
> > + regulator-min-microvolt = <1250000>;
> > + regulator-max-microvolt = <3300000>;
> > + };
> > +
> > + axp_ldo5: ldo5 {
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <3300000>;
> > + };
> > + };
> > +};
> > +
> > --
> > 1.8.3.2
> >
--
Carlo Caione
next prev parent reply other threads:[~2014-03-08 11:38 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-01 16:45 [PATCH 0/7] mfd: AXP20x: Add support for AXP202 and AXP209 Carlo Caione
2014-03-01 16:45 ` [PATCH 1/7] mfd: AXP20x: Add mfd driver for AXP20x PMIC Carlo Caione
2014-03-07 18:09 ` Maxime Ripard
2014-03-08 11:31 ` Carlo Caione
2014-03-09 9:11 ` Maxime Ripard
2014-03-01 16:45 ` [PATCH 2/7] mfd: AXP20x: Add bindings documentation Carlo Caione
2014-03-07 18:13 ` Maxime Ripard
2014-03-08 11:38 ` Carlo Caione [this message]
2014-03-01 16:45 ` [PATCH 3/7] ARM: dts: cubieboard2: Add AXP209 support Carlo Caione
2014-03-07 18:13 ` Maxime Ripard
2014-03-01 16:45 ` [PATCH 4/7] input: misc: Add driver for AXP20x Power Enable Key Carlo Caione
2014-03-07 18:18 ` Maxime Ripard
2014-03-01 16:45 ` [PATCH 5/7] input: misc: Add ABI docs for AXP20x PEK Carlo Caione
2014-03-01 16:45 ` [PATCH 6/7] regulator: AXP20x: Add support for regulators subsystem Carlo Caione
2014-03-03 1:56 ` Mark Brown
2014-03-04 20:56 ` [linux-sunxi] " Carlo Caione
2014-03-11 19:24 ` Carlo Caione
2014-03-11 19:29 ` Mark Brown
2014-03-11 21:06 ` Carlo Caione
2014-03-12 0:38 ` Mark Brown
2014-03-07 18:22 ` Maxime Ripard
2014-03-08 11:43 ` [linux-sunxi] " Carlo Caione
2014-03-09 7:51 ` Mark Brown
2014-03-09 8:56 ` Carlo Caione
2014-03-01 16:45 ` [PATCH 7/7] ARM: dts: Cubieboard2: Add support for AXP209 regulators Carlo Caione
2014-03-01 16:56 ` [PATCH 0/7] mfd: AXP20x: Add support for AXP202 and AXP209 Hans de Goede
2014-03-01 17:17 ` Carlo Caione
2014-03-01 19:29 ` [linux-sunxi] " Hans de Goede
2014-03-05 20:02 ` Hans de Goede
2014-03-05 20:46 ` Carlo Caione
2014-03-06 10:07 ` Hans de Goede
2014-03-06 10:10 ` Carlo Caione
2014-03-06 11:05 ` Hans de Goede
2014-03-06 11:32 ` Carlo Caione
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=20140308113843.GB14009@localhost.fastwebnet.it \
--to=carlo@caione$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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