On Wed, Jun 26, 2024 at 9:35 PM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote: > > 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. > Ok, will add minitems:3 here, and update example. Best regards Shengjiu Wang > > Best regards, > Krzysztof >