Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Rob Herring 
On Mon, Dec 11, 2017 at 9:51 AM, Rob Herring  wrote:
> On Thu, Dec 7, 2017 at 5:04 PM, Dan Murphy  wrote:
>> Rob
>>
>>
>> On 12/07/2017 04:49 PM, Rob Herring wrote:
>>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
 This adds the devicetree bindings for the LM3692x
 I2C LED string driver.

 Acked-by: Pavel Machek 
 Signed-off-by: Dan Murphy 
 ---

 v8 - Added address-cells and size-cells as well as child node reg - 
 https://patchwork.kernel.org/patch/10091259/
 v7 - No changes - https://patchwork.kernel.org/patch/10087475/
 v6 - No changes -https://patchwork.kernel.org/patch/10085567/
 v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
 v4 - Fix example node, added trigger entry, removed ambiguous x for 
 compatible and
 added common.txt pointer for label - 
 https://patchwork.kernel.org/patch/10060107
 v3 - No changes
 v2 - No changes - https://patchwork.kernel.org/patch/10056677/

  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
 ++
  1 file changed, 47 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

 diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
 b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 new file mode 100644
 index ..84f69342d879
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 @@ -0,0 +1,47 @@
 +* Texas Instruments - LM3692x Highly Efficient White LED Driver
 +
 +The LM3692x is an ultra-compact, highly efficient,
 +white-LED driver designed for LCD display backlighting.
 +
 +The main difference between the LM36922 and LM36923 is the number of
 +LED strings it supports.  The LM36922 supports two strings while the 
 LM36923
 +supports three strings.
 +
 +Required properties:
 +- compatible:
 +"ti,lm36922"
 +"ti,lm36923"
 +- reg :  I2C slave address
 +- #address-cells : 1
 +- #size-cells : 0
 +
 +Optional properties:
 +- label : see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Should be a child prop.
>>
>> Thanks I forgot to move this to Optional Child Properties.
>>
>>>
 +- enable-gpios : gpio pin to enable/disable the device.
 +- vled-supply : LED supply
 +- linux,default-trigger : (optional)
 +   see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Ditto.
