[PATCH v3 1/4] dt-bindings: leds: nxp,pca963x: fix reg maximum for pca9635
Loic Poulain <[email protected]>
| Newsgroups | org.kernel.vger.linux-leds,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
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 let pca9635 fall through the else branch with the correct maximum of 15. Signed-off-by: Loic Poulain <[email protected]> --- Documentation/devicetree/bindings/leds/nxp,pca963x.yaml | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml b/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml index 938d0e48fe51bce82779c4457c8e99cb6d80fe70..4034139e3f770a1035208490a9436bdc33387279 100644 --- a/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml +++ b/Documentation/devicetree/bindings/leds/nxp,pca963x.yaml @@ -88,12 +88,24 @@ allOf: properties: reg: maximum: 3 - else: + - if: + properties: + compatible: + contains: + enum: + - nxp,pca9634 + then: patternProperties: "^led@[0-9a-f]+$": properties: reg: maximum: 7 + else: + patternProperties: + "^led@[0-9a-f]+$": + properties: + reg: + maximum: 15 additionalProperties: false -- 2.34.1