* [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes
@ 2014-09-25 6:31 Johannes Pointner
2014-10-06 21:22 ` Lee Jones
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Pointner @ 2014-09-25 6:31 UTC (permalink / raw)
To: Lee Jones, Samuel Ortiz
Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
devicetree, linux-kernel
Add documentation for compatible property of subnodes.
Signed-off-by: Johannes Pointner <johannes.pointner@br-automation.com>
---
Documentation/devicetree/bindings/regulator/tps65217.txt | 7 ++++++-
.../devicetree/bindings/video/backlight/tps65217-backlight.txt | 9 ++++++---
2 files changed, 12 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
index 4f05d20..8ae3a3b 100644
--- a/Documentation/devicetree/bindings/regulator/tps65217.txt
+++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
@@ -14,6 +14,9 @@ Required properties:
Optional properties:
- ti,pmic-shutdown-controller: Telling the PMIC to shutdown on PWR_EN toggle.
+Required subnode properties:
+- compatible: "ti,tps65217-pmic"
+
The valid names for regulators are:
tps65217: dcdc1, dcdc2, dcdc3, ldo1, ldo2, ldo3 and ldo4
@@ -23,9 +26,11 @@ Example:
tps: tps@24 {
compatible = "ti,tps65217";
+ reg = <0x24>;
ti,pmic-shutdown-controller;
- regulators {
+ tps_pmic: regulators {
+ compatible = "ti,tps65217-pmic";
dcdc1_reg: dcdc1 {
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <1800000>;
diff --git a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
index 5fb9279..f92f154 100644
--- a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
+++ b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
@@ -6,19 +6,22 @@ used to drive LEDs for use as backlights.
Required properties:
- compatible: "ti,tps65217"
- reg: I2C slave address
-- backlight: node for specifying WLED1 and WLED2 lines in TPS65217
+- backlight: subnode for specifying WLED1 and WLED2 lines in TPS65217
+
+Required subnode properties:
+- compatible: "ti,tps65217-bl"
- isel: selection bit, valid values: 1 for ISEL1 (low-level) and 2 for ISEL2 (high-level)
- fdim: PWM dimming frequency, valid values: 100, 200, 500, 1000
- default-brightness: valid values: 0-100
-Each regulator is defined using the standard binding for regulators.
Example:
tps: tps@24 {
reg = <0x24>;
compatible = "ti,tps65217";
- backlight {
+ tps_bl: backlight {
+ compatible = "ti,tps65217-bl";
isel = <1>; /* 1 - ISET1, 2 ISET2 */
fdim = <100>; /* TPS65217_BL_FDIM_100HZ */
default-brightness = <50>;
--
2.1.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes
2014-09-25 6:31 [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes Johannes Pointner
@ 2014-10-06 21:22 ` Lee Jones
2014-10-08 9:14 ` Johannes Pointner
0 siblings, 1 reply; 6+ messages in thread
From: Lee Jones @ 2014-10-06 21:22 UTC (permalink / raw)
To: Johannes Pointner
Cc: Samuel Ortiz, Rob Herring, Pawel Moll, Mark Rutland,
Ian Campbell, Kumar Gala, devicetree, linux-kernel
On Thu, 25 Sep 2014, Johannes Pointner wrote:
> Add documentation for compatible property of subnodes.
>
> Signed-off-by: Johannes Pointner <johannes.pointner@br-automation.com>
> ---
> Documentation/devicetree/bindings/regulator/tps65217.txt | 7 ++++++-
> .../devicetree/bindings/video/backlight/tps65217-backlight.txt | 9 ++++++---
> 2 files changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
> index 4f05d20..8ae3a3b 100644
> --- a/Documentation/devicetree/bindings/regulator/tps65217.txt
> +++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
> @@ -14,6 +14,9 @@ Required properties:
> Optional properties:
> - ti,pmic-shutdown-controller: Telling the PMIC to shutdown on PWR_EN toggle.
>
> +Required subnode properties:
> +- compatible: "ti,tps65217-pmic"
> +
> The valid names for regulators are:
> tps65217: dcdc1, dcdc2, dcdc3, ldo1, ldo2, ldo3 and ldo4
>
> @@ -23,9 +26,11 @@ Example:
>
> tps: tps@24 {
> compatible = "ti,tps65217";
> + reg = <0x24>;
> ti,pmic-shutdown-controller;
>
> - regulators {
> + tps_pmic: regulators {
How is the tps_pmic label used?
> + compatible = "ti,tps65217-pmic";
I'd like to see a '\n' here.
> dcdc1_reg: dcdc1 {
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <1800000>;
> diff --git a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> index 5fb9279..f92f154 100644
> --- a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> +++ b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> @@ -6,19 +6,22 @@ used to drive LEDs for use as backlights.
> Required properties:
> - compatible: "ti,tps65217"
> - reg: I2C slave address
> -- backlight: node for specifying WLED1 and WLED2 lines in TPS65217
> +- backlight: subnode for specifying WLED1 and WLED2 lines in TPS65217
> +
> +Required subnode properties:
> +- compatible: "ti,tps65217-bl"
> - isel: selection bit, valid values: 1 for ISEL1 (low-level) and 2 for ISEL2 (high-level)
> - fdim: PWM dimming frequency, valid values: 100, 200, 500, 1000
> - default-brightness: valid values: 0-100
>
> -Each regulator is defined using the standard binding for regulators.
>
> Example:
>
> tps: tps@24 {
> reg = <0x24>;
> compatible = "ti,tps65217";
I'd like to see a '\n' here.
> - backlight {
> + tps_bl: backlight {
Same here. Is this label used?
> + compatible = "ti,tps65217-bl";
> isel = <1>; /* 1 - ISET1, 2 ISET2 */
> fdim = <100>; /* TPS65217_BL_FDIM_100HZ */
> default-brightness = <50>;
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes
2014-10-06 21:22 ` Lee Jones
@ 2014-10-08 9:14 ` Johannes Pointner
2014-10-08 10:47 ` Lee Jones
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Pointner @ 2014-10-08 9:14 UTC (permalink / raw)
To: Lee Jones
Cc: Samuel Ortiz, Rob Herring, Pawel Moll, Mark Rutland,
Ian Campbell, Kumar Gala, devicetree, linux-kernel
2014-10-06 23:22 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
> On Thu, 25 Sep 2014, Johannes Pointner wrote:
>
>> Add documentation for compatible property of subnodes.
>>
>> Signed-off-by: Johannes Pointner <johannes.pointner@br-automation.com>
>> ---
>> Documentation/devicetree/bindings/regulator/tps65217.txt | 7 ++++++-
>> .../devicetree/bindings/video/backlight/tps65217-backlight.txt | 9 ++++++---
>> 2 files changed, 12 insertions(+), 4 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
>> index 4f05d20..8ae3a3b 100644
>> --- a/Documentation/devicetree/bindings/regulator/tps65217.txt
>> +++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
>> @@ -14,6 +14,9 @@ Required properties:
>> Optional properties:
>> - ti,pmic-shutdown-controller: Telling the PMIC to shutdown on PWR_EN toggle.
>>
>> +Required subnode properties:
>> +- compatible: "ti,tps65217-pmic"
>> +
>> The valid names for regulators are:
>> tps65217: dcdc1, dcdc2, dcdc3, ldo1, ldo2, ldo3 and ldo4
>>
>> @@ -23,9 +26,11 @@ Example:
>>
>> tps: tps@24 {
>> compatible = "ti,tps65217";
>> + reg = <0x24>;
>> ti,pmic-shutdown-controller;
>>
>> - regulators {
>> + tps_pmic: regulators {
>
> How is the tps_pmic label used?
>
>> + compatible = "ti,tps65217-pmic";
>
> I'd like to see a '\n' here.
I will add a newline here.
>
>> dcdc1_reg: dcdc1 {
>> regulator-min-microvolt = <900000>;
>> regulator-max-microvolt = <1800000>;
>> diff --git a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
>> index 5fb9279..f92f154 100644
>> --- a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
>> +++ b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
>> @@ -6,19 +6,22 @@ used to drive LEDs for use as backlights.
>> Required properties:
>> - compatible: "ti,tps65217"
>> - reg: I2C slave address
>> -- backlight: node for specifying WLED1 and WLED2 lines in TPS65217
>> +- backlight: subnode for specifying WLED1 and WLED2 lines in TPS65217
>> +
>> +Required subnode properties:
>> +- compatible: "ti,tps65217-bl"
>> - isel: selection bit, valid values: 1 for ISEL1 (low-level) and 2 for ISEL2 (high-level)
>> - fdim: PWM dimming frequency, valid values: 100, 200, 500, 1000
>> - default-brightness: valid values: 0-100
>>
>> -Each regulator is defined using the standard binding for regulators.
>>
>> Example:
>>
>> tps: tps@24 {
>> reg = <0x24>;
>> compatible = "ti,tps65217";
>
> I'd like to see a '\n' here.
I will add a newline here.
>
>> - backlight {
>> + tps_bl: backlight {
>
> Same here. Is this label used?
>
>> + compatible = "ti,tps65217-bl";
>> isel = <1>; /* 1 - ISET1, 2 ISET2 */
>> fdim = <100>; /* TPS65217_BL_FDIM_100HZ */
>> default-brightness = <50>;
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
Regarding the labels: The backlight label could be used like the
following. I use it like this in my tests with the tilcdc panel driver
which has a backlight support.
panel {
backlight = <&tps_bl>;
panel-info {
ac-bias = <255>;
ac-bias-intrpt = <0>;
dma-burst-sz = <16>;
bpp = <32>;
fdd = <0x80>;
sync-edge = <0>;
sync-ctrl = <1>;
.
.
.
I'm not sure if there could be actual use for the pmic label, but I
thought if I add a label to backlight, I have to add one to the pmic
too to keep it similar.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes
2014-10-08 9:14 ` Johannes Pointner
@ 2014-10-08 10:47 ` Lee Jones
2014-10-08 10:59 ` Johannes Pointner
0 siblings, 1 reply; 6+ messages in thread
From: Lee Jones @ 2014-10-08 10:47 UTC (permalink / raw)
To: Johannes Pointner
Cc: Samuel Ortiz, Rob Herring, Pawel Moll, Mark Rutland,
Ian Campbell, Kumar Gala, devicetree, linux-kernel
On Wed, 08 Oct 2014, Johannes Pointner wrote:
> 2014-10-06 23:22 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
> > On Thu, 25 Sep 2014, Johannes Pointner wrote:
> >
> >> Add documentation for compatible property of subnodes.
> >>
> >> Signed-off-by: Johannes Pointner <johannes.pointner@br-automation.com>
> >> ---
> >> Documentation/devicetree/bindings/regulator/tps65217.txt | 7 ++++++-
> >> .../devicetree/bindings/video/backlight/tps65217-backlight.txt | 9 ++++++---
> >> 2 files changed, 12 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
> >> index 4f05d20..8ae3a3b 100644
> >> --- a/Documentation/devicetree/bindings/regulator/tps65217.txt
> >> +++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
> >> @@ -14,6 +14,9 @@ Required properties:
> >> Optional properties:
> >> - ti,pmic-shutdown-controller: Telling the PMIC to shutdown on PWR_EN toggle.
> >>
> >> +Required subnode properties:
> >> +- compatible: "ti,tps65217-pmic"
> >> +
> >> The valid names for regulators are:
> >> tps65217: dcdc1, dcdc2, dcdc3, ldo1, ldo2, ldo3 and ldo4
> >>
> >> @@ -23,9 +26,11 @@ Example:
> >>
> >> tps: tps@24 {
> >> compatible = "ti,tps65217";
> >> + reg = <0x24>;
> >> ti,pmic-shutdown-controller;
> >>
> >> - regulators {
> >> + tps_pmic: regulators {
> >
> > How is the tps_pmic label used?
> >
> >> + compatible = "ti,tps65217-pmic";
> >
> > I'd like to see a '\n' here.
> I will add a newline here.
> >
> >> dcdc1_reg: dcdc1 {
> >> regulator-min-microvolt = <900000>;
> >> regulator-max-microvolt = <1800000>;
> >> diff --git a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> >> index 5fb9279..f92f154 100644
> >> --- a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> >> +++ b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> >> @@ -6,19 +6,22 @@ used to drive LEDs for use as backlights.
> >> Required properties:
> >> - compatible: "ti,tps65217"
> >> - reg: I2C slave address
> >> -- backlight: node for specifying WLED1 and WLED2 lines in TPS65217
> >> +- backlight: subnode for specifying WLED1 and WLED2 lines in TPS65217
> >> +
> >> +Required subnode properties:
> >> +- compatible: "ti,tps65217-bl"
> >> - isel: selection bit, valid values: 1 for ISEL1 (low-level) and 2 for ISEL2 (high-level)
> >> - fdim: PWM dimming frequency, valid values: 100, 200, 500, 1000
> >> - default-brightness: valid values: 0-100
> >>
> >> -Each regulator is defined using the standard binding for regulators.
> >>
> >> Example:
> >>
> >> tps: tps@24 {
> >> reg = <0x24>;
> >> compatible = "ti,tps65217";
> >
> > I'd like to see a '\n' here.
> I will add a newline here.
> >
> >> - backlight {
> >> + tps_bl: backlight {
> >
> > Same here. Is this label used?
> >
> >> + compatible = "ti,tps65217-bl";
> >> isel = <1>; /* 1 - ISET1, 2 ISET2 */
> >> fdim = <100>; /* TPS65217_BL_FDIM_100HZ */
> >> default-brightness = <50>;
> >
>
> Regarding the labels: The backlight label could be used like the
> following. I use it like this in my tests with the tilcdc panel driver
> which has a backlight support.
>
> panel {
> backlight = <&tps_bl>;
>
> panel-info {
> ac-bias = <255>;
> ac-bias-intrpt = <0>;
> dma-burst-sz = <16>;
> bpp = <32>;
> fdd = <0x80>;
> sync-edge = <0>;
> sync-ctrl = <1>;
> .
> .
> .
>
> I'm not sure if there could be actual use for the pmic label, but I
> thought if I add a label to backlight, I have to add one to the pmic
> too to keep it similar.
No, not at all. If it's not used, you should remove it.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes
2014-10-08 10:47 ` Lee Jones
@ 2014-10-08 10:59 ` Johannes Pointner
2014-10-08 13:30 ` Lee Jones
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Pointner @ 2014-10-08 10:59 UTC (permalink / raw)
To: Lee Jones
Cc: Samuel Ortiz, Rob Herring, Pawel Moll, Mark Rutland,
Ian Campbell, Kumar Gala, devicetree, linux-kernel
2014-10-08 12:47 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
> On Wed, 08 Oct 2014, Johannes Pointner wrote:
>
>> 2014-10-06 23:22 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
>> > On Thu, 25 Sep 2014, Johannes Pointner wrote:
>> >
>> >> Add documentation for compatible property of subnodes.
>> >>
>> >> Signed-off-by: Johannes Pointner <johannes.pointner@br-automation.com>
>> >> ---
>> >> Documentation/devicetree/bindings/regulator/tps65217.txt | 7 ++++++-
>> >> .../devicetree/bindings/video/backlight/tps65217-backlight.txt | 9 ++++++---
>> >> 2 files changed, 12 insertions(+), 4 deletions(-)
>> >>
>> >> diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
>> >> index 4f05d20..8ae3a3b 100644
>> >> --- a/Documentation/devicetree/bindings/regulator/tps65217.txt
>> >> +++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
>> >> @@ -14,6 +14,9 @@ Required properties:
>> >> Optional properties:
>> >> - ti,pmic-shutdown-controller: Telling the PMIC to shutdown on PWR_EN toggle.
>> >>
>> >> +Required subnode properties:
>> >> +- compatible: "ti,tps65217-pmic"
>> >> +
>> >> The valid names for regulators are:
>> >> tps65217: dcdc1, dcdc2, dcdc3, ldo1, ldo2, ldo3 and ldo4
>> >>
>> >> @@ -23,9 +26,11 @@ Example:
>> >>
>> >> tps: tps@24 {
>> >> compatible = "ti,tps65217";
>> >> + reg = <0x24>;
>> >> ti,pmic-shutdown-controller;
>> >>
>> >> - regulators {
>> >> + tps_pmic: regulators {
>> >
>> > How is the tps_pmic label used?
>> >
>> >> + compatible = "ti,tps65217-pmic";
>> >
>> > I'd like to see a '\n' here.
>> I will add a newline here.
>> >
>> >> dcdc1_reg: dcdc1 {
>> >> regulator-min-microvolt = <900000>;
>> >> regulator-max-microvolt = <1800000>;
>> >> diff --git a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
>> >> index 5fb9279..f92f154 100644
>> >> --- a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
>> >> +++ b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
>> >> @@ -6,19 +6,22 @@ used to drive LEDs for use as backlights.
>> >> Required properties:
>> >> - compatible: "ti,tps65217"
>> >> - reg: I2C slave address
>> >> -- backlight: node for specifying WLED1 and WLED2 lines in TPS65217
>> >> +- backlight: subnode for specifying WLED1 and WLED2 lines in TPS65217
>> >> +
>> >> +Required subnode properties:
>> >> +- compatible: "ti,tps65217-bl"
>> >> - isel: selection bit, valid values: 1 for ISEL1 (low-level) and 2 for ISEL2 (high-level)
>> >> - fdim: PWM dimming frequency, valid values: 100, 200, 500, 1000
>> >> - default-brightness: valid values: 0-100
>> >>
>> >> -Each regulator is defined using the standard binding for regulators.
>> >>
>> >> Example:
>> >>
>> >> tps: tps@24 {
>> >> reg = <0x24>;
>> >> compatible = "ti,tps65217";
>> >
>> > I'd like to see a '\n' here.
>> I will add a newline here.
>> >
>> >> - backlight {
>> >> + tps_bl: backlight {
>> >
>> > Same here. Is this label used?
>> >
>> >> + compatible = "ti,tps65217-bl";
>> >> isel = <1>; /* 1 - ISET1, 2 ISET2 */
>> >> fdim = <100>; /* TPS65217_BL_FDIM_100HZ */
>> >> default-brightness = <50>;
>> >
>>
>> Regarding the labels: The backlight label could be used like the
>> following. I use it like this in my tests with the tilcdc panel driver
>> which has a backlight support.
>>
>> panel {
>> backlight = <&tps_bl>;
>>
>> panel-info {
>> ac-bias = <255>;
>> ac-bias-intrpt = <0>;
>> dma-burst-sz = <16>;
>> bpp = <32>;
>> fdd = <0x80>;
>> sync-edge = <0>;
>> sync-ctrl = <1>;
>> .
>> .
>> .
>>
>> I'm not sure if there could be actual use for the pmic label, but I
>> thought if I add a label to backlight, I have to add one to the pmic
>> too to keep it similar.
>
> No, not at all. If it's not used, you should remove it.
OK, my mistake, sorry.
I'll remove the pmic part of the patch and add the newline to the
backlight documentation and resend the patch, if this is OK for you?
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes
2014-10-08 10:59 ` Johannes Pointner
@ 2014-10-08 13:30 ` Lee Jones
0 siblings, 0 replies; 6+ messages in thread
From: Lee Jones @ 2014-10-08 13:30 UTC (permalink / raw)
To: Johannes Pointner
Cc: Samuel Ortiz, Rob Herring, Pawel Moll, Mark Rutland,
Ian Campbell, Kumar Gala, devicetree, linux-kernel
On Wed, 08 Oct 2014, Johannes Pointner wrote:
> 2014-10-08 12:47 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
> > On Wed, 08 Oct 2014, Johannes Pointner wrote:
> >
> >> 2014-10-06 23:22 GMT+02:00 Lee Jones <lee.jones@linaro.org>:
> >> > On Thu, 25 Sep 2014, Johannes Pointner wrote:
> >> >
> >> >> Add documentation for compatible property of subnodes.
> >> >>
> >> >> Signed-off-by: Johannes Pointner <johannes.pointner@br-automation.com>
> >> >> ---
> >> >> Documentation/devicetree/bindings/regulator/tps65217.txt | 7 ++++++-
> >> >> .../devicetree/bindings/video/backlight/tps65217-backlight.txt | 9 ++++++---
> >> >> 2 files changed, 12 insertions(+), 4 deletions(-)
> >> >>
> >> >> diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
> >> >> index 4f05d20..8ae3a3b 100644
> >> >> --- a/Documentation/devicetree/bindings/regulator/tps65217.txt
> >> >> +++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
> >> >> @@ -14,6 +14,9 @@ Required properties:
> >> >> Optional properties:
> >> >> - ti,pmic-shutdown-controller: Telling the PMIC to shutdown on PWR_EN toggle.
> >> >>
> >> >> +Required subnode properties:
> >> >> +- compatible: "ti,tps65217-pmic"
> >> >> +
> >> >> The valid names for regulators are:
> >> >> tps65217: dcdc1, dcdc2, dcdc3, ldo1, ldo2, ldo3 and ldo4
> >> >>
> >> >> @@ -23,9 +26,11 @@ Example:
> >> >>
> >> >> tps: tps@24 {
> >> >> compatible = "ti,tps65217";
> >> >> + reg = <0x24>;
> >> >> ti,pmic-shutdown-controller;
> >> >>
> >> >> - regulators {
> >> >> + tps_pmic: regulators {
> >> >
> >> > How is the tps_pmic label used?
> >> >
> >> >> + compatible = "ti,tps65217-pmic";
> >> >
> >> > I'd like to see a '\n' here.
> >> I will add a newline here.
> >> >
> >> >> dcdc1_reg: dcdc1 {
> >> >> regulator-min-microvolt = <900000>;
> >> >> regulator-max-microvolt = <1800000>;
> >> >> diff --git a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> >> >> index 5fb9279..f92f154 100644
> >> >> --- a/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> >> >> +++ b/Documentation/devicetree/bindings/video/backlight/tps65217-backlight.txt
> >> >> @@ -6,19 +6,22 @@ used to drive LEDs for use as backlights.
> >> >> Required properties:
> >> >> - compatible: "ti,tps65217"
> >> >> - reg: I2C slave address
> >> >> -- backlight: node for specifying WLED1 and WLED2 lines in TPS65217
> >> >> +- backlight: subnode for specifying WLED1 and WLED2 lines in TPS65217
> >> >> +
> >> >> +Required subnode properties:
> >> >> +- compatible: "ti,tps65217-bl"
> >> >> - isel: selection bit, valid values: 1 for ISEL1 (low-level) and 2 for ISEL2 (high-level)
> >> >> - fdim: PWM dimming frequency, valid values: 100, 200, 500, 1000
> >> >> - default-brightness: valid values: 0-100
> >> >>
> >> >> -Each regulator is defined using the standard binding for regulators.
> >> >>
> >> >> Example:
> >> >>
> >> >> tps: tps@24 {
> >> >> reg = <0x24>;
> >> >> compatible = "ti,tps65217";
> >> >
> >> > I'd like to see a '\n' here.
> >> I will add a newline here.
> >> >
> >> >> - backlight {
> >> >> + tps_bl: backlight {
> >> >
> >> > Same here. Is this label used?
> >> >
> >> >> + compatible = "ti,tps65217-bl";
> >> >> isel = <1>; /* 1 - ISET1, 2 ISET2 */
> >> >> fdim = <100>; /* TPS65217_BL_FDIM_100HZ */
> >> >> default-brightness = <50>;
> >> >
> >>
> >> Regarding the labels: The backlight label could be used like the
> >> following. I use it like this in my tests with the tilcdc panel driver
> >> which has a backlight support.
> >>
> >> panel {
> >> backlight = <&tps_bl>;
> >>
> >> panel-info {
> >> ac-bias = <255>;
> >> ac-bias-intrpt = <0>;
> >> dma-burst-sz = <16>;
> >> bpp = <32>;
> >> fdd = <0x80>;
> >> sync-edge = <0>;
> >> sync-ctrl = <1>;
> >> .
> >> .
> >> .
> >>
> >> I'm not sure if there could be actual use for the pmic label, but I
> >> thought if I add a label to backlight, I have to add one to the pmic
> >> too to keep it similar.
> >
> > No, not at all. If it's not used, you should remove it.
>
> OK, my mistake, sorry.
> I'll remove the pmic part of the patch and add the newline to the
> backlight documentation and resend the patch, if this is OK for you?
Yes, fine.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-10-08 13:30 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-25 6:31 [PATCH v2 2/2] dt: bindings: tps65217: add compatible property for subnodes Johannes Pointner
2014-10-06 21:22 ` Lee Jones
2014-10-08 9:14 ` Johannes Pointner
2014-10-08 10:47 ` Lee Jones
2014-10-08 10:59 ` Johannes Pointner
2014-10-08 13:30 ` Lee Jones
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).