dt-bindings: interrupt-controller: marvell,cp110-icu: Document address-cells

[PATCH] dt-bindings: interrupt-controller: marvell,cp110-icu: Document address-cells

Posted by Krzysztof Kozlowski 1 month, 1 week ago

The CP110 ICU children are interrupt controllers and can be referenced
in interrupt-map properties (e.g. in
arch/arm64/boot/dts/marvell/armada-cp11x.dtsi), thus the nodes should
have address-cells property.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

See
https://lore.kernel.org/linux-devicetree/175589786306.518460.17450425303264875357.robh@kernel.org/
for reference.
---
 .../bindings/interrupt-controller/marvell,cp110-icu.yaml       | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/marvell,cp110-icu.yaml b/Documentation/devicetree/bindings/interrupt-controller/marvell,cp110-icu.yaml
index 9d4f06f45372..ddfce217e119 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/marvell,cp110-icu.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/marvell,cp110-icu.yaml
@@ -49,6 +49,9 @@ patternProperties:
       reg:
         maxItems: 1
 
+      '#address-cells':
+        const: 0
+
       '#interrupt-cells':
         const: 2
 
-- 
2.48.1

Re: [PATCH] dt-bindings: interrupt-controller: marvell,cp110-icu: Document address-cells

Posted by Rob Herring (Arm) 1 month, 1 week ago

On Sat, 23 Aug 2025 18:32:59 +0200, Krzysztof Kozlowski wrote:
> The CP110 ICU children are interrupt controllers and can be referenced
> in interrupt-map properties (e.g. in
> arch/arm64/boot/dts/marvell/armada-cp11x.dtsi), thus the nodes should
> have address-cells property.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> See
> https://lore.kernel.org/linux-devicetree/175589786306.518460.17450425303264875357.robh@kernel.org/
> for reference.
> ---
>  .../bindings/interrupt-controller/marvell,cp110-icu.yaml       | 3 +++
>  1 file changed, 3 insertions(+)
> 

Applied, thanks!

Re: [PATCH] dt-bindings: interrupt-controller: marvell,cp110-icu: Document address-cells

Posted by Miquel Raynal 1 month, 1 week ago

Hi,

On 23/08/2025 at 18:32:59 +02, Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:

> The CP110 ICU children are interrupt controllers and can be referenced
> in interrupt-map properties (e.g. in
> arch/arm64/boot/dts/marvell/armada-cp11x.dtsi), thus the nodes should
> have address-cells property.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>

Thanks,
Miquèl