On Wed, Dec 02, 2020 at 09:23:40PM -0000, Grant Edwards wrote: > On 2020-12-02, Andrew Lunn <and...@lunn.ch> wrote: > >> > So it will access the MDIO bus of the PHY that is attached to the > >> > MAC. > >> > >> If that's the case, wouldn't the ioctl() calls "just work" even when > >> only the fixed-phy mdio bus and fake PHY are declared in the device > >> tree? > > > > The fixed-link PHY is connected to the MAC. So the IOCTL calls will be > > made to the fixed-link fake MDIO bus. > > So how do you control which of the two mdio buses is connected to > the MAC?
The bus is not connected to the MAC. The PHY is. https://elixir.bootlin.com/linux/HEAD/source/drivers/net/ethernet/cadence/macb_main.c#L699 if (dn) ret = phylink_of_phy_connect(bp->phylink, dn, 0); if (!dn || (ret && !macb_phy_handle_exists(dn))) { phydev = phy_find_first(bp->mii_bus); if (!phydev) { netdev_err(dev, "no PHY found\n"); return -ENXIO; } /* attach the mac to the phy */ ret = phylink_connect_phy(bp->phylink, phydev); } The call to phylink_of_phy_connect() will go looking in device tree to find the phy-handle. If it exists, it follows it to the PHY, and the PHY is connected to the MAC. This code also handles fixed link. But in this case, because it is using phylink, not phylib, the emulation is different. The phylib fixed-link has the limitation of only emulating C22 PHYs upto 1Gbps. 2.5G, 10G etc is becoming more popular, so Russell King implemented fixed-link in phylink differently. phylib has emulated MDIO registers which the generic PHY driver uses. phylink however incorporates fixed-link into the core code, there is no emulation. And IOCTL is a stub. Back to the code. If there is no PHY via device tree, it searches its own MDIO bus for the first PHY, and connects that to the MAC. This is obviously not ideal when you have multiple devices on the bus, so using a phy-handle is best practice. This code is here for backwards compatibility. macb has some funky backwards compatibility code with respect to its MDIO bus. So you might be hitting a corner case which is not handled correct. > > There are plenty of examples to follow. > > That's true, but knowing which ones do what you're trying to do is the > hard part. If you already know how to do it, it's easy to find > examples showing it. :) Feel free to ask. Andrew
