dt-bindings: net: dsa: mt7530: Allow interrupts-extended dependency

[PATCH] dt-bindings: net: dsa: mt7530: Allow interrupts-extended dependency

Posted by AngeloGioacchino Del Regno 3 weeks, 5 days ago

When the MT7530 switch is configured as an interrupt-controller it
also needs an interrupt but, in this case, only "interrupts" was
allowed.

Some devicetrees instead use the interrupts-extended property as a
shorter form, and in place of "interrupts" and "interrupt-parent",
as an equivalent.

For this reason, when interrupt-controller is present, depend on
either `interrupts` or `interrupts-extended`; this also resolves
some dtbs_check warnings.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
---
 .../devicetree/bindings/net/dsa/mediatek,mt7530.yaml       | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml
index 815a90808901..ffeb8d5836fe 100644
--- a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml
+++ b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml
@@ -279,8 +279,11 @@ allOf:
         - resets
         - reset-names
 
-  - dependencies:
-      interrupt-controller: [ interrupts ]
+  - anyOf:
+      - dependencies:
+          interrupt-controller: [ interrupts ]
+      - dependencies:
+          interrupt-controller: [ interrupts-extended ]
 
   - if:
       properties:
-- 
2.52.0

Re: [PATCH] dt-bindings: net: dsa: mt7530: Allow interrupts-extended dependency

Posted by Rob Herring 3 weeks, 5 days ago

On Tue, Jan 13, 2026 at 12:00:20PM +0100, AngeloGioacchino Del Regno wrote:
> When the MT7530 switch is configured as an interrupt-controller it
> also needs an interrupt but, in this case, only "interrupts" was
> allowed.
> 
> Some devicetrees instead use the interrupts-extended property as a
> shorter form, and in place of "interrupts" and "interrupt-parent",
> as an equivalent.
> 
> For this reason, when interrupt-controller is present, depend on
> either `interrupts` or `interrupts-extended`; this also resolves
> some dtbs_check warnings.
> 
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> ---
>  .../devicetree/bindings/net/dsa/mediatek,mt7530.yaml       | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml
> index 815a90808901..ffeb8d5836fe 100644
> --- a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml
> +++ b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml
> @@ -279,8 +279,11 @@ allOf:
>          - resets
>          - reset-names
>  
> -  - dependencies:
> -      interrupt-controller: [ interrupts ]
> +  - anyOf:
> +      - dependencies:
> +          interrupt-controller: [ interrupts ]
> +      - dependencies:
> +          interrupt-controller: [ interrupts-extended ]

We already fixup this case:

dependencies:
  interrupts: ...

And there's a few other cases like yours. So I've pushed a dtschema 
change to handle this case.

Rob