From 5622575c5e4bffb07307f39903bb95caad7d172d Mon Sep 17 00:00:00 2001 From: Michael Brown Date: Tue, 22 Jun 2021 14:48:10 +0100 Subject: [realtek] Work around hardware bug on RTL8211B The RTL8211B seems to have a bug that prevents the link from coming up unless the MII_MMD_DATA register is cleared. The Linux kernel driver applies this workaround (in rtl8211b_resume()) only to the specific RTL8211B PHY model, along with a matching workaround to set bit 9 of MII_MMD_DATA when suspending the PHY. Since we have no need to ever suspend the PHY, and since writing a zero ought to be harmless, we just clear the register unconditionally. Debugged-by: Nikolay Pertsev Tested-by: Nikolay Pertsev Signed-off-by: Michael Brown --- src/drivers/net/realtek.c | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'src/drivers') diff --git a/src/drivers/net/realtek.c b/src/drivers/net/realtek.c index 0af3416..a43efb6 100644 --- a/src/drivers/net/realtek.c +++ b/src/drivers/net/realtek.c @@ -420,6 +420,16 @@ static int realtek_phy_reset ( struct realtek_nic *rtl ) { */ } + /* Some cards (e.g. RTL8211B) have a hardware errata that + * requires the MII_MMD_DATA register to be cleared before the + * link will come up. + */ + if ( ( rc = mii_write ( &rtl->mii, MII_MMD_DATA, 0 ) ) != 0 ) { + /* Ignore failures, since the register may not be + * present on all PHYs. + */ + } + /* Restart autonegotiation */ if ( ( rc = mii_restart ( &rtl->mii ) ) != 0 ) { DBGC ( rtl, "REALTEK %p could not restart MII: %s\n", -- cgit v1.1