On 20/11/2024 11:33, Ciprian Marian Costea wrote: > On 11/20/2024 11:12 AM, Krzysztof Kozlowski wrote: >> On 20/11/2024 09:45, Krzysztof Kozlowski wrote: >>> On Tue, Nov 19, 2024 at 10:10:51AM +0200, Ciprian Costea wrote: >>>> reg: >>>> maxItems: 1 >>>> @@ -136,6 +138,23 @@ required: >>>> - reg >>>> - interrupts >>>> >>>> +allOf: >>>> + - $ref: can-controller.yaml# >>>> + - if: >>>> + properties: >>>> + compatible: >>>> + contains: >>>> + const: nxp,s32g2-flexcan >>>> + then: >>>> + properties: >>>> + interrupts: >>>> + minItems: 4 >>>> + maxItems: 4 >>> >>> Top level says max is 1. You need to keep there widest constraints. >> And list items here instead... >> >> Best regards, >> Krzysztof > > Hello Krzysztof, > > Just to confirm before making any changes: > Are you referring to directly change 'maxItems' to value 4 ? Instead of No, I want you to create a list here. List the items. Nothing about "maxItems" in my message above (unless you quote earlier but then respond under proper quote). Just like other bindings are doing. https://elixir.bootlin.com/linux/v6.11-rc6/source/Documentation/devicetree/bindings/ufs/qcom,ufs.yaml#L127 > using this 'if' condition under 'allOf' ? Best regards, Krzysztof