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
