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: linux-gpio@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 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752138AbcF1G7D (ORCPT ); Tue, 28 Jun 2016 02:59:03 -0400 Received: from mail-wm0-f53.google.com ([74.125.82.53]:37467 "EHLO mail-wm0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752051AbcF1G67 (ORCPT ); Tue, 28 Jun 2016 02:58:59 -0400 Date: Tue, 28 Jun 2016 07:59:50 +0100 From: Lee Jones 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 Subject: Re: [PATCH v3 02/10] Documentation: regulator: tps65218: Update examples 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-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1467093980-11458-3-git-send-email-j-keerthy@ti.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is a more common format for DT bindings: dt-bindings: regulator: On Tue, 28 Jun 2016, Keerthy wrote: > This updates the device tree according to the preferred way of parsing Nit: Device Tree > the nodes using the regulator framework. > > Acked-by: Rob Herring > Signed-off-by: Keerthy > --- > .../devicetree/bindings/regulator/tps65218.txt | 87 ++++++++++++++++++---- > 1 file changed, 71 insertions(+), 16 deletions(-) > > 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 > > 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. ./regulator.txt > + The valid names for regulators are: > + tps65217: regulator-dcdc1, regulator-dcdc2, regulator-dcdc3, regulator-dcdc4, > + regulator-dcdc5, regulator-dcdc6, regulator-ldo1, regulator-ls3 > + Each regulator is defined using the standard binding for regulators. > > Example: > +tps65218: tps65218@24 { Is this label used? > + reg = <0x24>; > + compatible = "ti,tps65218"; > + interrupts = ; /* NMIn */ > + interrupt-controller; > + #interrupt-cells = <2>; [...] -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Tue, 28 Jun 2016 07:59:50 +0100 Subject: [PATCH v3 02/10] Documentation: regulator: tps65218: Update examples In-Reply-To: <1467093980-11458-3-git-send-email-j-keerthy@ti.com> References: <1467093980-11458-1-git-send-email-j-keerthy@ti.com> <1467093980-11458-3-git-send-email-j-keerthy@ti.com> Message-ID: <20160628065950.GE6720@dell> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org This is a more common format for DT bindings: dt-bindings: regulator: On Tue, 28 Jun 2016, Keerthy wrote: > This updates the device tree according to the preferred way of parsing Nit: Device Tree > the nodes using the regulator framework. > > Acked-by: Rob Herring > Signed-off-by: Keerthy > --- > .../devicetree/bindings/regulator/tps65218.txt | 87 ++++++++++++++++++---- > 1 file changed, 71 insertions(+), 16 deletions(-) > > 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 > > 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. ./regulator.txt > + The valid names for regulators are: > + tps65217: regulator-dcdc1, regulator-dcdc2, regulator-dcdc3, regulator-dcdc4, > + regulator-dcdc5, regulator-dcdc6, regulator-ldo1, regulator-ls3 > + Each regulator is defined using the standard binding for regulators. > > Example: > +tps65218: tps65218 at 24 { Is this label used? > + reg = <0x24>; > + compatible = "ti,tps65218"; > + interrupts = ; /* NMIn */ > + interrupt-controller; > + #interrupt-cells = <2>; [...] -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog