Re: [PATCH] dt-bindings: net: dsa: mt7530: Allow interrupts-extended dependency
From: Rob Herring <robh@kernel.org>
Date: 2026-01-13 22:30:04
Also in:
linux-arm-kernel, linux-devicetree, linux-mediatek, lkml
On Tue, Jan 13, 2026 at 12:00:20PM +0100, AngeloGioacchino Del Regno wrote:
quoted hunk ↗ jump to hunk
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