Re: [PATCH v2] dt-bindings: interrupt-controller: Convert nxp,lpc3220-mic.txt to yaml format

[Leonardo Felipe Takao Hirata]

Hello Krzysztof and Vladimir,

Thanks a lot for your time. I really appreciate your feedbacks.

On 26/02/2025 7:14 AM, Krzysztof Kozlowski wrote:
> 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

If I understood correctly, the first item of interrupts is dedicated to IRQ type 
and the second to FIQ type. Then, I was thinking about listing them like:

 - description: Regular interrupt request
 - description: Fast interrupt request

What do you guys think?

Best regards,
Leonardo Hirata