Re: [PATCH v4 1/3] dt-bindings: leds: is31fl32xx: convert the binding to yaml
From: Rob Herring <robh@kernel.org>
Date: 2025-07-17 19:31:19
Also in:
linux-leds, lkml
On Thu, Jul 17, 2025 at 04:02:07PM +0100, Pawel Zalewski wrote:
quoted hunk ↗ jump to hunk
From: Lucca Fachinetti <redacted> Add datasheets for reference, NB that I was not able to find an up-to-date, funtional direct URL for si-en products datasheet so they were skipped. Signed-off-by: Lucca Fachinetti <redacted> Co-developed-by: Pawel Zalewski <redacted> Signed-off-by: Pawel Zalewski <redacted> --- .../devicetree/bindings/leds/issi,is31fl3236.yaml | 114 +++++++++++++++++++++ .../devicetree/bindings/leds/leds-is31fl32xx.txt | 52 ---------- 2 files changed, 114 insertions(+), 52 deletions(-)diff --git a/Documentation/devicetree/bindings/leds/issi,is31fl3236.yaml b/Documentation/devicetree/bindings/leds/issi,is31fl3236.yaml new file mode 100644 index 0000000000000000000000000000000000000000..abcd4c448e94db1d1e339f7ee165c3e04a1d354d --- /dev/null +++ b/Documentation/devicetree/bindings/leds/issi,is31fl3236.yaml@@ -0,0 +1,114 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/leds/issi,is31fl3236.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: is31fl32xx and Si-En SN32xx IC LED driver + +maintainers: + - Pavel Machek <pavel@ucw.cz> + - Lee Jones <lee@kernel.org> + +description: | + The is31fl32xx/SN32xx family of LED drivers are I2C devices with multiple + constant-current channels, each with independent 256-level PWM control. + Each LED is represented as a sub-node of the device. + + For more product information please see the links below: + https://www.lumissil.com/assets/pdf/core/IS31FL3236_DS.pdf + https://www.lumissil.com/assets/pdf/core/IS31FL3236A_DS.pdf + https://www.lumissil.com/assets/pdf/core/IS31FL3235_DS.pdf + https://www.lumissil.com/assets/pdf/core/IS31FL3218_DS.pdf + https://www.lumissil.com/assets/pdf/core/IS31FL3216_DS.pdf + +properties: + compatible: + enum: + - issi,is31fl3236 + - issi,is31fl3236a + - issi,is31fl3235 + - issi,is31fl3218 + - issi,is31fl3216 + - si-en,sn3218 + - si-en,sn3216 + + reg: + maxItems: 1 + + "#address-cells": + const: 1 + + "#size-cells": + const: 0 + +patternProperties: + "^led@[1-9a-f][0-9a-f]$": + type: object + $ref: common.yaml# + unevaluatedProperties: false + + properties: + reg: + description: + LED channel number (1..N) + minimum: 0x1 + maximum: 0x24 + + required: + - reg + +allOf: + - if: + properties: + compatible: + contains: + enum: + - issi,is31fl3236 + - issi,is31fl3235 + - issi,is31fl3218 + - issi,is31fl3216 + - si-en,sn3218 + - si-en,sn3216 + then: + properties: + issi,22kHz-pwm: false
This is already false because you haven't defined the property anywhere. Though this property didn't exist in the existing binding, so why are you adding it (the commit msg needs an explanation)? Rob