[PATCH v4 1/4] dt-bindings: leds: nxp,pca963x: fix reg maximum for pca9635
From: Loic Poulain
Date: Wed Jul 08 2026 - 10:24:29 EST
The pca9635 supports 16 LED channels, unlike the pca9634 which only
supports 8. The allOf conditional grouped both chips under a single
else branch capping reg at a maximum of 7.
Give pca9634 its own if/then block and set maximum: 15 unconditionally
in the top-level led@ node, making it the default for pca9635 and any
future compatible. Also tighten the node name regex from [0-9a-f]+ to
[0-9a-f] to match the single-digit hardware limit.
Signed-off-by: Loic Poulain <loic.poulain@xxxxxxxxxxxxxxxx>
---
Documentation/devicetree/bindings/leds/nxp,pca963x.yaml | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml b/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml
index 938d0e48fe51bce82779c4457c8e99cb6d80fe70..4fda602912f7a9443b8a77e46bc015e244f82e08 100644
--- a/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml
+++ b/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml
@@ -62,14 +62,14 @@ properties:
open-drain, newer chips to totem pole).
patternProperties:
- "^led@[0-9a-f]+$":
+ "^led@[0-9a-f]$":
type: object
$ref: common.yaml#
unevaluatedProperties: false
properties:
reg:
- minimum: 0
+ maximum: 15
required:
- reg
@@ -88,7 +88,13 @@ allOf:
properties:
reg:
maximum: 3
- else:
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - nxp,pca9634
+ then:
patternProperties:
"^led@[0-9a-f]+$":
properties:
--
2.34.1