Hi Marek, On Sun, Apr 16, 2017 at 8:07 PM, Marek Vasut <marek.va...@gmail.com> wrote: > Add DT bindings for the ROHM BD9571MWV-M PMIC. This PMIC has > the following features: > - multiple voltage monitors for 1V8, 2V5, 3V3 voltage rail > - one voltage regulator for DVFS > - two GPIOs > > Signed-off-by: Marek Vasut <marek.vasut+rene...@gmail.com> > Cc: devicet...@vger.kernel.org > Cc: Rob Herring <r...@kernel.org> > Cc: Geert Uytterhoeven <geert+rene...@glider.be> > Cc: linux-renesas-soc@vger.kernel.org > --- > .../devicetree/bindings/mfd/bd9571mwv.txt | 49 > ++++++++++++++++++++++ > 1 file changed, 49 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/bd9571mwv.txt > > diff --git a/Documentation/devicetree/bindings/mfd/bd9571mwv.txt > b/Documentation/devicetree/bindings/mfd/bd9571mwv.txt > new file mode 100644 > index 000000000000..0952ee3bc30b > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/bd9571mwv.txt > @@ -0,0 +1,49 @@ > +* ROHM BD9571MWV Power Management Integrated Circuit (PMIC) bindings > + > +Required properties: > + - compatible : Should be "rohm,bd9571mwv". > + - reg : I2C slave address. > + - interrupt-parent : Phandle to the parent interrupt controller. > + - interrupts : The interrupt line the device is connected to. > + - interrupt-controller : Marks the device node as an interrupt > controller. > + - #interrupt-cells : The number of cells to describe an IRQ, should be 2. > + The first cell is the IRQ number. > + The second cell is the flags, encoded as trigger > + masks from ../interrupt-controller/interrupts.txt. > + - gpio-controller : Marks the device node as a GPIO Controller. > + - #gpio-cells : Should be two. The first cell is the pin number > and > + the second cell is used to specify flags. > + See ../gpio/gpio.txt for more information. > + - regulators: : List of child nodes that specify the regulator > + initialization data. Child nodes must be named > + after their hardware counterparts: > + - vd18 > + - vd25 > + - vd33 > + - dvfs
Missing vd09, which is used for VDD0.8 on Salvator-X. > + Each child node is defined using the standard > + binding for regulators. > + > +Example: > + > + pmic: bd9571mwv@30 { > + compatible = "rohm,bd9571mwv"; > + reg = <0x30>; > + interrupt-parent = <&gpio2>; > + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; > + interrupt-controller; > + #interrupt-cells = <2>; > + gpio-controller; > + #gpio-cells = <2>; > + > + regulators { > + dvfs: dvfs { > + compatible = "regulator-fixed"; What's the purpose of this compatible value? > + regulator-name = "dvfs"; > + regulator-min-microvolt = <750000>; > + regulator-max-microvolt = <1030000>; > + regulator-boot-on; > + regulator-always-on; > + }; > + }; > + }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds