Hi, On Sun, May 1, 2016 at 4:57 PM, Michael Haas <haas@xxxxxxxxxxxxxxxxxxxx> wrote: > Add binding documentation for the ac power supply part of the AXP20x > pmic. > > Signed-off-by: Michael Haas <haas@xxxxxxxxxxxxxxxxxxxx> > Acked-by: Rob Herring <robh@xxxxxxxxxx> > --- > .../bindings/power_supply/axp20x_ac_power.txt | 34 ++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/power_supply/axp20x_ac_power.txt > > diff --git a/Documentation/devicetree/bindings/power_supply/axp20x_ac_power.txt b/Documentation/devicetree/bindings/power_supply/axp20x_ac_power.txt > new file mode 100644 > index 0000000..1cbdcfb > --- /dev/null > +++ b/Documentation/devicetree/bindings/power_supply/axp20x_ac_power.txt > @@ -0,0 +1,34 @@ > +AXP20x AC power supply > + > +Required Properties: > +-compatible: "x-powers,axp202-ac-power-supply" > + > +This node is a subnode of the axp20x PMIC. > + > +Example: > + > +axp209: pmic@34 { > + compatible = "x-powers,axp209"; > + reg = <0x34>; > + interrupt-parent = <&nmi_intc>; > + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; > + interrupt-controller; > + #interrupt-cells = <1>; > + > + regulators { > + x-powers,dcdc-freq = <1500>; > + > + vdd_cpu: dcdc2 { > + regulator-always-on; > + regulator-min-microvolt = <1000000>; > + regulator-max-microvolt = <1450000>; > + regulator-name = "vdd-cpu"; > + }; > + > + ... > + }; You don't need to include this in the example. Just a "..." would suffice. > + > + ac-power-supply: ac-power-supply { > + compatible = "x-powers,axp202-ac-power-supply"; > + }; Also this should come first in alphabetic order. Otherwise, Acked-by: Chen-Yu Tsai <wens@xxxxxxxx> > +}; > -- > 2.8.0 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html