[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251008195730.GA3771722-robh@kernel.org>
Date: Wed, 8 Oct 2025 14:57:30 -0500
From: Rob Herring <robh@...nel.org>
To: Harrison Carter <hcarter@...goodpenguin.co.uk>
Cc: Lee Jones <lee@...nel.org>, Pavel Machek <pavel@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Dan Murphy <dmurphy@...com>,
linux-leds@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: leds: ti,lm3532: Convert to DT Schema
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@...goodpenguin.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@...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
>
Powered by blists - more mailing lists