From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v3 02/10] Documentation: regulator: tps65218: Update examples Date: Tue, 28 Jun 2016 07:59:50 +0100 Message-ID: <20160628065950.GE6720@dell> References: <1467093980-11458-1-git-send-email-j-keerthy@ti.com> <1467093980-11458-3-git-send-email-j-keerthy@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <1467093980-11458-3-git-send-email-j-keerthy@ti.com> Sender: linux-input-owner@vger.kernel.org To: Keerthy Cc: dmitry.torokhov@gmail.com, linus.walleij@linaro.org, gnurou@gmail.com, broonie@kernel.org, tony@atomide.com, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-gpio@vger.kernel.org, linux-input@vger.kernel.org, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org This is a more common format for DT bindings: dt-bindings: regulator:=20 On Tue, 28 Jun 2016, Keerthy wrote: > This updates the device tree according to the preferred way of parsin= g Nit: Device Tree > the nodes using the regulator framework. >=20 > Acked-by: Rob Herring > Signed-off-by: Keerthy > --- > .../devicetree/bindings/regulator/tps65218.txt | 87 ++++++++++++= ++++++---- > 1 file changed, 71 insertions(+), 16 deletions(-) >=20 > diff --git a/Documentation/devicetree/bindings/regulator/tps65218.txt= b/Documentation/devicetree/bindings/regulator/tps65218.txt > index fccc1d2..5e1888f 100644 > --- a/Documentation/devicetree/bindings/regulator/tps65218.txt > +++ b/Documentation/devicetree/bindings/regulator/tps65218.txt > @@ -1,23 +1,78 @@ > TPS65218 family of regulators > =20 > Required properties: > -For tps65218 regulators/LDOs > -- compatible: > - - "ti,tps65218-dcdc1" for DCDC1 > - - "ti,tps65218-dcdc2" for DCDC2 > - - "ti,tps65218-dcdc3" for DCDC3 > - - "ti,tps65218-dcdc4" for DCDC4 > - - "ti,tps65218-dcdc5" for DCDC5 > - - "ti,tps65218-dcdc6" for DCDC6 > - - "ti,tps65218-ldo1" for LDO1 > - > -Optional properties: > -- Any optional property defined in bindings/regulator/regulator.txt > +- compatible: "ti,tps65218" > +- reg: I2C slave address '\n' here for clarity. > +- list of regulators provided by this controller, must be named Sentences start with an uppercase char. > + after their hardware counterparts: dcdc[1-6] and ldo1 > +- This is the list of child nodes that specify the regulator > + initialization data for defined regulators. Not all regulators for= the given > + device need to be present. The definition for each of these nodes = is defined > + using the standard binding for regulators found at > + Documentation/devicetree/bindings/regulator/regulator.txt. Use relative path names for brevity. =2E/regulator.txt > + The valid names for regulators are: > + tps65217: regulator-dcdc1, regulator-dcdc2, regulator-dcdc3, regul= ator-dcdc4, > + regulator-dcdc5, regulator-dcdc6, regulator-ldo1, regulator-ls3 > + Each regulator is defined using the standard binding for regulator= s. > =20 > Example: > +tps65218: tps65218@24 { Is this label used? > + reg =3D <0x24>; > + compatible =3D "ti,tps65218"; > + interrupts =3D ; /* NMIn */ > + interrupt-controller; > + #interrupt-cells =3D <2>; [...] --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-input" = in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html