On 04.01.2019 16:57, Andrew Lunn wrote: >> I wonder, if I use the phylib functions instead of the ad-hoc ones in >> the MAC driver, is there still a problem with synchronization ? > > You would need to look deep into phylib. When does it reset the PHY? > Configure auto-neg, setup interrupts, etc? It looks like both are > going to do this, so i expect they are going to mess each other up. > Marek, I recently went through this exercise when switching r8169 driver to phylib. As Andrew explained: First add needed functionality to the respective PHY driver(s), then you can switch the network driver. You can look at f1e911d5d0df ("r8169: add basic phylib support") plus related changes, and to what was added to the Realtek PHY driver module.
- [PATCH 17/19] usbnet: smsc95xx: Replace generic_mii_ioctl(... Marek Vasut
- [PATCH 16/19] usbnet: smsc95xx: Replace mii_check_media() Marek Vasut
- [PATCH 11/19] usbnet: smsc95xx: Replace smsc95xx_mdio_writ... Marek Vasut
- [PATCH 14/19] usbnet: smsc95xx: Replace mii_nway_restart() Marek Vasut
- [PATCH 10/19] usbnet: smsc95xx: Replace smsc95xx_mdio_read... Marek Vasut
- Re: [PATCH 10/19] usbnet: smsc95xx: Replace smsc95xx_... Andrew Lunn
- Re: [PATCH 10/19] usbnet: smsc95xx: Replace smsc9... Marek Vasut
- Re: [PATCH 10/19] usbnet: smsc95xx: Replace s... Andrew Lunn
- Re: [PATCH 10/19] usbnet: smsc95xx: Repla... Marek Vasut
- Re: [PATCH 10/19] usbnet: smsc95xx: ... Andrew Lunn
- Re: [PATCH 10/19] usbnet: smsc95... Heiner Kallweit
- [PATCH 07/19] usbnet: smsc95xx: Split the reset function Marek Vasut
- Re: [PATCH 07/19] usbnet: smsc95xx: Split the reset f... Oliver Neukum
- [PATCH 03/19] usbnet: smsc95xx: Remove smsc95xx_{read,writ... Marek Vasut
- [PATCH 05/19] usbnet: smsc95xx: Remove smsc95xx_{read,writ... Marek Vasut
- [PATCH 09/19] usbnet: smsc95xx: Connect to phydev Marek Vasut
- [PATCH 15/19] usbnet: smsc95xx: Replace mii_ethtool_gset() Marek Vasut
- [PATCH 04/19] usbnet: smsc95xx: Remove __smsc95xx_{read,wr... Marek Vasut
- [PATCH 06/19] usbnet: smsc95xx: Remove __smsc95xx_mdio_{re... Marek Vasut