On 26/06/2024 11:34, Krzysztof Kozlowski wrote: >>>> items: >>>> - description: WAKEUPMIX Audio XCVR Interrupt 1 >>>> - description: WAKEUPMIX Audio XCVR Interrupt 2 >>>> + - description: SPDIF wakeup interrupt from PHY >>>> minItems: 1 >>>> >>>> clocks: >>>> @@ -88,6 +89,7 @@ required: >>>> - dma-names >>>> >>>> allOf: >>>> + - $ref: dai-common.yaml# >>>> - if: >>>> properties: >>>> compatible: >>>> @@ -112,7 +114,7 @@ allOf: >>>> else: >>>> properties: >>>> interrupts: >>>> - maxItems: 1 >>>> + maxItems: 3 >>> >>> I have doubts this was tested. The existing example should fail, >>> although I did not see any reports. Not sure why. >> >> it is tested, because the minItems is 1. so the example can pass >> > > That's a very unexpected change in dtschema. It indeed passes, but is > not correct. Why interrupts are flexible? This should be constrained. > Let's wait for Rob's confirmation that this is not a bug in dtschema. Yep we need here minItems as well, so this is strictly constrained or please explain why this should be flexible. Best regards, Krzysztof
