On 8/26/19 6:52 PM, Voon Weifeng wrote:
> From: Ong Boon Leong <boon.leong....@intel.com>
>
> Make mdiobus_scan() to try harder to look for any PHY that only talks C45.
If you are not using Device Tree or ACPI, and you are letting the MDIO
bus be scanned, it sounds like there should be a way for you to provide
a hint as to which addresses should be scanned (that's
mii_bus::phy_mask) and possibly enhance that with a mask of possible C45
devices?
>
> Signed-off-by: Ong Boon Leong <boon.leong....@intel.com>
> Signed-off-by: Voon Weifeng <weifeng.v...@intel.com>
>
> diff --git a/drivers/net/phy/mdio_bus.c b/drivers/net/phy/mdio_bus.c
> index bd04fe762056..30dbc48b4c7e 100644
> --- a/drivers/net/phy/mdio_bus.c
> +++ b/drivers/net/phy/mdio_bus.c
> @@ -525,8 +525,12 @@ struct phy_device *mdiobus_scan(struct mii_bus *bus, int
> addr)
> int err;
>
> phydev = get_phy_device(bus, addr, false);
> - if (IS_ERR(phydev))
> - return phydev;
> + if (IS_ERR(phydev)) {
> + /* Try C45 to ensure we don't miss PHY that only talks C45 */
> + phydev = get_phy_device(bus, addr, true);
> + if (IS_ERR(phydev))
> + return phydev;
> + }
>
> /*
> * For DT, see if the auto-probed phy has a correspoding child
>
--
Florian
