RE: [PATCH v3 1/2] dt-bindings: interrupt-controller: aspeed: Add parent node compatibles and refine documentation
From: Ryan Chen <ryan_chen@aspeedtech.com>
Date: 2025-07-23 08:08:29
Also in:
linux-aspeed, linux-devicetree, lkml
Subject: Re: [PATCH v3 1/2] dt-bindings: interrupt-controller: aspeed: Add parent node compatibles and refine documentation On 22/07/2025 11:51, Ryan Chen wrote:quoted
+ INTC0 is used to assert GIC if interrupt in INTC1 asserted. + INTC1 is used to assert INTC0 if interrupt of modules asserted. + +-----+ +---------+ + | GIC |---| INTC0 | + +-----+ +---------+ + +---------+ + | |---module0 + | INTC0_0 |---module1 + | |---... + +---------+---module31 + |---.... | + +---------+ + | | +---------+ + | INTC0_11| +---| INTC1 | + | | +---------+ + +---------+ +---------+---module0 + | INTC1_0 |---module1 + | |---... + +---------+---module31 + ... + +---------+---module0 + | INTC1_5 |---module1 + | |---... + +---------+---module31You binding also said intc1 is the parent of intc-ic, so where is here intc-ic? This diagram and new binding do not match at all.
The corresponded compatible is following.
+-----+ +---------+
| GIC |---| INTC0 | -> (parent : aspeed,ast2700-intc0)
+-----+ +---------+
+---------+
| |---module0
| INTC0_0 |---module1
(child : aspeed,ast2700-intc-ic)
| |---...
+---------+---module31
|---.... |
+---------+
| | +---------+
| INTC0_11 | +---------------------------- | INTC1 | -> -> (parent : aspeed,ast2700-intc1)
(child : aspeed,ast2700-intc-ic)
| | +---------+
+---------+ +-------- -+---module0
| INTC1_0 | --> (child : aspeed,ast2700-intc-ic)
| |---...
+--------- +---module31
...
+--------- +---module0
| INTC1_5 | --> ((child : aspeed,ast2700-intc-ic))
| |---...
+--------- +---module31
Best regards, Krzysztof