On 02.07.2018 23:02, Andrew Lunn wrote: >> +static int r8169_mdio_read_reg(struct mii_bus *mii_bus, int phyaddr, int >> phyreg) >> +{ >> + struct rtl8169_private *tp = mii_bus->priv; >> + >> + return rtl_readphy(tp, phyreg); > > So there is no support for phyaddr? > Right, the chip can access only the one internal PHY, therefore it doesn't support phyaddr.
> It would be better to trap the phyaddr which are not supported and > return 0xffff. > OK > Andrew >