c06fb25d1f
Some checks failed
Build Kernel / Build all affected Kernels (push) Has been cancelled
Build all core packages / Build all core packages for selected target (push) Has been cancelled
Build and Push prebuilt tools container / Build and Push all prebuilt containers (push) Has been cancelled
Build Toolchains / Build Toolchains for each target (push) Has been cancelled
Build host tools / Build host tools for linux and macos based systems (push) Has been cancelled
Coverity scan build / Coverity x86/64 build (push) Has been cancelled
33 lines
1.1 KiB
Diff
33 lines
1.1 KiB
Diff
From d18f8524afab3a65b29207fdf69303efd5f2db88 Mon Sep 17 00:00:00 2001
|
|
From: Phil Elwell <phil@raspberrypi.com>
|
|
Date: Tue, 15 Dec 2020 16:38:37 +0000
|
|
Subject: [PATCH 0270/1085] net: lan78xx: Ack pending PHY ints when resetting
|
|
|
|
lan78xx_link_reset explicitly clears the MAC's view of the PHY's IRQ
|
|
status. In doing so it potentially leaves the PHY with a pending
|
|
interrupt that will never be acknowledged, at which point no further
|
|
interrupts will be generated.
|
|
|
|
Avoid the problem by acknowledging any pending PHY interrupt after
|
|
clearing the MAC's status bit.
|
|
|
|
See: https://github.com/raspberrypi/linux/issues/2937
|
|
|
|
Signed-off-by: Phil Elwell <phil@raspberrypi.com>
|
|
---
|
|
drivers/net/usb/lan78xx.c | 3 +++
|
|
1 file changed, 3 insertions(+)
|
|
|
|
--- a/drivers/net/usb/lan78xx.c
|
|
+++ b/drivers/net/usb/lan78xx.c
|
|
@@ -1440,6 +1440,9 @@ static int lan78xx_link_reset(struct lan
|
|
if (unlikely(ret < 0))
|
|
return ret;
|
|
|
|
+ /* Acknowledge any pending PHY interrupt, lest it be the last */
|
|
+ phy_read(phydev, LAN88XX_INT_STS);
|
|
+
|
|
mutex_lock(&phydev->lock);
|
|
phy_read_status(phydev);
|
|
link = phydev->link;
|