27c9d80f51
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
52 lines
1.8 KiB
Diff
52 lines
1.8 KiB
Diff
From 4459c1e7e1a6e33d39f1cdb2524a02cb15512589 Mon Sep 17 00:00:00 2001
|
|
From: Phil Elwell <phil@raspberrypi.org>
|
|
Date: Fri, 9 Aug 2019 08:51:43 +0100
|
|
Subject: [PATCH 0170/1085] net: bcmgenet: Workaround #2 for Pi4 Ethernet fail
|
|
|
|
Some combinations of Pi 4Bs and Ethernet switches don't reliably get a
|
|
DCHP-assigned IP address, leaving the unit with a self=assigned 169.254
|
|
address. In the failure case, the Pi is left able to receive packets
|
|
but not send them, suggesting that the MAC<->PHY link is getting into
|
|
a bad state.
|
|
|
|
It has been found empirically that skipping a reset step by the genet
|
|
driver prevents the failures. No downsides have been discovered yet,
|
|
and unlike the forced renegotiation it doesn't increase the time to
|
|
get an IP address, so the workaround is enabled by default; add
|
|
|
|
genet.skip_umac_reset=n
|
|
|
|
to the command line to disable it.
|
|
|
|
See: https://github.com/raspberrypi/linux/issues/3108
|
|
|
|
Signed-off-by: Phil Elwell <phil@raspberrypi.org>
|
|
---
|
|
drivers/net/ethernet/broadcom/genet/bcmgenet.c | 8 ++++++++
|
|
1 file changed, 8 insertions(+)
|
|
|
|
--- a/drivers/net/ethernet/broadcom/genet/bcmgenet.c
|
|
+++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
|
|
@@ -67,6 +67,9 @@
|
|
|
|
/* Forward declarations */
|
|
static void bcmgenet_set_rx_mode(struct net_device *dev);
|
|
+static bool skip_umac_reset = true;
|
|
+module_param(skip_umac_reset, bool, 0444);
|
|
+MODULE_PARM_DESC(skip_umac_reset, "Skip UMAC reset step");
|
|
|
|
static inline void bcmgenet_writel(u32 value, void __iomem *offset)
|
|
{
|
|
@@ -2495,6 +2498,11 @@ static void reset_umac(struct bcmgenet_p
|
|
bcmgenet_rbuf_ctrl_set(priv, 0);
|
|
udelay(10);
|
|
|
|
+ if (skip_umac_reset) {
|
|
+ pr_warn("Skipping UMAC reset\n");
|
|
+ return;
|
|
+ }
|
|
+
|
|
/* issue soft reset and disable MAC while updating its registers */
|
|
spin_lock_bh(&priv->reg_lock);
|
|
bcmgenet_umac_writel(priv, CMD_SW_RESET, UMAC_CMD);
|