On 26/02/2025 10:41, Vladimir Zapolskiy wrote: >> + >> +properties: >> + compatible: >> + enum: >> + - nxp,lpc3220-mic >> + - nxp,lpc3220-sic >> + >> + reg: >> + maxItems: 1 >> + >> + interrupt-controller: true >> + >> + '#interrupt-cells': >> + const: 2 >> + >> + interrupts: >> + items: >> + - description: >> + IRQ number. > > For sake of better understanding SIC1 and SIC2 interrupt controllers > are chained to MIC, that's why there is 'interrupts' property present, > and here 0/1 interrupt values are for regular IRQ, 30/31 are for fast IRQ. > > Also please add here > > minItems: 2 > maxItems: 2 > > I believe that the 'interrupts' property can be just left without any > given description, or just give a simple description like > > IRQ and FIQ outputs of sub interrupt controllers to the main interrupt controller If they are obvious, e.g. maxItems: 2 would be enough, but I understand that these are distinctive interrupts from dedicated blocks - 1 and 2 - so they should be listed. - description: IRQ/FIO of SIC1 (or whatever name is here better) - description: IRQ/FIO of SIC2 Best regards, Krzysztof
