| Message ID | 20250218132702.114669-2-clamor95@gmail.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | mfd: lm3533: convert to use OF | expand |
On Tue, Feb 18, 2025 at 03:26:59PM +0200, Svyatoslav Ryhel wrote: > Add bindings for the LM3533 - a complete power source for > backlight, keypad, and indicator LEDs in smartphone handsets. > The high-voltage inductive boost converter provides the > power for two series LED strings display backlight and keypad > functions. > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com> > --- > .../devicetree/bindings/mfd/ti,lm3533.yaml | 231 ++++++++++++++++++ > 1 file changed, 231 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > > diff --git a/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > new file mode 100644 > index 000000000000..83542f0c7bf7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > @@ -0,0 +1,231 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/mfd/ti,lm3533.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: TI LM3533 Complete Lighting Power Solution > + > +description: | Use '>' rather than '|' if only formatting is paragraphs. > + The LM3533 is a complete power source for backlight, > + keypad, and indicator LEDs in smartphone handsets. The > + high-voltage inductive boost converter provides the > + power for two series LED strings display backlight and > + keypad functions. Wrap lines at 80 blank line here > + https://www.ti.com/product/LM3533 > + > +maintainers: > + - Svyatoslav Ryhel <clamor95@gmail.com> > + > +properties: > + compatible: > + const: ti,lm3533 > + > + reg: > + maxItems: 1 > + > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > + enable-gpios: > + description: GPIO to use to enable/disable the backlight (HWEN pin). > + maxItems: 1 > + > + ti,boost-ovp-microvolt: > + description: > + Boost OVP select (16V, 24V, 32V, 40V) > + enum: [ 16000000, 24000000, 32000000, 40000000 ] > + default: 16000000 > + > + ti,boost-freq-hz: > + description: > + Boost frequency select (500KHz or 1MHz) > + enum: [ 500000, 1000000 ] > + default: 500000 > + > + light-sensor@0: > + type: object > + description: > + Properties for an illumination sensor. > + > + properties: > + compatible: > + const: ti,lm3533-als > + > + reg: > + const: 0 > + > + ti,resistor-value-ohm: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: | Don't need '|'. Elsewhere too. > + Internal configuration resister value when ALS is in Analog Sensor > + mode and PWM mode is disabled. > + minimum: 1575 > + maximum: 200000 > + > + ti,pwm-mode: > + type: boolean > + description: | > + Switch for mode in which ALS is running. If this propertly is > + set then ALS is running in PWM mode, internal resistor value is > + set to high-impedance (0) and resistor-value-ohm propertly is > + ignored. > + > + required: > + - compatible > + - reg > + > + additionalProperties: false Move this above 'properties'. > + > +required: > + - compatible > + - reg > + - '#address-cells' > + - '#size-cells' > + - enable-gpios > + > +patternProperties: > + "^backlight@[01]$": > + type: object > + description: > + Properties for a backlight device. > + > + $ref: /schemas/leds/backlight/common.yaml# > + > + properties: > + compatible: > + const: ti,lm3533-backlight > + > + reg: > + description: | > + The control bank that is used to program the two current sinks. The > + LM3533 has two control banks (A and B) and are represented as 0 or 1 > + in this property. The two current sinks can be controlled > + independently with both banks, or bank A can be configured to control > + both sinks with the led-sources property. > + minimum: 0 > + maximum: 1 > + > + default-brightness: true > + > + ti,max-current-microamp: > + description: > + Maximum current in µA with a 800 µA step. > + enum: [ 5000, 5800, 6600, 7400, 8200, 9000, 9800, > + 10600, 11400, 12200, 13000, 13800, 14600, > + 15400, 16200, 17000, 17800, 18600, 19400, > + 20200, 21000, 21800, 22600, 23400, 24200, > + 25000, 25800, 26600, 27400, 28200, 29000, > + 29800 ] > + default: 5000 > + > + ti,pwm-config-mask: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: | > + Control Bank PWM Configuration Register mask that allows to configure > + PWM input in Zones 0-4 > + BIT(0) - PWM Input is enabled > + BIT(1) - PWM Input is enabled in Zone 0 > + BIT(2) - PWM Input is enabled in Zone 1 > + BIT(3) - PWM Input is enabled in Zone 2 > + BIT(4) - PWM Input is enabled in Zone 3 > + BIT(5) - PWM Input is enabled in Zone 4 > + > + ti,linear-mapping-mode: > + description: | > + Enable linear mapping mode. If disabled, then it will use exponential > + mapping mode in which the ramp up/down appears to have a more uniform > + transition to the human eye. > + type: boolean > + > + ti,hardware-controlled: > + description: | > + Each backlight has its own voltage Control Bank (A and B) and there are > + two HVLED sinks which by default are linked to respective Bank. Setting > + this property will link both sinks to a Control Bank of backlight where > + property is defined. > + type: boolean > + > + required: > + - compatible > + - reg > + > + additionalProperties: false > + > + "^led@[0-3]$": > + type: object > + description: > + Properties for a led device. > + > + $ref: /schemas/leds/common.yaml# > + > + properties: > + compatible: > + const: ti,lm3533-leds > + > + reg: > + description: > + 4 led banks > + minimum: 0 > + maximum: 3 > + > + linux,default-trigger: true > + > + ti,max-current-microamp: > + description: > + Maximum current in µA with a 800 µA step. > + enum: [ 5000, 5800, 6600, 7400, 8200, 9000, 9800, > + 10600, 11400, 12200, 13000, 13800, 14600, > + 15400, 16200, 17000, 17800, 18600, 19400, > + 20200, 21000, 21800, 22600, 23400, 24200, > + 25000, 25800, 26600, 27400, 28200, 29000, > + 29800 ] > + default: 5000 > + > + ti,pwm-config-mask: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: > + Same descryption and function as for backlight. > + > + required: > + - compatible > + - reg > + > + additionalProperties: false > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/gpio/gpio.h> > + > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + led-controller@36 { > + compatible = "ti,lm3533"; > + reg = <0x36>; > + > + enable-gpios = <&gpio 110 GPIO_ACTIVE_HIGH>; > + > + ti,boost-ovp-microvolt = <24000000>; > + ti,boost-freq-hz = <500000>; > + > + #address-cells = <1>; > + #size-cells = <0>; > + > + backlight@0 { > + compatible = "ti,lm3533-backlight"; > + reg = <0>; > + > + ti,max-current-microamp = <23400>; > + default-brightness = <113>; > + ti,hardware-controlled; > + }; Please make the example complete. > + }; > + }; > +... > -- > 2.43.0 >
пт, 21 лют. 2025 р. о 22:38 Rob Herring <robh@kernel.org> пише: > > On Tue, Feb 18, 2025 at 03:26:59PM +0200, Svyatoslav Ryhel wrote: > > Add bindings for the LM3533 - a complete power source for > > backlight, keypad, and indicator LEDs in smartphone handsets. > > The high-voltage inductive boost converter provides the > > power for two series LED strings display backlight and keypad > > functions. > > > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com> > > --- > > .../devicetree/bindings/mfd/ti,lm3533.yaml | 231 ++++++++++++++++++ > > 1 file changed, 231 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > > > > diff --git a/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > > new file mode 100644 > > index 000000000000..83542f0c7bf7 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > > @@ -0,0 +1,231 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/mfd/ti,lm3533.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: TI LM3533 Complete Lighting Power Solution > > + > > +description: | > > Use '>' rather than '|' if only formatting is paragraphs. > yaml check did not complain, fine. > > + The LM3533 is a complete power source for backlight, > > + keypad, and indicator LEDs in smartphone handsets. The > > + high-voltage inductive boost converter provides the > > + power for two series LED strings display backlight and > > + keypad functions. > > Wrap lines at 80 > Checkpatch and yaml check do not complain, why? 80 char limit was removed, no? > blank line here > > > + https://www.ti.com/product/LM3533 > > + > > +maintainers: > > + - Svyatoslav Ryhel <clamor95@gmail.com> > > + > > +properties: > > + compatible: > > + const: ti,lm3533 > > + > > + reg: > > + maxItems: 1 > > + > > + '#address-cells': > > + const: 1 > > + > > + '#size-cells': > > + const: 0 > > + > > + enable-gpios: > > + description: GPIO to use to enable/disable the backlight (HWEN pin). > > + maxItems: 1 > > + > > + ti,boost-ovp-microvolt: > > + description: > > + Boost OVP select (16V, 24V, 32V, 40V) > > + enum: [ 16000000, 24000000, 32000000, 40000000 ] > > + default: 16000000 > > + > > + ti,boost-freq-hz: > > + description: > > + Boost frequency select (500KHz or 1MHz) > > + enum: [ 500000, 1000000 ] > > + default: 500000 > > + > > + light-sensor@0: > > + type: object > > + description: > > + Properties for an illumination sensor. > > + > > + properties: > > + compatible: > > + const: ti,lm3533-als > > + > > + reg: > > + const: 0 > > + > > + ti,resistor-value-ohm: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: | > > Don't need '|'. Elsewhere too. > > > + Internal configuration resister value when ALS is in Analog Sensor > > + mode and PWM mode is disabled. > > + minimum: 1575 > > + maximum: 200000 > > + > > + ti,pwm-mode: > > + type: boolean > > + description: | > > + Switch for mode in which ALS is running. If this propertly is > > + set then ALS is running in PWM mode, internal resistor value is > > + set to high-impedance (0) and resistor-value-ohm propertly is > > + ignored. > > + > > + required: > > + - compatible > > + - reg > > + > > + additionalProperties: false > > Move this above 'properties'. > yaml check did not complain. additionalProperties is always set after all properties description, no? > > + > > +required: > > + - compatible > > + - reg > > + - '#address-cells' > > + - '#size-cells' > > + - enable-gpios > > + > > +patternProperties: > > + "^backlight@[01]$": > > + type: object > > + description: > > + Properties for a backlight device. > > + > > + $ref: /schemas/leds/backlight/common.yaml# > > + > > + properties: > > + compatible: > > + const: ti,lm3533-backlight > > + > > + reg: > > + description: | > > + The control bank that is used to program the two current sinks. The > > + LM3533 has two control banks (A and B) and are represented as 0 or 1 > > + in this property. The two current sinks can be controlled > > + independently with both banks, or bank A can be configured to control > > + both sinks with the led-sources property. > > + minimum: 0 > > + maximum: 1 > > + > > + default-brightness: true > > + > > + ti,max-current-microamp: > > + description: > > + Maximum current in 渙 with a 800 渙 step. > > + enum: [ 5000, 5800, 6600, 7400, 8200, 9000, 9800, > > + 10600, 11400, 12200, 13000, 13800, 14600, > > + 15400, 16200, 17000, 17800, 18600, 19400, > > + 20200, 21000, 21800, 22600, 23400, 24200, > > + 25000, 25800, 26600, 27400, 28200, 29000, > > + 29800 ] > > + default: 5000 > > + > > + ti,pwm-config-mask: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: | > > + Control Bank PWM Configuration Register mask that allows to configure > > + PWM input in Zones 0-4 > > + BIT(0) - PWM Input is enabled > > + BIT(1) - PWM Input is enabled in Zone 0 > > + BIT(2) - PWM Input is enabled in Zone 1 > > + BIT(3) - PWM Input is enabled in Zone 2 > > + BIT(4) - PWM Input is enabled in Zone 3 > > + BIT(5) - PWM Input is enabled in Zone 4 > > + > > + ti,linear-mapping-mode: > > + description: | > > + Enable linear mapping mode. If disabled, then it will use exponential > > + mapping mode in which the ramp up/down appears to have a more uniform > > + transition to the human eye. > > + type: boolean > > + > > + ti,hardware-controlled: > > + description: | > > + Each backlight has its own voltage Control Bank (A and B) and there are > > + two HVLED sinks which by default are linked to respective Bank. Setting > > + this property will link both sinks to a Control Bank of backlight where > > + property is defined. > > + type: boolean > > + > > + required: > > + - compatible > > + - reg > > + > > + additionalProperties: false > > + > > + "^led@[0-3]$": > > + type: object > > + description: > > + Properties for a led device. > > + > > + $ref: /schemas/leds/common.yaml# > > + > > + properties: > > + compatible: > > + const: ti,lm3533-leds > > + > > + reg: > > + description: > > + 4 led banks > > + minimum: 0 > > + maximum: 3 > > + > > + linux,default-trigger: true > > + > > + ti,max-current-microamp: > > + description: > > + Maximum current in 渙 with a 800 渙 step. > > + enum: [ 5000, 5800, 6600, 7400, 8200, 9000, 9800, > > + 10600, 11400, 12200, 13000, 13800, 14600, > > + 15400, 16200, 17000, 17800, 18600, 19400, > > + 20200, 21000, 21800, 22600, 23400, 24200, > > + 25000, 25800, 26600, 27400, 28200, 29000, > > + 29800 ] > > + default: 5000 > > + > > + ti,pwm-config-mask: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: > > + Same descryption and function as for backlight. > > + > > + required: > > + - compatible > > + - reg > > + > > + additionalProperties: false > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + #include <dt-bindings/gpio/gpio.h> > > + > > + i2c { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + led-controller@36 { > > + compatible = "ti,lm3533"; > > + reg = <0x36>; > > + > > + enable-gpios = <&gpio 110 GPIO_ACTIVE_HIGH>; > > + > > + ti,boost-ovp-microvolt = <24000000>; > > + ti,boost-freq-hz = <500000>; > > + > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + backlight@0 { > > + compatible = "ti,lm3533-backlight"; > > + reg = <0>; > > + > > + ti,max-current-microamp = <23400>; > > + default-brightness = <113>; > > + ti,hardware-controlled; > > + }; > > Please make the example complete. > > > + }; > > + }; > > +... > > -- > > 2.43.0 > >
On Sat, 22 Feb 2025 09:01:18 +0200 Svyatoslav Ryhel <clamor95@gmail.com> wrote: > пт, 21 лют. 2025 р. о 22:38 Rob Herring <robh@kernel.org> пише: > > > > On Tue, Feb 18, 2025 at 03:26:59PM +0200, Svyatoslav Ryhel wrote: > > > Add bindings for the LM3533 - a complete power source for > > > backlight, keypad, and indicator LEDs in smartphone handsets. > > > The high-voltage inductive boost converter provides the > > > power for two series LED strings display backlight and keypad > > > functions. > > > > > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com> > > > --- > > > .../devicetree/bindings/mfd/ti,lm3533.yaml | 231 ++++++++++++++++++ > > > 1 file changed, 231 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > > > > > > diff --git a/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > > > new file mode 100644 > > > index 000000000000..83542f0c7bf7 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml > > > @@ -0,0 +1,231 @@ > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > > +%YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/mfd/ti,lm3533.yaml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: TI LM3533 Complete Lighting Power Solution > > > + > > > +description: | > > > > Use '>' rather than '|' if only formatting is paragraphs. > > > > yaml check did not complain, fine. It can't tell whether you want it formatted exactly or if paragraphs is enough. > > > > + The LM3533 is a complete power source for backlight, > > > + keypad, and indicator LEDs in smartphone handsets. The > > > + high-voltage inductive boost converter provides the > > > + power for two series LED strings display backlight and > > > + keypad functions. > > > > Wrap lines at 80 > > > > Checkpatch and yaml check do not complain, why? 80 char limit was removed, no? Check was to prevent over 80 chars, here you are far too short.
On Tue, 18 Feb 2025 15:26:59 +0200 Svyatoslav Ryhel <clamor95@gmail.com> wrote: > Add bindings for the LM3533 - a complete power source for > backlight, keypad, and indicator LEDs in smartphone handsets. > The high-voltage inductive boost converter provides the > power for two series LED strings display backlight and keypad > functions. Wrap patch descriptions to 75 chars as describe in submitting-patches.rst > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
сб, 22 лют. 2025 р. о 16:29 Jonathan Cameron <jic23@kernel.org> пише: > > On Tue, 18 Feb 2025 15:26:59 +0200 > Svyatoslav Ryhel <clamor95@gmail.com> wrote: > > > Add bindings for the LM3533 - a complete power source for > > backlight, keypad, and indicator LEDs in smartphone handsets. > > The high-voltage inductive boost converter provides the > > power for two series LED strings display backlight and keypad > > functions. > > Wrap patch descriptions to 75 chars as describe in submitting-patches.rst > Alright, though why then checkpatch script has max line length 100 chars? https://github.com/torvalds/linux/commit/bdc48fa11e46f867ea4d75fa59ee87a7f48be144 > > > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
On Sat, 22 Feb 2025 16:39:31 +0200 Svyatoslav Ryhel <clamor95@gmail.com> wrote: > сб, 22 лют. 2025 р. о 16:29 Jonathan Cameron <jic23@kernel.org> пише: > > > > On Tue, 18 Feb 2025 15:26:59 +0200 > > Svyatoslav Ryhel <clamor95@gmail.com> wrote: > > > > > Add bindings for the LM3533 - a complete power source for > > > backlight, keypad, and indicator LEDs in smartphone handsets. > > > The high-voltage inductive boost converter provides the > > > power for two series LED strings display backlight and keypad > > > functions. > > > > Wrap patch descriptions to 75 chars as describe in submitting-patches.rst > > > > Alright, though why then checkpatch script has max line length 100 chars? > > https://github.com/torvalds/linux/commit/bdc48fa11e46f867ea4d75fa59ee87a7f48be144 Because that script is intended to catch when things are very wrong not slightly so. It provides guidance that you should look at and consider whether to respond to. Checking for short wrap is trickier to do as perhaps the formatting is intended in some cases. Jonathan > > > > > > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
diff --git a/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml new file mode 100644 index 000000000000..83542f0c7bf7 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/ti,lm3533.yaml @@ -0,0 +1,231 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/mfd/ti,lm3533.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: TI LM3533 Complete Lighting Power Solution + +description: | + The LM3533 is a complete power source for backlight, + keypad, and indicator LEDs in smartphone handsets. The + high-voltage inductive boost converter provides the + power for two series LED strings display backlight and + keypad functions. + https://www.ti.com/product/LM3533 + +maintainers: + - Svyatoslav Ryhel <clamor95@gmail.com> + +properties: + compatible: + const: ti,lm3533 + + reg: + maxItems: 1 + + '#address-cells': + const: 1 + + '#size-cells': + const: 0 + + enable-gpios: + description: GPIO to use to enable/disable the backlight (HWEN pin). + maxItems: 1 + + ti,boost-ovp-microvolt: + description: + Boost OVP select (16V, 24V, 32V, 40V) + enum: [ 16000000, 24000000, 32000000, 40000000 ] + default: 16000000 + + ti,boost-freq-hz: + description: + Boost frequency select (500KHz or 1MHz) + enum: [ 500000, 1000000 ] + default: 500000 + + light-sensor@0: + type: object + description: + Properties for an illumination sensor. + + properties: + compatible: + const: ti,lm3533-als + + reg: + const: 0 + + ti,resistor-value-ohm: + $ref: /schemas/types.yaml#/definitions/uint32 + description: | + Internal configuration resister value when ALS is in Analog Sensor + mode and PWM mode is disabled. + minimum: 1575 + maximum: 200000 + + ti,pwm-mode: + type: boolean + description: | + Switch for mode in which ALS is running. If this propertly is + set then ALS is running in PWM mode, internal resistor value is + set to high-impedance (0) and resistor-value-ohm propertly is + ignored. + + required: + - compatible + - reg + + additionalProperties: false + +required: + - compatible + - reg + - '#address-cells' + - '#size-cells' + - enable-gpios + +patternProperties: + "^backlight@[01]$": + type: object + description: + Properties for a backlight device. + + $ref: /schemas/leds/backlight/common.yaml# + + properties: + compatible: + const: ti,lm3533-backlight + + reg: + description: | + The control bank that is used to program the two current sinks. The + LM3533 has two control banks (A and B) and are represented as 0 or 1 + in this property. The two current sinks can be controlled + independently with both banks, or bank A can be configured to control + both sinks with the led-sources property. + minimum: 0 + maximum: 1 + + default-brightness: true + + ti,max-current-microamp: + description: + Maximum current in µA with a 800 µA step. + enum: [ 5000, 5800, 6600, 7400, 8200, 9000, 9800, + 10600, 11400, 12200, 13000, 13800, 14600, + 15400, 16200, 17000, 17800, 18600, 19400, + 20200, 21000, 21800, 22600, 23400, 24200, + 25000, 25800, 26600, 27400, 28200, 29000, + 29800 ] + default: 5000 + + ti,pwm-config-mask: + $ref: /schemas/types.yaml#/definitions/uint32 + description: | + Control Bank PWM Configuration Register mask that allows to configure + PWM input in Zones 0-4 + BIT(0) - PWM Input is enabled + BIT(1) - PWM Input is enabled in Zone 0 + BIT(2) - PWM Input is enabled in Zone 1 + BIT(3) - PWM Input is enabled in Zone 2 + BIT(4) - PWM Input is enabled in Zone 3 + BIT(5) - PWM Input is enabled in Zone 4 + + ti,linear-mapping-mode: + description: | + Enable linear mapping mode. If disabled, then it will use exponential + mapping mode in which the ramp up/down appears to have a more uniform + transition to the human eye. + type: boolean + + ti,hardware-controlled: + description: | + Each backlight has its own voltage Control Bank (A and B) and there are + two HVLED sinks which by default are linked to respective Bank. Setting + this property will link both sinks to a Control Bank of backlight where + property is defined. + type: boolean + + required: + - compatible + - reg + + additionalProperties: false + + "^led@[0-3]$": + type: object + description: + Properties for a led device. + + $ref: /schemas/leds/common.yaml# + + properties: + compatible: + const: ti,lm3533-leds + + reg: + description: + 4 led banks + minimum: 0 + maximum: 3 + + linux,default-trigger: true + + ti,max-current-microamp: + description: + Maximum current in µA with a 800 µA step. + enum: [ 5000, 5800, 6600, 7400, 8200, 9000, 9800, + 10600, 11400, 12200, 13000, 13800, 14600, + 15400, 16200, 17000, 17800, 18600, 19400, + 20200, 21000, 21800, 22600, 23400, 24200, + 25000, 25800, 26600, 27400, 28200, 29000, + 29800 ] + default: 5000 + + ti,pwm-config-mask: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Same descryption and function as for backlight. + + required: + - compatible + - reg + + additionalProperties: false + +additionalProperties: false + +examples: + - | + #include <dt-bindings/gpio/gpio.h> + + i2c { + #address-cells = <1>; + #size-cells = <0>; + + led-controller@36 { + compatible = "ti,lm3533"; + reg = <0x36>; + + enable-gpios = <&gpio 110 GPIO_ACTIVE_HIGH>; + + ti,boost-ovp-microvolt = <24000000>; + ti,boost-freq-hz = <500000>; + + #address-cells = <1>; + #size-cells = <0>; + + backlight@0 { + compatible = "ti,lm3533-backlight"; + reg = <0>; + + ti,max-current-microamp = <23400>; + default-brightness = <113>; + ti,hardware-controlled; + }; + }; + }; +...
Add bindings for the LM3533 - a complete power source for backlight, keypad, and indicator LEDs in smartphone handsets. The high-voltage inductive boost converter provides the power for two series LED strings display backlight and keypad functions. Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com> --- .../devicetree/bindings/mfd/ti,lm3533.yaml | 231 ++++++++++++++++++ 1 file changed, 231 insertions(+) create mode 100644 Documentation/devicetree/bindings/mfd/ti,lm3533.yaml