>>
>> Ack
>>
>>>
 +
 +Required child properties:
 +- reg : 0
 +
 +Example:
 +
 +lm3692x@36 {
>>>
>>> leds@36
>>
>> Rob why does this need to be leds?  Is this because it would be a child of 
>> an I2C node?
>
> Because node names are supposed to follow the generic type of device
> they are (e.g. serial, ethernet, bus, etc.), not the model of device.
> And yes, there are many examples that don't follow this.

Actually, it should be led-controller and led or leds be used for the
LED child nodes (and gpio-led or pwd-led bindings), so like this:

led-controller@12 {
  reg = <12>;

  led@0 {
reg = <0>;
  };
  led@1 {
reg = <1>;
  };
};

Rob

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Rob Herring 
On Mon, Dec 11, 2017 at 9:51 AM, Rob Herring  wrote:
> On Thu, Dec 7, 2017 at 5:04 PM, Dan Murphy  wrote:
>> Rob
>>
>>
>> On 12/07/2017 04:49 PM, Rob Herring wrote:
>>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
 This adds the devicetree bindings for the LM3692x
 I2C LED string driver.

 Acked-by: Pavel Machek 
 Signed-off-by: Dan Murphy 
 ---

 v8 - Added address-cells and size-cells as well as child node reg - 
 https://patchwork.kernel.org/patch/10091259/
 v7 - No changes - https://patchwork.kernel.org/patch/10087475/
 v6 - No changes -https://patchwork.kernel.org/patch/10085567/
 v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
 v4 - Fix example node, added trigger entry, removed ambiguous x for 
 compatible and
 added common.txt pointer for label - 
 https://patchwork.kernel.org/patch/10060107
 v3 - No changes
 v2 - No changes - https://patchwork.kernel.org/patch/10056677/

  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
 ++
  1 file changed, 47 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

 diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
 b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 new file mode 100644
 index ..84f69342d879
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 @@ -0,0 +1,47 @@
 +* Texas Instruments - LM3692x Highly Efficient White LED Driver
 +
 +The LM3692x is an ultra-compact, highly efficient,
 +white-LED driver designed for LCD display backlighting.
 +
 +The main difference between the LM36922 and LM36923 is the number of
 +LED strings it supports.  The LM36922 supports two strings while the 
 LM36923
 +supports three strings.
 +
 +Required properties:
 +- compatible:
 +"ti,lm36922"
 +"ti,lm36923"
 +- reg :  I2C slave address
 +- #address-cells : 1
 +- #size-cells : 0
 +
 +Optional properties:
 +- label : see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Should be a child prop.
>>
>> Thanks I forgot to move this to Optional Child Properties.
>>
>>>
 +- enable-gpios : gpio pin to enable/disable the device.
 +- vled-supply : LED supply
 +- linux,default-trigger : (optional)
 +   see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Ditto.
>>
>> Ack
>>
>>>
 +
 +Required child properties:
 +- reg : 0
 +
 +Example:
 +
 +lm3692x@36 {
>>>
>>> leds@36
>>
>> Rob why does this need to be leds?  Is this because it would be a child of 
>> an I2C node?
>
> Because node names are supposed to follow the generic type of device
> they are (e.g. serial, ethernet, bus, etc.), not the model of device.
> And yes, there are many examples that don't follow this.

Actually, it should be led-controller and led or leds be used for the
LED child nodes (and gpio-led or pwd-led bindings), so like this:

led-controller@12 {
  reg = <12>;

  led@0 {
reg = <0>;
  };
  led@1 {
reg = <1>;
  };
};

Rob

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Dan Murphy 
Rob

On 12/11/2017 09:51 AM, Rob Herring wrote:
> On Thu, Dec 7, 2017 at 5:04 PM, Dan Murphy  wrote:
>> Rob
>>
>>
>> On 12/07/2017 04:49 PM, Rob Herring wrote:
>>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
 This adds the devicetree bindings for the LM3692x
 I2C LED string driver.

 Acked-by: Pavel Machek 
 Signed-off-by: Dan Murphy 
 ---

 v8 - Added address-cells and size-cells as well as child node reg - 
 https://patchwork.kernel.org/patch/10091259/
 v7 - No changes - https://patchwork.kernel.org/patch/10087475/
 v6 - No changes -https://patchwork.kernel.org/patch/10085567/
 v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
 v4 - Fix example node, added trigger entry, removed ambiguous x for 
 compatible and
 added common.txt pointer for label - 
 https://patchwork.kernel.org/patch/10060107
 v3 - No changes
 v2 - No changes - https://patchwork.kernel.org/patch/10056677/

  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
 ++
  1 file changed, 47 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

 diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
 b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 new file mode 100644
 index ..84f69342d879
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 @@ -0,0 +1,47 @@
 +* Texas Instruments - LM3692x Highly Efficient White LED Driver
 +
 +The LM3692x is an ultra-compact, highly efficient,
 +white-LED driver designed for LCD display backlighting.
 +
 +The main difference between the LM36922 and LM36923 is the number of
 +LED strings it supports.  The LM36922 supports two strings while the 
 LM36923
 +supports three strings.
 +
 +Required properties:
 +- compatible:
 +"ti,lm36922"
 +"ti,lm36923"
 +- reg :  I2C slave address
 +- #address-cells : 1
 +- #size-cells : 0
 +
 +Optional properties:
 +- label : see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Should be a child prop.
>>
>> Thanks I forgot to move this to Optional Child Properties.
>>
>>>
 +- enable-gpios : gpio pin to enable/disable the device.
 +- vled-supply : LED supply
 +- linux,default-trigger : (optional)
 +   see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Ditto.
>>
>> Ack
>>
>>>
 +
 +Required child properties:
 +- reg : 0
 +
 +Example:
 +
 +lm3692x@36 {
>>>
>>> leds@36
>>
>> Rob why does this need to be leds?  Is this because it would be a child of 
>> an I2C node?
> 
> Because node names are supposed to follow the generic type of device
> they are (e.g. serial, ethernet, bus, etc.), not the model of device.
> And yes, there are many examples that don't follow this.
> 

OK.  I guess I will need to update the code to take the compatible or for my 
case the extract the name from
the struct i2c_device_id.

Jacek

I am going to have to update the code to extract the name differently when 
there is no label.  I don't think
pulling the parent node name will scale in the future.


Dan

>> Jacek
>> Would this not cause and issue for your proposal to take the parent node 
>> name as part of the LED label?
>>
>> Dan


-- 
--
Dan Murphy

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Dan Murphy 
Rob

On 12/11/2017 09:51 AM, Rob Herring wrote:
> On Thu, Dec 7, 2017 at 5:04 PM, Dan Murphy  wrote:
>> Rob
>>
>>
>> On 12/07/2017 04:49 PM, Rob Herring wrote:
>>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
 This adds the devicetree bindings for the LM3692x
 I2C LED string driver.

 Acked-by: Pavel Machek 
 Signed-off-by: Dan Murphy 
 ---

 v8 - Added address-cells and size-cells as well as child node reg - 
 https://patchwork.kernel.org/patch/10091259/
 v7 - No changes - https://patchwork.kernel.org/patch/10087475/
 v6 - No changes -https://patchwork.kernel.org/patch/10085567/
 v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
 v4 - Fix example node, added trigger entry, removed ambiguous x for 
 compatible and
 added common.txt pointer for label - 
 https://patchwork.kernel.org/patch/10060107
 v3 - No changes
 v2 - No changes - https://patchwork.kernel.org/patch/10056677/

  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
 ++
  1 file changed, 47 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

 diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
 b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 new file mode 100644
 index ..84f69342d879
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 @@ -0,0 +1,47 @@
 +* Texas Instruments - LM3692x Highly Efficient White LED Driver
 +
 +The LM3692x is an ultra-compact, highly efficient,
 +white-LED driver designed for LCD display backlighting.
 +
 +The main difference between the LM36922 and LM36923 is the number of
 +LED strings it supports.  The LM36922 supports two strings while the 
 LM36923
 +supports three strings.
 +
 +Required properties:
 +- compatible:
 +"ti,lm36922"
 +"ti,lm36923"
 +- reg :  I2C slave address
 +- #address-cells : 1
 +- #size-cells : 0
 +
 +Optional properties:
 +- label : see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Should be a child prop.
>>
>> Thanks I forgot to move this to Optional Child Properties.
>>
>>>
 +- enable-gpios : gpio pin to enable/disable the device.
 +- vled-supply : LED supply
 +- linux,default-trigger : (optional)
 +   see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Ditto.
>>
>> Ack
>>
>>>
 +
 +Required child properties:
 +- reg : 0
 +
 +Example:
 +
 +lm3692x@36 {
>>>
>>> leds@36
>>
>> Rob why does this need to be leds?  Is this because it would be a child of 
>> an I2C node?
> 
> Because node names are supposed to follow the generic type of device
> they are (e.g. serial, ethernet, bus, etc.), not the model of device.
> And yes, there are many examples that don't follow this.
> 

OK.  I guess I will need to update the code to take the compatible or for my 
case the extract the name from
the struct i2c_device_id.

Jacek

I am going to have to update the code to extract the name differently when 
there is no label.  I don't think
pulling the parent node name will scale in the future.


Dan

>> Jacek
>> Would this not cause and issue for your proposal to take the parent node 
>> name as part of the LED label?
>>
>> Dan


-- 
--
Dan Murphy

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Rob Herring 
On Sat, Dec 9, 2017 at 3:21 PM, Jacek Anaszewski
 wrote:
> On 12/08/2017 12:04 AM, Dan Murphy wrote:
>> Rob
>>
>>
>> On 12/07/2017 04:49 PM, Rob Herring wrote:
>>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
 This adds the devicetree bindings for the LM3692x
 I2C LED string driver.

 Acked-by: Pavel Machek 
 Signed-off-by: Dan Murphy 
 ---

 v8 - Added address-cells and size-cells as well as child node reg - 
 https://patchwork.kernel.org/patch/10091259/
 v7 - No changes - https://patchwork.kernel.org/patch/10087475/
 v6 - No changes -https://patchwork.kernel.org/patch/10085567/
 v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
 v4 - Fix example node, added trigger entry, removed ambiguous x for 
 compatible and
 added common.txt pointer for label - 
 https://patchwork.kernel.org/patch/10060107
 v3 - No changes
 v2 - No changes - https://patchwork.kernel.org/patch/10056677/

  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
 ++
  1 file changed, 47 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

 diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
 b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 new file mode 100644
 index ..84f69342d879
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 @@ -0,0 +1,47 @@
 +* Texas Instruments - LM3692x Highly Efficient White LED Driver
 +
 +The LM3692x is an ultra-compact, highly efficient,
 +white-LED driver designed for LCD display backlighting.
 +
 +The main difference between the LM36922 and LM36923 is the number of
 +LED strings it supports.  The LM36922 supports two strings while the 
 LM36923
 +supports three strings.
 +
 +Required properties:
 +   - compatible:
 +   "ti,lm36922"
 +   "ti,lm36923"
 +   - reg :  I2C slave address
 +   - #address-cells : 1
 +   - #size-cells : 0
 +
 +Optional properties:
 +   - label : see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Should be a child prop.
>>
>> Thanks I forgot to move this to Optional Child Properties.
>>
>>>
 +   - enable-gpios : gpio pin to enable/disable the device.
 +   - vled-supply : LED supply
 +   - linux,default-trigger : (optional)
 +  see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Ditto.
>>
>> Ack
>>
>>>
 +
 +Required child properties:
 +   - reg : 0
 +
 +Example:
 +
 +lm3692x@36 {
>>>
>>> leds@36
>>
>> Rob why does this need to be leds?  Is this because it would be a child of 
>> an I2C node?
>>
>> Jacek
>> Would this not cause and issue for your proposal to take the parent node 
>> name as part of the LED label?
>
> Yes, it would.
>
> Rob, does something prevent us from adding a requirement that
> LED controller DT node has to contain the chip name (besides the
> unit address)? Many current LED controller nodes apply this pattern.

Only because we've been lax on the naming. We want to use generic
names because the DT spec says to and it provides a way to match nodes
to validation checks.

I only think we should use the chip name if the device doesn't fit
some standard class.

> Also, I would appreciate if you could express your opinion on
> the patch [0].
>
> [0] https://patchwork.kernel.org/patch/10089047/

I'll have to go find it. It is not in my queue, so it wasn't sent to
the DT list.

Rob

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Rob Herring 
On Sat, Dec 9, 2017 at 3:21 PM, Jacek Anaszewski
 wrote:
> On 12/08/2017 12:04 AM, Dan Murphy wrote:
>> Rob
>>
>>
>> On 12/07/2017 04:49 PM, Rob Herring wrote:
>>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
 This adds the devicetree bindings for the LM3692x
 I2C LED string driver.

 Acked-by: Pavel Machek 
 Signed-off-by: Dan Murphy 
 ---

 v8 - Added address-cells and size-cells as well as child node reg - 
 https://patchwork.kernel.org/patch/10091259/
 v7 - No changes - https://patchwork.kernel.org/patch/10087475/
 v6 - No changes -https://patchwork.kernel.org/patch/10085567/
 v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
 v4 - Fix example node, added trigger entry, removed ambiguous x for 
 compatible and
 added common.txt pointer for label - 
 https://patchwork.kernel.org/patch/10060107
 v3 - No changes
 v2 - No changes - https://patchwork.kernel.org/patch/10056677/

  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
 ++
  1 file changed, 47 insertions(+)
  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

 diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
 b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 new file mode 100644
 index ..84f69342d879
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
 @@ -0,0 +1,47 @@
 +* Texas Instruments - LM3692x Highly Efficient White LED Driver
 +
 +The LM3692x is an ultra-compact, highly efficient,
 +white-LED driver designed for LCD display backlighting.
 +
 +The main difference between the LM36922 and LM36923 is the number of
 +LED strings it supports.  The LM36922 supports two strings while the 
 LM36923
 +supports three strings.
 +
 +Required properties:
 +   - compatible:
 +   "ti,lm36922"
 +   "ti,lm36923"
 +   - reg :  I2C slave address
 +   - #address-cells : 1
 +   - #size-cells : 0
 +
 +Optional properties:
 +   - label : see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Should be a child prop.
>>
>> Thanks I forgot to move this to Optional Child Properties.
>>
>>>
 +   - enable-gpios : gpio pin to enable/disable the device.
 +   - vled-supply : LED supply
 +   - linux,default-trigger : (optional)
 +  see Documentation/devicetree/bindings/leds/common.txt
>>>
>>> Ditto.
>>
>> Ack
>>
>>>
 +
 +Required child properties:
 +   - reg : 0
 +
 +Example:
 +
 +lm3692x@36 {
>>>
>>> leds@36
>>
>> Rob why does this need to be leds?  Is this because it would be a child of 
>> an I2C node?
>>
>> Jacek
>> Would this not cause and issue for your proposal to take the parent node 
>> name as part of the LED label?
>
> Yes, it would.
>
> Rob, does something prevent us from adding a requirement that
> LED controller DT node has to contain the chip name (besides the
> unit address)? Many current LED controller nodes apply this pattern.

Only because we've been lax on the naming. We want to use generic
names because the DT spec says to and it provides a way to match nodes
to validation checks.

I only think we should use the chip name if the device doesn't fit
some standard class.

> Also, I would appreciate if you could express your opinion on
> the patch [0].
>
> [0] https://patchwork.kernel.org/patch/10089047/

I'll have to go find it. It is not in my queue, so it wasn't sent to
the DT list.

Rob

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Rob Herring 
On Thu, Dec 7, 2017 at 5:04 PM, Dan Murphy  wrote:
> Rob
>
>
> On 12/07/2017 04:49 PM, Rob Herring wrote:
>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
>>> This adds the devicetree bindings for the LM3692x
>>> I2C LED string driver.
>>>
>>> Acked-by: Pavel Machek 
>>> Signed-off-by: Dan Murphy 
>>> ---
>>>
>>> v8 - Added address-cells and size-cells as well as child node reg - 
>>> https://patchwork.kernel.org/patch/10091259/
>>> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
>>> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
>>> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
>>> v4 - Fix example node, added trigger entry, removed ambiguous x for 
>>> compatible and
>>> added common.txt pointer for label - 
>>> https://patchwork.kernel.org/patch/10060107
>>> v3 - No changes
>>> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
>>>
>>>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
>>> ++
>>>  1 file changed, 47 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
>>> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> new file mode 100644
>>> index ..84f69342d879
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> @@ -0,0 +1,47 @@
>>> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
>>> +
>>> +The LM3692x is an ultra-compact, highly efficient,
>>> +white-LED driver designed for LCD display backlighting.
>>> +
>>> +The main difference between the LM36922 and LM36923 is the number of
>>> +LED strings it supports.  The LM36922 supports two strings while the 
>>> LM36923
>>> +supports three strings.
>>> +
>>> +Required properties:
>>> +- compatible:
>>> +"ti,lm36922"
>>> +"ti,lm36923"
>>> +- reg :  I2C slave address
>>> +- #address-cells : 1
>>> +- #size-cells : 0
>>> +
>>> +Optional properties:
>>> +- label : see Documentation/devicetree/bindings/leds/common.txt
>>
>> Should be a child prop.
>
> Thanks I forgot to move this to Optional Child Properties.
>
>>
>>> +- enable-gpios : gpio pin to enable/disable the device.
>>> +- vled-supply : LED supply
>>> +- linux,default-trigger : (optional)
>>> +   see Documentation/devicetree/bindings/leds/common.txt
>>
>> Ditto.
>
> Ack
>
>>
>>> +
>>> +Required child properties:
>>> +- reg : 0
>>> +
>>> +Example:
>>> +
>>> +lm3692x@36 {
>>
>> leds@36
>
> Rob why does this need to be leds?  Is this because it would be a child of an 
> I2C node?

Because node names are supposed to follow the generic type of device
they are (e.g. serial, ethernet, bus, etc.), not the model of device.
And yes, there are many examples that don't follow this.

> Jacek
> Would this not cause and issue for your proposal to take the parent node name 
> as part of the LED label?
>
> Dan

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-11 Thread Rob Herring 
On Thu, Dec 7, 2017 at 5:04 PM, Dan Murphy  wrote:
> Rob
>
>
> On 12/07/2017 04:49 PM, Rob Herring wrote:
>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
>>> This adds the devicetree bindings for the LM3692x
>>> I2C LED string driver.
>>>
>>> Acked-by: Pavel Machek 
>>> Signed-off-by: Dan Murphy 
>>> ---
>>>
>>> v8 - Added address-cells and size-cells as well as child node reg - 
>>> https://patchwork.kernel.org/patch/10091259/
>>> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
>>> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
>>> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
>>> v4 - Fix example node, added trigger entry, removed ambiguous x for 
>>> compatible and
>>> added common.txt pointer for label - 
>>> https://patchwork.kernel.org/patch/10060107
>>> v3 - No changes
>>> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
>>>
>>>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
>>> ++
>>>  1 file changed, 47 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
>>> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> new file mode 100644
>>> index ..84f69342d879
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> @@ -0,0 +1,47 @@
>>> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
>>> +
>>> +The LM3692x is an ultra-compact, highly efficient,
>>> +white-LED driver designed for LCD display backlighting.
>>> +
>>> +The main difference between the LM36922 and LM36923 is the number of
>>> +LED strings it supports.  The LM36922 supports two strings while the 
>>> LM36923
>>> +supports three strings.
>>> +
>>> +Required properties:
>>> +- compatible:
>>> +"ti,lm36922"
>>> +"ti,lm36923"
>>> +- reg :  I2C slave address
>>> +- #address-cells : 1
>>> +- #size-cells : 0
>>> +
>>> +Optional properties:
>>> +- label : see Documentation/devicetree/bindings/leds/common.txt
>>
>> Should be a child prop.
>
> Thanks I forgot to move this to Optional Child Properties.
>
>>
>>> +- enable-gpios : gpio pin to enable/disable the device.
>>> +- vled-supply : LED supply
>>> +- linux,default-trigger : (optional)
>>> +   see Documentation/devicetree/bindings/leds/common.txt
>>
>> Ditto.
>
> Ack
>
>>
>>> +
>>> +Required child properties:
>>> +- reg : 0
>>> +
>>> +Example:
>>> +
>>> +lm3692x@36 {
>>
>> leds@36
>
> Rob why does this need to be leds?  Is this because it would be a child of an 
> I2C node?

Because node names are supposed to follow the generic type of device
they are (e.g. serial, ethernet, bus, etc.), not the model of device.
And yes, there are many examples that don't follow this.

> Jacek
> Would this not cause and issue for your proposal to take the parent node name 
> as part of the LED label?
>
> Dan

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-09 Thread Jacek Anaszewski 
On 12/08/2017 12:04 AM, Dan Murphy wrote:
> Rob
> 
> 
> On 12/07/2017 04:49 PM, Rob Herring wrote:
>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
>>> This adds the devicetree bindings for the LM3692x
>>> I2C LED string driver.
>>>
>>> Acked-by: Pavel Machek 
>>> Signed-off-by: Dan Murphy 
>>> ---
>>>
>>> v8 - Added address-cells and size-cells as well as child node reg - 
>>> https://patchwork.kernel.org/patch/10091259/
>>> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
>>> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
>>> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
>>> v4 - Fix example node, added trigger entry, removed ambiguous x for 
>>> compatible and
>>> added common.txt pointer for label - 
>>> https://patchwork.kernel.org/patch/10060107
>>> v3 - No changes
>>> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
>>>
>>>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
>>> ++
>>>  1 file changed, 47 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
>>> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> new file mode 100644
>>> index ..84f69342d879
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> @@ -0,0 +1,47 @@
>>> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
>>> +
>>> +The LM3692x is an ultra-compact, highly efficient,
>>> +white-LED driver designed for LCD display backlighting.
>>> +
>>> +The main difference between the LM36922 and LM36923 is the number of
>>> +LED strings it supports.  The LM36922 supports two strings while the 
>>> LM36923
>>> +supports three strings.
>>> +
>>> +Required properties:
>>> +   - compatible:
>>> +   "ti,lm36922"
>>> +   "ti,lm36923"
>>> +   - reg :  I2C slave address
>>> +   - #address-cells : 1
>>> +   - #size-cells : 0
>>> +
>>> +Optional properties:
>>> +   - label : see Documentation/devicetree/bindings/leds/common.txt
>>
>> Should be a child prop.
> 
> Thanks I forgot to move this to Optional Child Properties.
> 
>>
>>> +   - enable-gpios : gpio pin to enable/disable the device.
>>> +   - vled-supply : LED supply
>>> +   - linux,default-trigger : (optional)
>>> +  see Documentation/devicetree/bindings/leds/common.txt
>>
>> Ditto.
> 
> Ack
> 
>>
>>> +
>>> +Required child properties:
>>> +   - reg : 0
>>> +
>>> +Example:
>>> +
>>> +lm3692x@36 {
>>
>> leds@36
> 
> Rob why does this need to be leds?  Is this because it would be a child of an 
> I2C node?
> 
> Jacek
> Would this not cause and issue for your proposal to take the parent node name 
> as part of the LED label?

Yes, it would.

Rob, does something prevent us from adding a requirement that
LED controller DT node has to contain the chip name (besides the
unit address)? Many current LED controller nodes apply this pattern.

Also, I would appreciate if you could express your opinion on
the patch [0].

[0] https://patchwork.kernel.org/patch/10089047/

-- 
Best regards,
Jacek Anaszewski

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-09 Thread Jacek Anaszewski 
On 12/08/2017 12:04 AM, Dan Murphy wrote:
> Rob
> 
> 
> On 12/07/2017 04:49 PM, Rob Herring wrote:
>> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
>>> This adds the devicetree bindings for the LM3692x
>>> I2C LED string driver.
>>>
>>> Acked-by: Pavel Machek 
>>> Signed-off-by: Dan Murphy 
>>> ---
>>>
>>> v8 - Added address-cells and size-cells as well as child node reg - 
>>> https://patchwork.kernel.org/patch/10091259/
>>> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
>>> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
>>> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
>>> v4 - Fix example node, added trigger entry, removed ambiguous x for 
>>> compatible and
>>> added common.txt pointer for label - 
>>> https://patchwork.kernel.org/patch/10060107
>>> v3 - No changes
>>> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
>>>
>>>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
>>> ++
>>>  1 file changed, 47 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
>>> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> new file mode 100644
>>> index ..84f69342d879
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>> @@ -0,0 +1,47 @@
>>> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
>>> +
>>> +The LM3692x is an ultra-compact, highly efficient,
>>> +white-LED driver designed for LCD display backlighting.
>>> +
>>> +The main difference between the LM36922 and LM36923 is the number of
>>> +LED strings it supports.  The LM36922 supports two strings while the 
>>> LM36923
>>> +supports three strings.
>>> +
>>> +Required properties:
>>> +   - compatible:
>>> +   "ti,lm36922"
>>> +   "ti,lm36923"
>>> +   - reg :  I2C slave address
>>> +   - #address-cells : 1
>>> +   - #size-cells : 0
>>> +
>>> +Optional properties:
>>> +   - label : see Documentation/devicetree/bindings/leds/common.txt
>>
>> Should be a child prop.
> 
> Thanks I forgot to move this to Optional Child Properties.
> 
>>
>>> +   - enable-gpios : gpio pin to enable/disable the device.
>>> +   - vled-supply : LED supply
>>> +   - linux,default-trigger : (optional)
>>> +  see Documentation/devicetree/bindings/leds/common.txt
>>
>> Ditto.
> 
> Ack
> 
>>
>>> +
>>> +Required child properties:
>>> +   - reg : 0
>>> +
>>> +Example:
>>> +
>>> +lm3692x@36 {
>>
>> leds@36
> 
> Rob why does this need to be leds?  Is this because it would be a child of an 
> I2C node?
> 
> Jacek
> Would this not cause and issue for your proposal to take the parent node name 
> as part of the LED label?

Yes, it would.

Rob, does something prevent us from adding a requirement that
LED controller DT node has to contain the chip name (besides the
unit address)? Many current LED controller nodes apply this pattern.

Also, I would appreciate if you could express your opinion on
the patch [0].

[0] https://patchwork.kernel.org/patch/10089047/

-- 
Best regards,
Jacek Anaszewski

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-07 Thread Dan Murphy 
Rob


On 12/07/2017 04:49 PM, Rob Herring wrote:
> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
>> This adds the devicetree bindings for the LM3692x
>> I2C LED string driver.
>>
>> Acked-by: Pavel Machek 
>> Signed-off-by: Dan Murphy 
>> ---
>>
>> v8 - Added address-cells and size-cells as well as child node reg - 
>> https://patchwork.kernel.org/patch/10091259/
>> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
>> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
>> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
>> v4 - Fix example node, added trigger entry, removed ambiguous x for 
>> compatible and
>> added common.txt pointer for label - 
>> https://patchwork.kernel.org/patch/10060107
>> v3 - No changes
>> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
>>
>>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
>> ++
>>  1 file changed, 47 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>
>> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
>> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>> new file mode 100644
>> index ..84f69342d879
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>> @@ -0,0 +1,47 @@
>> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
>> +
>> +The LM3692x is an ultra-compact, highly efficient,
>> +white-LED driver designed for LCD display backlighting.
>> +
>> +The main difference between the LM36922 and LM36923 is the number of
>> +LED strings it supports.  The LM36922 supports two strings while the LM36923
>> +supports three strings.
>> +
>> +Required properties:
>> +- compatible:
>> +"ti,lm36922"
>> +"ti,lm36923"
>> +- reg :  I2C slave address
>> +- #address-cells : 1
>> +- #size-cells : 0
>> +
>> +Optional properties:
>> +- label : see Documentation/devicetree/bindings/leds/common.txt
> 
> Should be a child prop.

Thanks I forgot to move this to Optional Child Properties.

> 
>> +- enable-gpios : gpio pin to enable/disable the device.
>> +- vled-supply : LED supply
>> +- linux,default-trigger : (optional)
>> +   see Documentation/devicetree/bindings/leds/common.txt
> 
> Ditto.

Ack

> 
>> +
>> +Required child properties:
>> +- reg : 0
>> +
>> +Example:
>> +
>> +lm3692x@36 {
> 
> leds@36

Rob why does this need to be leds?  Is this because it would be a child of an 
I2C node?

Jacek
Would this not cause and issue for your proposal to take the parent node name 
as part of the LED label?

Dan

> 
> It looks like you could just add the compatible strings to the existing 
> binding you're modifying?
> 
>> +compatible = "ti,lm3692x";
>> +reg = <0x36>;
>> +#address-cells = <1>;
>> +#size-cells = <0>;
>> +
>> +enable-gpios = < 28 GPIO_ACTIVE_HIGH>;
>> +vled-supply = <>;
>> +
>> +backlight: backlight@0 {
>> +reg = <0>;
>> +label = "backlight_cluster";
>> +linux,default-trigger = "backlight";
>> +};
>> +}
>> +
>> +For more product information please see the link below:
>> +http://www.ti.com/lit/ds/snvsa29/snvsa29.pdf
>> -- 
>> 2.15.0.124.g7668cbc60
>>


-- 
--
Dan Murphy

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-07 Thread Dan Murphy 
Rob


On 12/07/2017 04:49 PM, Rob Herring wrote:
> On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
>> This adds the devicetree bindings for the LM3692x
>> I2C LED string driver.
>>
>> Acked-by: Pavel Machek 
>> Signed-off-by: Dan Murphy 
>> ---
>>
>> v8 - Added address-cells and size-cells as well as child node reg - 
>> https://patchwork.kernel.org/patch/10091259/
>> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
>> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
>> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
>> v4 - Fix example node, added trigger entry, removed ambiguous x for 
>> compatible and
>> added common.txt pointer for label - 
>> https://patchwork.kernel.org/patch/10060107
>> v3 - No changes
>> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
>>
>>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
>> ++
>>  1 file changed, 47 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>>
>> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
>> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>> new file mode 100644
>> index ..84f69342d879
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
>> @@ -0,0 +1,47 @@
>> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
>> +
>> +The LM3692x is an ultra-compact, highly efficient,
>> +white-LED driver designed for LCD display backlighting.
>> +
>> +The main difference between the LM36922 and LM36923 is the number of
>> +LED strings it supports.  The LM36922 supports two strings while the LM36923
>> +supports three strings.
>> +
>> +Required properties:
>> +- compatible:
>> +"ti,lm36922"
>> +"ti,lm36923"
>> +- reg :  I2C slave address
>> +- #address-cells : 1
>> +- #size-cells : 0
>> +
>> +Optional properties:
>> +- label : see Documentation/devicetree/bindings/leds/common.txt
> 
> Should be a child prop.

Thanks I forgot to move this to Optional Child Properties.

> 
>> +- enable-gpios : gpio pin to enable/disable the device.
>> +- vled-supply : LED supply
>> +- linux,default-trigger : (optional)
>> +   see Documentation/devicetree/bindings/leds/common.txt
> 
> Ditto.

Ack

> 
>> +
>> +Required child properties:
>> +- reg : 0
>> +
>> +Example:
>> +
>> +lm3692x@36 {
> 
> leds@36

Rob why does this need to be leds?  Is this because it would be a child of an 
I2C node?

Jacek
Would this not cause and issue for your proposal to take the parent node name 
as part of the LED label?

Dan

> 
> It looks like you could just add the compatible strings to the existing 
> binding you're modifying?
> 
>> +compatible = "ti,lm3692x";
>> +reg = <0x36>;
>> +#address-cells = <1>;
>> +#size-cells = <0>;
>> +
>> +enable-gpios = < 28 GPIO_ACTIVE_HIGH>;
>> +vled-supply = <>;
>> +
>> +backlight: backlight@0 {
>> +reg = <0>;
>> +label = "backlight_cluster";
>> +linux,default-trigger = "backlight";
>> +};
>> +}
>> +
>> +For more product information please see the link below:
>> +http://www.ti.com/lit/ds/snvsa29/snvsa29.pdf
>> -- 
>> 2.15.0.124.g7668cbc60
>>


-- 
--
Dan Murphy

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-07 Thread Rob Herring 
On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
> This adds the devicetree bindings for the LM3692x
> I2C LED string driver.
> 
> Acked-by: Pavel Machek 
> Signed-off-by: Dan Murphy 
> ---
> 
> v8 - Added address-cells and size-cells as well as child node reg - 
> https://patchwork.kernel.org/patch/10091259/
> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
> v4 - Fix example node, added trigger entry, removed ambiguous x for 
> compatible and
> added common.txt pointer for label - 
> https://patchwork.kernel.org/patch/10060107
> v3 - No changes
> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
> 
>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
> ++
>  1 file changed, 47 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
> 
> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
> new file mode 100644
> index ..84f69342d879
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
> @@ -0,0 +1,47 @@
> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
> +
> +The LM3692x is an ultra-compact, highly efficient,
> +white-LED driver designed for LCD display backlighting.
> +
> +The main difference between the LM36922 and LM36923 is the number of
> +LED strings it supports.  The LM36922 supports two strings while the LM36923
> +supports three strings.
> +
> +Required properties:
> + - compatible:
> + "ti,lm36922"
> + "ti,lm36923"
> + - reg :  I2C slave address
> + - #address-cells : 1
> + - #size-cells : 0
> +
> +Optional properties:
> + - label : see Documentation/devicetree/bindings/leds/common.txt

Should be a child prop.

> + - enable-gpios : gpio pin to enable/disable the device.
> + - vled-supply : LED supply
> + - linux,default-trigger : (optional)
> +see Documentation/devicetree/bindings/leds/common.txt

Ditto.

> +
> +Required child properties:
> + - reg : 0
> +
> +Example:
> +
> +lm3692x@36 {

leds@36

It looks like you could just add the compatible strings to the existing 
binding you're modifying?

> + compatible = "ti,lm3692x";
> + reg = <0x36>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + enable-gpios = < 28 GPIO_ACTIVE_HIGH>;
> + vled-supply = <>;
> +
> + backlight: backlight@0 {
> + reg = <0>;
> + label = "backlight_cluster";
> + linux,default-trigger = "backlight";
> + };
> +}
> +
> +For more product information please see the link below:
> +http://www.ti.com/lit/ds/snvsa29/snvsa29.pdf
> -- 
> 2.15.0.124.g7668cbc60
>

Re: [PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-07 Thread Rob Herring 
On Tue, Dec 05, 2017 at 02:46:29PM -0600, Dan Murphy wrote:
> This adds the devicetree bindings for the LM3692x
> I2C LED string driver.
> 
> Acked-by: Pavel Machek 
> Signed-off-by: Dan Murphy 
> ---
> 
> v8 - Added address-cells and size-cells as well as child node reg - 
> https://patchwork.kernel.org/patch/10091259/
> v7 - No changes - https://patchwork.kernel.org/patch/10087475/
> v6 - No changes -https://patchwork.kernel.org/patch/10085567/
> v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
> v4 - Fix example node, added trigger entry, removed ambiguous x for 
> compatible and
> added common.txt pointer for label - 
> https://patchwork.kernel.org/patch/10060107
> v3 - No changes
> v2 - No changes - https://patchwork.kernel.org/patch/10056677/
> 
>  .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 
> ++
>  1 file changed, 47 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt
> 
> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
> b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
> new file mode 100644
> index ..84f69342d879
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
> @@ -0,0 +1,47 @@
> +* Texas Instruments - LM3692x Highly Efficient White LED Driver
> +
> +The LM3692x is an ultra-compact, highly efficient,
> +white-LED driver designed for LCD display backlighting.
> +
> +The main difference between the LM36922 and LM36923 is the number of
> +LED strings it supports.  The LM36922 supports two strings while the LM36923
> +supports three strings.
> +
> +Required properties:
> + - compatible:
> + "ti,lm36922"
> + "ti,lm36923"
> + - reg :  I2C slave address
> + - #address-cells : 1
> + - #size-cells : 0
> +
> +Optional properties:
> + - label : see Documentation/devicetree/bindings/leds/common.txt

Should be a child prop.

> + - enable-gpios : gpio pin to enable/disable the device.
> + - vled-supply : LED supply
> + - linux,default-trigger : (optional)
> +see Documentation/devicetree/bindings/leds/common.txt

Ditto.

> +
> +Required child properties:
> + - reg : 0
> +
> +Example:
> +
> +lm3692x@36 {

leds@36

It looks like you could just add the compatible strings to the existing 
binding you're modifying?

> + compatible = "ti,lm3692x";
> + reg = <0x36>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + enable-gpios = < 28 GPIO_ACTIVE_HIGH>;
> + vled-supply = <>;
> +
> + backlight: backlight@0 {
> + reg = <0>;
> + label = "backlight_cluster";
> + linux,default-trigger = "backlight";
> + };
> +}
> +
> +For more product information please see the link below:
> +http://www.ti.com/lit/ds/snvsa29/snvsa29.pdf
> -- 
> 2.15.0.124.g7668cbc60
>

[PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-05 Thread Dan Murphy 
This adds the devicetree bindings for the LM3692x
I2C LED string driver.

Acked-by: Pavel Machek 
Signed-off-by: Dan Murphy 
---

v8 - Added address-cells and size-cells as well as child node reg - 
https://patchwork.kernel.org/patch/10091259/
v7 - No changes - https://patchwork.kernel.org/patch/10087475/
v6 - No changes -https://patchwork.kernel.org/patch/10085567/
v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
v4 - Fix example node, added trigger entry, removed ambiguous x for compatible 
and
added common.txt pointer for label - https://patchwork.kernel.org/patch/10060107
v3 - No changes
v2 - No changes - https://patchwork.kernel.org/patch/10056677/

 .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 ++
 1 file changed, 47 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
new file mode 100644
index ..84f69342d879
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
@@ -0,0 +1,47 @@
+* Texas Instruments - LM3692x Highly Efficient White LED Driver
+
+The LM3692x is an ultra-compact, highly efficient,
+white-LED driver designed for LCD display backlighting.
+
+The main difference between the LM36922 and LM36923 is the number of
+LED strings it supports.  The LM36922 supports two strings while the LM36923
+supports three strings.
+
+Required properties:
+   - compatible:
+   "ti,lm36922"
+   "ti,lm36923"
+   - reg :  I2C slave address
+   - #address-cells : 1
+   - #size-cells : 0
+
+Optional properties:
+   - label : see Documentation/devicetree/bindings/leds/common.txt
+   - enable-gpios : gpio pin to enable/disable the device.
+   - vled-supply : LED supply
+   - linux,default-trigger : (optional)
+  see Documentation/devicetree/bindings/leds/common.txt
+
+Required child properties:
+   - reg : 0
+
+Example:
+
+lm3692x@36 {
+   compatible = "ti,lm3692x";
+   reg = <0x36>;
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   enable-gpios = < 28 GPIO_ACTIVE_HIGH>;
+   vled-supply = <>;
+
+   backlight: backlight@0 {
+   reg = <0>;
+   label = "backlight_cluster";
+   linux,default-trigger = "backlight";
+   };
+}
+
+For more product information please see the link below:
+http://www.ti.com/lit/ds/snvsa29/snvsa29.pdf
-- 
2.15.0.124.g7668cbc60

[PATCH v8 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

2017-12-05 Thread Dan Murphy 
This adds the devicetree bindings for the LM3692x
I2C LED string driver.

Acked-by: Pavel Machek 
Signed-off-by: Dan Murphy 
---

v8 - Added address-cells and size-cells as well as child node reg - 
https://patchwork.kernel.org/patch/10091259/
v7 - No changes - https://patchwork.kernel.org/patch/10087475/
v6 - No changes -https://patchwork.kernel.org/patch/10085567/
v5 - No Changes - https://patchwork.kernel.org/patch/10081071/
v4 - Fix example node, added trigger entry, removed ambiguous x for compatible 
and
added common.txt pointer for label - https://patchwork.kernel.org/patch/10060107
v3 - No changes
v2 - No changes - https://patchwork.kernel.org/patch/10056677/

 .../devicetree/bindings/leds/leds-lm3692x.txt  | 47 ++
 1 file changed, 47 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt 
b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
new file mode 100644
index ..84f69342d879
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
@@ -0,0 +1,47 @@
+* Texas Instruments - LM3692x Highly Efficient White LED Driver
+
+The LM3692x is an ultra-compact, highly efficient,
+white-LED driver designed for LCD display backlighting.
+
+The main difference between the LM36922 and LM36923 is the number of
+LED strings it supports.  The LM36922 supports two strings while the LM36923
+supports three strings.
+
+Required properties:
+   - compatible:
+   "ti,lm36922"
+   "ti,lm36923"
+   - reg :  I2C slave address
+   - #address-cells : 1
+   - #size-cells : 0
+
+Optional properties:
+   - label : see Documentation/devicetree/bindings/leds/common.txt
+   - enable-gpios : gpio pin to enable/disable the device.
+   - vled-supply : LED supply
+   - linux,default-trigger : (optional)
+  see Documentation/devicetree/bindings/leds/common.txt
+
+Required child properties:
+   - reg : 0
+
+Example:
+
+lm3692x@36 {
+   compatible = "ti,lm3692x";
+   reg = <0x36>;
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   enable-gpios = < 28 GPIO_ACTIVE_HIGH>;
+   vled-supply = <>;
+
+   backlight: backlight@0 {
+   reg = <0>;
+   label = "backlight_cluster";
+   linux,default-trigger = "backlight";
+   };
+}
+
+For more product information please see the link below:
+http://www.ti.com/lit/ds/snvsa29/snvsa29.pdf
-- 
2.15.0.124.g7668cbc60