Re: [PATCH net-next] net: phy: mediatek: do not match MT7530 PHY if 10/100 only
From: Caleb James DeLisle
Date: Fri Sep 18 2026 - 13:00:27 EST
On 18/09/2026 18:29, Andrew Lunn wrote:
On Fri, Sep 18, 2026 at 04:11:56PM +0000, Caleb James DeLisle wrote:
An EcoNet SoC 10/100 PHY uses the same ID as the MT7530 PHY, but a trueDon't you also need the opposite in mtk-fe-soc.c? Otherwise don't you
MT7530 PHY always has gigabit support so they can be distinguished by
the presence of the ESTATUS_1000_TFULL flag. Add a match_phy_device
which refuses to match if the PHY is not gigabit.
rely on the load order of the PHY drivers?
The collision is in econet-fe-soc.c of https://lore.kernel.org/netdev/20260827065930.2618729-3-cjd@xxxxxxxx/
And I notice now that I did add it in that patchset as well, but this implementation is better done and that patchset is probably going to have a v2 so I think it's better to accept it here.
Thanks,
Caleb
Andrew
---
pw-bot: cr