Re: [PATCH V3 1/2] mfd: Add ROHM BD9571MWV-M PMIC DT bindings

[Lee Jones]

On Tue, 02 May 2017, Marek Vasut 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 
> 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
> ---
>  .../devicetree/bindings/mfd/bd9571mwv.txt  | 49 
> ++
>  1 file changed, 49 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/bd9571mwv.txt

Applied, thanks.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

[PATCH V3 1/2] mfd: Add ROHM BD9571MWV-M PMIC DT bindings

[Marek Vasut]

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
---
 .../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