Re: [PATCH V4 1/2] mfd: Add ROHM BD9571MWV-M PMIC DT bindings
Hi Marek, You forgot to CC Lee (added ;-) On Mon, Jul 17, 2017 at 10:45 PM, Marek Vasutwrote: > 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 > Cc: devicet...@vger.kernel.org > Cc: Rob Herring > Cc: Geert Uytterhoeven > Cc: linux-renesas-soc@vger.kernel.org > Acked-by: Rob Herring > --- > V2: - Drop the compatible = "regulator-fixed" from the binding example, > it should not be there. > - List the VD09 regulator > V3: Replace bd9571@30 with pmic@30 > V4: No changes > --- > .../devicetree/bindings/mfd/bd9571mwv.txt | 49 > ++ > 1 file changed, 49 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/bd9571mwv.txt Anyway, as there were no changes in V4, Lee can just apply V3, for which he was CCed. Thanks! 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
[PATCH V4 1/2] mfd: Add ROHM BD9571MWV-M PMIC DT bindings
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 VasutCc: devicet...@vger.kernel.org Cc: Rob Herring Cc: Geert Uytterhoeven Cc: linux-renesas-soc@vger.kernel.org Acked-by: Rob Herring --- V2: - Drop the compatible = "regulator-fixed" from the binding example, it should not be there. - List the VD09 regulator V3: Replace bd9571@30 with pmic@30 V4: No changes --- .../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 ..9ab216a851d5 --- /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: +- vd09 +- vd18 +- vd25 +- vd33 +- dvfs + Each child node is defined using the standard + binding for regulators. + +Example: + + pmic: pmic@30 { + compatible = "rohm,bd9571mwv"; + reg = <0x30>; + interrupt-parent = <>; + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; + interrupt-controller; + #interrupt-cells = <2>; + gpio-controller; + #gpio-cells = <2>; + + regulators { + dvfs: dvfs { + regulator-name = "dvfs"; + regulator-min-microvolt = <75>; + regulator-max-microvolt = <103>; + regulator-boot-on; + regulator-always-on; + }; + }; + }; -- 2.11.0