Re: [PATCH v4 6/7] dt-bindings: net: dsa: mediatek,mt7530: add EcoNet EN751221 switches
From: Caleb James DeLisle
Date: Thu Nov 05 2026 - 16:48:33 EST
On 04/11/2026 08:20, Krzysztof Kozlowski wrote:
On 03/11/2026 14:15, Caleb James DeLisle wrote:
The drivers do not matter - we have one compatible for I2C/SPI/Serial+ const: econet,en751221-gswitchThat's almost the same compatible, which makes me wonder if you describe
the same device twice. Same device, same compatible.
Well, these are both MT7530 based, but one of them is on-die with the
SoC and the other one is an MCM so for the driver they are very
different - they're even loaded by different kmods. I think since one is
MMIO and the other is MDIO, using the same compatible won't cause a
driver problem - but I don't know if it's best practice to smooth over
such a difference or not. Your advice here would be most appreciated.
devices as well. The only question is about the hardware, what are these
devices and what are you here exactly describing. Compatible is en751221
but you call it MT7530, so even more confusion.
Let me address these in reverse order: The compatibles are econet,en751221-switch and econet,en751221-gswitch but the switch is an MT7530. This is a convention used for this switch: When it is integrated in an SoC and has customization for that SoC, it is identified using the name of the SoC suffixed by "-switch" (except in the case of mediatek,mt7621 where it takes the exact name of the SoC, but this looks like it was a mistake).
Regarding the en751221-switch vs. en751221-gswitch dichotomy, the meaningful difference to the DT is that the gswitch is a multi-chip-module so the CPU port is phy-mode "trgmii", and on the on-die switch it's "internal" since there's no PHY between it and the CPU. Moreover the gswitch requires the mediatek,mcm; attribute while the on-die switch forbids it. I can of course relax the restrictions so they can use the same compatible, but I don't know which way is most appropriate so I would like to follow your advice.
Thanks,
Caleb
Best regards,
Krzysztof