[v9,1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver

Message ID 20171211182659.21533-1-dmurphy@ti.com
State Accepted
Commit 1d9aa24b65c0084586c0f2fbabb4874754c7fbe5
Headers show
Series
  • [v9,1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver
Related show

Commit Message

Dan Murphy Dec. 11, 2017, 6:26 p.m.
This adds the devicetree bindings for the LM3692x
I2C LED string driver.

Acked-by: Pavel Machek <pavel@ucw.cz>

Signed-off-by: Dan Murphy <dmurphy@ti.com>

---

v9 - Moved 2 nodes to Optional Child and renamed node names to device type
https://patchwork.kernel.org/patch/10093757/

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      | 49 ++++++++++++++++++++++
 1 file changed, 49 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3692x.txt

-- 
2.15.0.124.g7668cbc60

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Jacek Anaszewski Dec. 11, 2017, 9:41 p.m. | #1
Dan,

On 12/11/2017 07:26 PM, Dan Murphy wrote:
> This adds the devicetree bindings for the LM3692x

> I2C LED string driver.

> 

> Acked-by: Pavel Machek <pavel@ucw.cz>

> Signed-off-by: Dan Murphy <dmurphy@ti.com>

> ---

> 

> v9 - Moved 2 nodes to Optional Child and renamed node names to device type

> https://patchwork.kernel.org/patch/10093757/

> 

> 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      | 49 ++++++++++++++++++++++

>  1 file changed, 49 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 000000000000..a93e19edfb42

> --- /dev/null

> +++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt

> @@ -0,0 +1,49 @@

> +* 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:

> +	- enable-gpios : gpio pin to enable/disable the device.

> +	- vled-supply : LED supply

> +

> +Required child properties:

> +	- reg : 0

> +

> +Optional child properties:

> +	- label : see Documentation/devicetree/bindings/leds/common.txt

> +	- linux,default-trigger :

> +	   see Documentation/devicetree/bindings/leds/common.txt

> +

> +Example:

> +

> +led-controller@36 {

> +	compatible = "ti,lm3692x";

> +	reg = <0x36>;

> +	#address-cells = <1>;

> +	#size-cells = <0>;

> +

> +	enable-gpios = <&gpio1 28 GPIO_ACTIVE_HIGH>;

> +	vled-supply = <&vbatt>;

> +

> +	led@0 {

> +		reg = <0>;

> +		label = "backlight_cluster";


We need to consequently adhere to LED class device naming
convention, so:

label = "white:backlight_cluster";

> +		linux,default-trigger = "backlight";

> +	};

> +}

> +

> +For more product information please see the link below:

> +http://www.ti.com/lit/ds/snvsa29/snvsa29.pdf

> 


-- 
Best regards,
Jacek Anaszewski
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
new file mode 100644
index 000000000000..a93e19edfb42
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
@@ -0,0 +1,49 @@ 
+* 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:
+	- enable-gpios : gpio pin to enable/disable the device.
+	- vled-supply : LED supply
+
+Required child properties:
+	- reg : 0
+
+Optional child properties:
+	- label : see Documentation/devicetree/bindings/leds/common.txt
+	- linux,default-trigger :
+	   see Documentation/devicetree/bindings/leds/common.txt
+
+Example:
+
+led-controller@36 {
+	compatible = "ti,lm3692x";
+	reg = <0x36>;
+	#address-cells = <1>;
+	#size-cells = <0>;
+
+	enable-gpios = <&gpio1 28 GPIO_ACTIVE_HIGH>;
+	vled-supply = <&vbatt>;
+
+	led@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