* [PATCH 0/2] dt-bindings: leds: Convert to DT Schema
@ 2025-10-02 9:20 Harrison Carter
2025-10-02 9:20 ` [PATCH 1/2] dt-bindings: leds: ti,lm3532: " Harrison Carter
2025-10-02 9:20 ` [PATCH 2/2] dt-bindings: leds: ti,lm3601x: " Harrison Carter
0 siblings, 2 replies; 5+ messages in thread
From: Harrison Carter @ 2025-10-02 9:20 UTC (permalink / raw)
To: Lee Jones, Pavel Machek, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Dan Murphy
Cc: linux-leds, devicetree, linux-kernel, Harrison Carter
This patch series aims to update TI's lm3532, and lm3601x to DT
Schema.
Signed-off-by: Harrison Carter <hcarter@thegoodpenguin.co.uk>
---
Harrison Carter (2):
dt-bindings: leds: ti,lm3532: Convert to DT Schema
dt-bindings: leds: ti,lm3601x: Convert to DT Schema
.../devicetree/bindings/leds/leds-lm3532.txt | 105 -------------
.../devicetree/bindings/leds/leds-lm3601x.txt | 51 ------
.../devicetree/bindings/leds/ti,lm3532.yaml | 172 +++++++++++++++++++++
.../devicetree/bindings/leds/ti,lm3601x.yaml | 100 ++++++++++++
4 files changed, 272 insertions(+), 156 deletions(-)
---
base-commit: 76eeb9b8de9880ca38696b2fb56ac45ac0a25c6c
change-id: 20250924-ti-leds-to-dt-ebcdbf5fdf22
Best regards,
--
Harrison Carter <hcarter@thegoodpenguin.co.uk>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/2] dt-bindings: leds: ti,lm3532: Convert to DT Schema
2025-10-02 9:20 [PATCH 0/2] dt-bindings: leds: Convert to DT Schema Harrison Carter
@ 2025-10-02 9:20 ` Harrison Carter
2025-10-08 19:57 ` Rob Herring
2025-10-02 9:20 ` [PATCH 2/2] dt-bindings: leds: ti,lm3601x: " Harrison Carter
1 sibling, 1 reply; 5+ messages in thread
From: Harrison Carter @ 2025-10-02 9:20 UTC (permalink / raw)
To: Lee Jones, Pavel Machek, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Dan Murphy
Cc: linux-leds, devicetree, linux-kernel, Harrison Carter
Converts the ti,lm3532 to DT Schema format
Signed-off-by: Harrison Carter <hcarter@thegoodpenguin.co.uk>
---
.../devicetree/bindings/leds/leds-lm3532.txt | 105 -------------
.../devicetree/bindings/leds/ti,lm3532.yaml | 172 +++++++++++++++++++++
2 files changed, 172 insertions(+), 105 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/leds-lm3532.txt b/Documentation/devicetree/bindings/leds/leds-lm3532.txt
deleted file mode 100644
index 097490a5ff91206ce8c8f0fe0a093a043f089416..0000000000000000000000000000000000000000
--- a/Documentation/devicetree/bindings/leds/leds-lm3532.txt
+++ /dev/null
@@ -1,105 +0,0 @@
-* Texas Instruments - lm3532 White LED driver with ambient light sensing
-capability.
-
-The LM3532 provides the 3 high-voltage, low-side current sinks. The device is
-programmable over an I2C-compatible interface and has independent
-current control for all three channels. The adaptive current regulation
-method allows for different LED currents in each current sink thus allowing
-for a wide variety of backlight and keypad applications.
-
-The main features of the LM3532 include dual ambient light sensor inputs
-each with 32 internal voltage setting resistors, 8-bit logarithmic and linear
-brightness control, dual external PWM brightness control inputs, and up to
-1000:1 dimming ratio with programmable fade in and fade out settings.
-
-Required properties:
- - compatible : "ti,lm3532"
- - reg : I2C slave address
- - #address-cells : 1
- - #size-cells : 0
-
-Optional properties:
- - enable-gpios : gpio pin to enable (active high)/disable the device.
- - ramp-up-us - The Run time ramp rates/step are from one current
- set-point to another after the device has reached its
- initial target set point from turn-on
- - ramp-down-us - The Run time ramp rates/step are from one current
- set-point to another after the device has reached its
- initial target set point from turn-on
- Range for ramp settings: 8us - 65536us
-
-Optional properties if ALS mode is used:
- - ti,als-vmin - Minimum ALS voltage defined in Volts
- - ti,als-vmax - Maximum ALS voltage defined in Volts
- Per the data sheet the max ALS voltage is 2V and the min is 0V
-
- - ti,als1-imp-sel - ALS1 impedance resistor selection in Ohms
- - ti,als2-imp-sel - ALS2 impedance resistor selection in Ohms
- Range for impedance select: 37000 Ohms - 1190 Ohms
- Values above 37kohms will be set to the "High Impedance" setting
-
- - ti,als-avrg-time-us - Determines the length of time the device needs to
- average the two ALS inputs. This is only used if
- the input mode is LM3532_ALS_INPUT_AVRG.
- Range: 17920us - 2293760us
- - ti,als-input-mode - Determines how the device uses the attached ALS
- devices.
- 0x00 - ALS1 and ALS2 input average
- 0x01 - ALS1 Input
- 0x02 - ALS2 Input
- 0x03 - Max of ALS1 and ALS2
-
-Required child properties:
- - reg : Indicates control bank the LED string is controlled by
- - led-sources : see Documentation/devicetree/bindings/leds/common.txt
- - ti,led-mode : Defines if the LED strings are manually controlled or
- if the LED strings are controlled by the ALS.
- 0x00 - LED strings are I2C controlled via full scale
- brightness control register
- 0x01 - LED strings are ALS controlled
-
-Optional LED child properties:
- - label : see Documentation/devicetree/bindings/leds/common.txt
- - linux,default-trigger :
- see Documentation/devicetree/bindings/leds/common.txt
- - led-max-microamp : Defines the full scale current value for each control
- bank. The range is from 5000uA-29800uA in increments
- of 800uA.
-
-Example:
-led-controller@38 {
- compatible = "ti,lm3532";
- #address-cells = <1>;
- #size-cells = <0>;
- reg = <0x38>;
-
- enable-gpios = <&gpio6 12 GPIO_ACTIVE_HIGH>;
- ramp-up-us = <1024>;
- ramp-down-us = <65536>;
-
- ti,als-vmin = <0>;
- ti,als-vmax = <2000>;
- ti,als1-imp-sel = <4110>;
- ti,als2-imp-sel = <2180>;
- ti,als-avrg-time-us = <17920>;
- ti,als-input-mode = <0x00>;
-
- led@0 {
- reg = <0>;
- led-sources = <2>;
- ti,led-mode = <1>;
- led-max-microamp = <21800>;
- label = ":backlight";
- linux,default-trigger = "backlight";
- };
-
- led@1 {
- reg = <1>;
- led-sources = <1>;
- ti,led-mode = <0>;
- label = ":kbd_backlight";
- };
-};
-
-For more product information please see the links below:
-https://www.ti.com/product/LM3532
diff --git a/Documentation/devicetree/bindings/leds/ti,lm3532.yaml b/Documentation/devicetree/bindings/leds/ti,lm3532.yaml
new file mode 100644
index 0000000000000000000000000000000000000000..425277e519bd6de9822822685e9f101d0b3ef40a
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/ti,lm3532.yaml
@@ -0,0 +1,172 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/leds/ti,lm3532.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments - lm3532 White LED driver with
+ ambient light sensing capability.
+
+description: |
+ The LM3532 provides the 3 high-voltage,
+ low-side current sinks. The device is programmable over an
+ I2C-compatible interface and has independent current
+ control for all three channels. The adaptive current
+ regulation method allows for different LED currents in
+ each current sink thus allowing for a wide variety of
+ backlight and keypad applications.
+
+ The main features of the LM3532 include dual ambient light
+ sensor inputs each with 32 internal voltage setting
+ resistors, 8-bit logarithmic and linear brightness
+ control, dual external PWM brightness control inputs,
+ and up to 1000:1 dimming ratio with programmable fade in
+ and fade out settings.
+
+ For more product information please see the link below:
+ https://www.ti.com/product/LM3532
+
+maintainers:
+ - Dan Murphy <dmurphy@ti.com>
+
+properties:
+ compatible:
+ const: ti,lm3532
+
+ reg:
+ maxItems: 1
+
+ '#address-cells':
+ const: 1
+
+ '#size-cells':
+ const: 0
+
+ enable-gpios:
+ description: GPIO pin to enable (active high)/disable
+ the device.
+ maxItems: 1
+
+ ti,als-avrg-time-us:
+ description: Determines the length of time the device
+ needs to average the two ALS inputs. This is only
+ used if the input mode is LM3532_ALS_INPUT_AVRG.
+ minimum: 17920
+ maximum: 2293760
+
+ ti,als-input-mode:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: Determines how the device uses the attached
+ ALS devices.
+ 0x00 - ALS1 and ALS2 input average
+ 0x01 - ALS1 Input
+ 0x02 - ALS2 Input
+ 0x03 - Max of ALS1 and ALS2
+ minimum: 0
+ maximum: 3
+
+patternProperties:
+ "^ramp-(up|down)-us$":
+ description: The Run time ramp rates/step are from one
+ current set-point to another after the device has
+ reached its initial target set point from turn-on
+ minimum: 8
+ maximum: 65536
+
+ "^ti,als-v(min|max)$":
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: Min/max ALS voltage in Volts.
+ minimum: 0
+ maximum: 2
+
+ "^ti,als(1|2)-imp-sel$":
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: ALS impedance resistor selection in Ohms.
+ Values above 37kohms will be set to the "High
+ Impedance" setting
+ minimum: 1190
+ maximum: 37000
+
+ "^led@[0,1]?([0-9]|[a-z])$":
+ $ref: common.yaml#
+ type: object
+ properties:
+ reg:
+ description: Indicates control bank the LED string
+ is controlled by
+
+ ti,led-mode:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description: Defines if the LED strings are manually
+ controlled or if the LED strings are controlled
+ by the ALS.
+ 0x00 - LED strings are I2C controlled via full
+ scale brightness control register
+ 0x01 - LED strings are ALS controlled
+ minimum: 0
+ maximum: 1
+
+ led-max-microamp:
+ minimum: 5000
+ maximum: 29800
+ multipleOf: 800
+
+ required:
+ - reg
+ - led-sources
+ - ti,led-mode
+
+ unevaluatedProperties: true
+
+required:
+ - compatible
+ - reg
+ - '#address-cells'
+ - '#size-cells'
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/leds/common.h>
+
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ led-controller@38 {
+ compatible = "ti,lm3532";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ reg = <0x38>;
+
+ enable-gpios = <&gpio6 12 GPIO_ACTIVE_HIGH>;
+ ramp-up-us = <1024>;
+ ramp-down-us = <65536>;
+
+ ti,als-vmin = <0>;
+ ti,als-vmax = <2>;
+ ti,als1-imp-sel = <4110>;
+ ti,als2-imp-sel = <2180>;
+ ti,als-avrg-time-us = <17920>;
+ ti,als-input-mode = <0x00>;
+
+ led@0 {
+ reg = <0>;
+ led-sources = <2>;
+ ti,led-mode = <1>;
+ led-max-microamp = <21600>;
+ label = ":backlight";
+ linux,default-trigger = "backlight";
+ };
+
+ led@1 {
+ reg = <1>;
+ led-sources = <1>;
+ ti,led-mode = <0>;
+ label = ":kbd_backlight";
+ };
+ };
+ };
+...
--
2.51.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] dt-bindings: leds: ti,lm3601x: Convert to DT Schema
2025-10-02 9:20 [PATCH 0/2] dt-bindings: leds: Convert to DT Schema Harrison Carter
2025-10-02 9:20 ` [PATCH 1/2] dt-bindings: leds: ti,lm3532: " Harrison Carter
@ 2025-10-02 9:20 ` Harrison Carter
2025-10-08 19:58 ` Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Harrison Carter @ 2025-10-02 9:20 UTC (permalink / raw)
To: Lee Jones, Pavel Machek, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Dan Murphy
Cc: linux-leds, devicetree, linux-kernel, Harrison Carter
Converts the ti,lm36010 and ti,lm36011 txt to dt schema
Signed-off-by: Harrison Carter <hcarter@thegoodpenguin.co.uk>
---
.../devicetree/bindings/leds/leds-lm3601x.txt | 51 -----------
.../devicetree/bindings/leds/ti,lm3601x.yaml | 100 +++++++++++++++++++++
2 files changed, 100 insertions(+), 51 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/leds-lm3601x.txt b/Documentation/devicetree/bindings/leds/leds-lm3601x.txt
deleted file mode 100644
index 17e940025dc26213314f5cfd54aa8e5bb09f86b7..0000000000000000000000000000000000000000
--- a/Documentation/devicetree/bindings/leds/leds-lm3601x.txt
+++ /dev/null
@@ -1,51 +0,0 @@
-* Texas Instruments - lm3601x Single-LED Flash Driver
-
-The LM3601X are ultra-small LED flash drivers that
-provide a high level of adjustability.
-
-Required properties:
- - compatible : Can be one of the following
- "ti,lm36010"
- "ti,lm36011"
- - reg : I2C slave address
- - #address-cells : 1
- - #size-cells : 0
-
-Required child properties:
- - reg : 0 - Indicates a IR mode
- 1 - Indicates a Torch (white LED) mode
-
-Required properties for flash LED child nodes:
- See Documentation/devicetree/bindings/leds/common.txt
- - flash-max-microamp : Range from 11mA - 1.5A
- - flash-max-timeout-us : Range from 40ms - 1600ms
- - led-max-microamp : Range from 2.4mA - 376mA
-
-Optional child properties:
- - function : see Documentation/devicetree/bindings/leds/common.txt
- - color : see Documentation/devicetree/bindings/leds/common.txt
- - label : see Documentation/devicetree/bindings/leds/common.txt (deprecated)
-
-Example:
-
-#include <dt-bindings/leds/common.h>
-
-led-controller@64 {
- compatible = "ti,lm36010";
- #address-cells = <1>;
- #size-cells = <0>;
- reg = <0x64>;
-
- led@0 {
- reg = <1>;
- function = LED_FUNCTION_TORCH;
- color = <LED_COLOR_ID_WHITE>;
- led-max-microamp = <376000>;
- flash-max-microamp = <1500000>;
- flash-max-timeout-us = <1600000>;
- };
-}
-
-For more product information please see the links below:
-https://www.ti.com/product/LM36010
-https://www.ti.com/product/LM36011
diff --git a/Documentation/devicetree/bindings/leds/ti,lm3601x.yaml b/Documentation/devicetree/bindings/leds/ti,lm3601x.yaml
new file mode 100644
index 0000000000000000000000000000000000000000..d7d8ee44d6fe37d13ee84888c5811df3e15a5d02
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/ti,lm3601x.yaml
@@ -0,0 +1,100 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/leds/ti,lm3601x.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments - lm3601x Single-LED Flash Driver
+
+description: |
+ The LM3601X are ultra-small LED flash drivers
+ that provide a high level of adjustability.
+
+ For more product information please see the links below:
+ https://www.ti.com/product/LM36010
+ https://www.ti.com/product/LM36011
+
+maintainers:
+ - Dan Murphy <dmurphy@ti.com>
+
+properties:
+ compatible:
+ enum:
+ - "ti,lm36010"
+ - "ti,lm36011"
+
+ reg:
+ maxItems: 1
+
+ '#address-cells':
+ const: 1
+
+ '#size-cells':
+ const: 0
+
+patternProperties:
+ "^led@[0,1]?([0-9]|[a-z])$":
+ type: object
+ $ref: common.yaml#
+ properties:
+ reg:
+ description: |
+ 0 - Indicates IR mode
+ 1 - Indicates Torch (white LED) mode
+ minimum: 0
+ maximum: 1
+
+ flash-max-microamp:
+ minimum: 11
+ maximum: 1500000
+
+ flash-max-timeout-us:
+ minimum: 40
+ maximum: 1600000
+
+ led-max-microamp:
+ minimum: 24
+ maximum: 376000
+
+ required:
+ - reg
+ - flash-max-microamp
+ - flash-max-timeout-us
+ - led-max-microamp
+
+ unevaluatedProperties: true
+
+required:
+ - compatible
+ - reg
+ - '#address-cells'
+ - '#size-cells'
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/leds/common.h>
+
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ led-controller@64 {
+ compatible = "ti,lm36010";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ reg = <0x64>;
+
+ led@1 {
+ reg = <1>;
+ function = LED_FUNCTION_TORCH;
+ color = <LED_COLOR_ID_WHITE>;
+ led-max-microamp = <376000>;
+ flash-max-microamp = <1500000>;
+ flash-max-timeout-us = <1600000>;
+ };
+ };
+ };
+...
+
--
2.51.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: leds: ti,lm3532: Convert to DT Schema
2025-10-02 9:20 ` [PATCH 1/2] dt-bindings: leds: ti,lm3532: " Harrison Carter
@ 2025-10-08 19:57 ` Rob Herring
0 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2025-10-08 19:57 UTC (permalink / raw)
To: Harrison Carter
Cc: Lee Jones, Pavel Machek, Krzysztof Kozlowski, Conor Dooley,
Dan Murphy, linux-leds, devicetree, linux-kernel
On Thu, Oct 02, 2025 at 10:20:15AM +0100, Harrison Carter wrote:
> Converts the ti,lm3532 to DT Schema format
>
> Signed-off-by: Harrison Carter <hcarter@thegoodpenguin.co.uk>
> ---
> .../devicetree/bindings/leds/leds-lm3532.txt | 105 -------------
> .../devicetree/bindings/leds/ti,lm3532.yaml | 172 +++++++++++++++++++++
> 2 files changed, 172 insertions(+), 105 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3532.txt b/Documentation/devicetree/bindings/leds/leds-lm3532.txt
> deleted file mode 100644
> index 097490a5ff91206ce8c8f0fe0a093a043f089416..0000000000000000000000000000000000000000
> --- a/Documentation/devicetree/bindings/leds/leds-lm3532.txt
> +++ /dev/null
> @@ -1,105 +0,0 @@
> -* Texas Instruments - lm3532 White LED driver with ambient light sensing
> -capability.
> -
> -The LM3532 provides the 3 high-voltage, low-side current sinks. The device is
> -programmable over an I2C-compatible interface and has independent
> -current control for all three channels. The adaptive current regulation
> -method allows for different LED currents in each current sink thus allowing
> -for a wide variety of backlight and keypad applications.
> -
> -The main features of the LM3532 include dual ambient light sensor inputs
> -each with 32 internal voltage setting resistors, 8-bit logarithmic and linear
> -brightness control, dual external PWM brightness control inputs, and up to
> -1000:1 dimming ratio with programmable fade in and fade out settings.
> -
> -Required properties:
> - - compatible : "ti,lm3532"
> - - reg : I2C slave address
> - - #address-cells : 1
> - - #size-cells : 0
> -
> -Optional properties:
> - - enable-gpios : gpio pin to enable (active high)/disable the device.
> - - ramp-up-us - The Run time ramp rates/step are from one current
> - set-point to another after the device has reached its
> - initial target set point from turn-on
> - - ramp-down-us - The Run time ramp rates/step are from one current
> - set-point to another after the device has reached its
> - initial target set point from turn-on
> - Range for ramp settings: 8us - 65536us
> -
> -Optional properties if ALS mode is used:
> - - ti,als-vmin - Minimum ALS voltage defined in Volts
> - - ti,als-vmax - Maximum ALS voltage defined in Volts
> - Per the data sheet the max ALS voltage is 2V and the min is 0V
> -
> - - ti,als1-imp-sel - ALS1 impedance resistor selection in Ohms
> - - ti,als2-imp-sel - ALS2 impedance resistor selection in Ohms
> - Range for impedance select: 37000 Ohms - 1190 Ohms
> - Values above 37kohms will be set to the "High Impedance" setting
> -
> - - ti,als-avrg-time-us - Determines the length of time the device needs to
> - average the two ALS inputs. This is only used if
> - the input mode is LM3532_ALS_INPUT_AVRG.
> - Range: 17920us - 2293760us
> - - ti,als-input-mode - Determines how the device uses the attached ALS
> - devices.
> - 0x00 - ALS1 and ALS2 input average
> - 0x01 - ALS1 Input
> - 0x02 - ALS2 Input
> - 0x03 - Max of ALS1 and ALS2
> -
> -Required child properties:
> - - reg : Indicates control bank the LED string is controlled by
> - - led-sources : see Documentation/devicetree/bindings/leds/common.txt
> - - ti,led-mode : Defines if the LED strings are manually controlled or
> - if the LED strings are controlled by the ALS.
> - 0x00 - LED strings are I2C controlled via full scale
> - brightness control register
> - 0x01 - LED strings are ALS controlled
> -
> -Optional LED child properties:
> - - label : see Documentation/devicetree/bindings/leds/common.txt
> - - linux,default-trigger :
> - see Documentation/devicetree/bindings/leds/common.txt
> - - led-max-microamp : Defines the full scale current value for each control
> - bank. The range is from 5000uA-29800uA in increments
> - of 800uA.
> -
> -Example:
> -led-controller@38 {
> - compatible = "ti,lm3532";
> - #address-cells = <1>;
> - #size-cells = <0>;
> - reg = <0x38>;
> -
> - enable-gpios = <&gpio6 12 GPIO_ACTIVE_HIGH>;
> - ramp-up-us = <1024>;
> - ramp-down-us = <65536>;
> -
> - ti,als-vmin = <0>;
> - ti,als-vmax = <2000>;
> - ti,als1-imp-sel = <4110>;
> - ti,als2-imp-sel = <2180>;
> - ti,als-avrg-time-us = <17920>;
> - ti,als-input-mode = <0x00>;
> -
> - led@0 {
> - reg = <0>;
> - led-sources = <2>;
> - ti,led-mode = <1>;
> - led-max-microamp = <21800>;
> - label = ":backlight";
> - linux,default-trigger = "backlight";
> - };
> -
> - led@1 {
> - reg = <1>;
> - led-sources = <1>;
> - ti,led-mode = <0>;
> - label = ":kbd_backlight";
> - };
> -};
> -
> -For more product information please see the links below:
> -https://www.ti.com/product/LM3532
> diff --git a/Documentation/devicetree/bindings/leds/ti,lm3532.yaml b/Documentation/devicetree/bindings/leds/ti,lm3532.yaml
> new file mode 100644
> index 0000000000000000000000000000000000000000..425277e519bd6de9822822685e9f101d0b3ef40a
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/leds/ti,lm3532.yaml
> @@ -0,0 +1,172 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/leds/ti,lm3532.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments - lm3532 White LED driver with
> + ambient light sensing capability.
Drop period.
> +
> +description: |
> + The LM3532 provides the 3 high-voltage,
> + low-side current sinks. The device is programmable over an
> + I2C-compatible interface and has independent current
> + control for all three channels. The adaptive current
> + regulation method allows for different LED currents in
> + each current sink thus allowing for a wide variety of
> + backlight and keypad applications.
> +
> + The main features of the LM3532 include dual ambient light
> + sensor inputs each with 32 internal voltage setting
> + resistors, 8-bit logarithmic and linear brightness
> + control, dual external PWM brightness control inputs,
> + and up to 1000:1 dimming ratio with programmable fade in
> + and fade out settings.
Wrap lines at 80.
> +
> + For more product information please see the link below:
> + https://www.ti.com/product/LM3532
> +
> +maintainers:
> + - Dan Murphy <dmurphy@ti.com>
> +
> +properties:
> + compatible:
> + const: ti,lm3532
> +
> + reg:
> + maxItems: 1
> +
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 0
> +
> + enable-gpios:
> + description: GPIO pin to enable (active high)/disable
Start paragraph on next line unless it fits on 1 line (and <80 chars).
> + the device.
> + maxItems: 1
> +
> + ti,als-avrg-time-us:
> + description: Determines the length of time the device
> + needs to average the two ALS inputs. This is only
> + used if the input mode is LM3532_ALS_INPUT_AVRG.
> + minimum: 17920
> + maximum: 2293760
> +
> + ti,als-input-mode:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: Determines how the device uses the attached
> + ALS devices.
> + 0x00 - ALS1 and ALS2 input average
> + 0x01 - ALS1 Input
> + 0x02 - ALS2 Input
> + 0x03 - Max of ALS1 and ALS2
> + minimum: 0
> + maximum: 3
> +
> +patternProperties:
> + "^ramp-(up|down)-us$":
> + description: The Run time ramp rates/step are from one
> + current set-point to another after the device has
> + reached its initial target set point from turn-on
> + minimum: 8
> + maximum: 65536
> +
> + "^ti,als-v(min|max)$":
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: Min/max ALS voltage in Volts.
> + minimum: 0
> + maximum: 2
> +
> + "^ti,als(1|2)-imp-sel$":
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: ALS impedance resistor selection in Ohms.
> + Values above 37kohms will be set to the "High
> + Impedance" setting
> + minimum: 1190
> + maximum: 37000
> +
> + "^led@[0,1]?([0-9]|[a-z])$":
Not the right unit-address. Should be hex values in the range of
whatever 'reg' is.
> + $ref: common.yaml#
> + type: object
> + properties:
> + reg:
> + description: Indicates control bank the LED string
> + is controlled by
Constraints.
> +
> + ti,led-mode:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: Defines if the LED strings are manually
> + controlled or if the LED strings are controlled
> + by the ALS.
> + 0x00 - LED strings are I2C controlled via full
> + scale brightness control register
> + 0x01 - LED strings are ALS controlled
You need '>' or '|' if there is formatting. Indent lists by 1 more tab.
> + minimum: 0
> + maximum: 1
> +
> + led-max-microamp:
> + minimum: 5000
> + maximum: 29800
> + multipleOf: 800
> +
> + required:
> + - reg
> + - led-sources
> + - ti,led-mode
> +
> + unevaluatedProperties: true
> +
> +required:
> + - compatible
> + - reg
> + - '#address-cells'
> + - '#size-cells'
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/leds/common.h>
> +
> + i2c {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + led-controller@38 {
> + compatible = "ti,lm3532";
> + #address-cells = <1>;
> + #size-cells = <0>;
> + reg = <0x38>;
> +
> + enable-gpios = <&gpio6 12 GPIO_ACTIVE_HIGH>;
> + ramp-up-us = <1024>;
> + ramp-down-us = <65536>;
> +
> + ti,als-vmin = <0>;
> + ti,als-vmax = <2>;
> + ti,als1-imp-sel = <4110>;
> + ti,als2-imp-sel = <2180>;
> + ti,als-avrg-time-us = <17920>;
> + ti,als-input-mode = <0x00>;
> +
> + led@0 {
> + reg = <0>;
> + led-sources = <2>;
> + ti,led-mode = <1>;
> + led-max-microamp = <21600>;
> + label = ":backlight";
> + linux,default-trigger = "backlight";
> + };
> +
> + led@1 {
> + reg = <1>;
> + led-sources = <1>;
> + ti,led-mode = <0>;
> + label = ":kbd_backlight";
> + };
> + };
> + };
> +...
>
> --
> 2.51.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] dt-bindings: leds: ti,lm3601x: Convert to DT Schema
2025-10-02 9:20 ` [PATCH 2/2] dt-bindings: leds: ti,lm3601x: " Harrison Carter
@ 2025-10-08 19:58 ` Rob Herring
0 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2025-10-08 19:58 UTC (permalink / raw)
To: Harrison Carter
Cc: Lee Jones, Pavel Machek, Krzysztof Kozlowski, Conor Dooley,
Dan Murphy, linux-leds, devicetree, linux-kernel
On Thu, Oct 02, 2025 at 10:20:16AM +0100, Harrison Carter wrote:
> Converts the ti,lm36010 and ti,lm36011 txt to dt schema
>
> Signed-off-by: Harrison Carter <hcarter@thegoodpenguin.co.uk>
> ---
> .../devicetree/bindings/leds/leds-lm3601x.txt | 51 -----------
> .../devicetree/bindings/leds/ti,lm3601x.yaml | 100 +++++++++++++++++++++
> 2 files changed, 100 insertions(+), 51 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/leds/leds-lm3601x.txt b/Documentation/devicetree/bindings/leds/leds-lm3601x.txt
> deleted file mode 100644
> index 17e940025dc26213314f5cfd54aa8e5bb09f86b7..0000000000000000000000000000000000000000
> --- a/Documentation/devicetree/bindings/leds/leds-lm3601x.txt
> +++ /dev/null
> @@ -1,51 +0,0 @@
> -* Texas Instruments - lm3601x Single-LED Flash Driver
> -
> -The LM3601X are ultra-small LED flash drivers that
> -provide a high level of adjustability.
> -
> -Required properties:
> - - compatible : Can be one of the following
> - "ti,lm36010"
> - "ti,lm36011"
> - - reg : I2C slave address
> - - #address-cells : 1
> - - #size-cells : 0
> -
> -Required child properties:
> - - reg : 0 - Indicates a IR mode
> - 1 - Indicates a Torch (white LED) mode
> -
> -Required properties for flash LED child nodes:
> - See Documentation/devicetree/bindings/leds/common.txt
> - - flash-max-microamp : Range from 11mA - 1.5A
> - - flash-max-timeout-us : Range from 40ms - 1600ms
> - - led-max-microamp : Range from 2.4mA - 376mA
> -
> -Optional child properties:
> - - function : see Documentation/devicetree/bindings/leds/common.txt
> - - color : see Documentation/devicetree/bindings/leds/common.txt
> - - label : see Documentation/devicetree/bindings/leds/common.txt (deprecated)
> -
> -Example:
> -
> -#include <dt-bindings/leds/common.h>
> -
> -led-controller@64 {
> - compatible = "ti,lm36010";
> - #address-cells = <1>;
> - #size-cells = <0>;
> - reg = <0x64>;
> -
> - led@0 {
> - reg = <1>;
> - function = LED_FUNCTION_TORCH;
> - color = <LED_COLOR_ID_WHITE>;
> - led-max-microamp = <376000>;
> - flash-max-microamp = <1500000>;
> - flash-max-timeout-us = <1600000>;
> - };
> -}
> -
> -For more product information please see the links below:
> -https://www.ti.com/product/LM36010
> -https://www.ti.com/product/LM36011
> diff --git a/Documentation/devicetree/bindings/leds/ti,lm3601x.yaml b/Documentation/devicetree/bindings/leds/ti,lm3601x.yaml
> new file mode 100644
> index 0000000000000000000000000000000000000000..d7d8ee44d6fe37d13ee84888c5811df3e15a5d02
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/leds/ti,lm3601x.yaml
> @@ -0,0 +1,100 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/leds/ti,lm3601x.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments - lm3601x Single-LED Flash Driver
> +
> +description: |
> + The LM3601X are ultra-small LED flash drivers
> + that provide a high level of adjustability.
> +
> + For more product information please see the links below:
> + https://www.ti.com/product/LM36010
> + https://www.ti.com/product/LM36011
> +
> +maintainers:
> + - Dan Murphy <dmurphy@ti.com>
> +
> +properties:
> + compatible:
> + enum:
> + - "ti,lm36010"
> + - "ti,lm36011"
Don't need quotes.
> +
> + reg:
> + maxItems: 1
> +
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 0
> +
> +patternProperties:
> + "^led@[0,1]?([0-9]|[a-z])$":
'^led@[01]$'
> + type: object
> + $ref: common.yaml#
> + properties:
> + reg:
> + description: |
> + 0 - Indicates IR mode
> + 1 - Indicates Torch (white LED) mode
> + minimum: 0
> + maximum: 1
> +
> + flash-max-microamp:
> + minimum: 11
> + maximum: 1500000
> +
> + flash-max-timeout-us:
> + minimum: 40
> + maximum: 1600000
> +
> + led-max-microamp:
> + minimum: 24
> + maximum: 376000
> +
> + required:
> + - reg
> + - flash-max-microamp
> + - flash-max-timeout-us
> + - led-max-microamp
> +
> + unevaluatedProperties: true
> +
> +required:
> + - compatible
> + - reg
> + - '#address-cells'
> + - '#size-cells'
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/leds/common.h>
> +
> + i2c {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + led-controller@64 {
> + compatible = "ti,lm36010";
> + #address-cells = <1>;
> + #size-cells = <0>;
> + reg = <0x64>;
> +
> + led@1 {
> + reg = <1>;
> + function = LED_FUNCTION_TORCH;
> + color = <LED_COLOR_ID_WHITE>;
> + led-max-microamp = <376000>;
> + flash-max-microamp = <1500000>;
> + flash-max-timeout-us = <1600000>;
> + };
> + };
> + };
> +...
> +
>
> --
> 2.51.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2025-10-08 19:58 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-02 9:20 [PATCH 0/2] dt-bindings: leds: Convert to DT Schema Harrison Carter
2025-10-02 9:20 ` [PATCH 1/2] dt-bindings: leds: ti,lm3532: " Harrison Carter
2025-10-08 19:57 ` Rob Herring
2025-10-02 9:20 ` [PATCH 2/2] dt-bindings: leds: ti,lm3601x: " Harrison Carter
2025-10-08 19:58 ` Rob Herring
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).