r8169: fix jumbo configuration for RTL8168evl
authorHeiner Kallweit <hkallweit1@gmail.com>
Sun, 1 Dec 2019 09:27:14 +0000 (10:27 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sun, 1 Dec 2019 21:14:21 +0000 (13:14 -0800)
Alan reported [0] that network is broken since the referenced commit
when using jumbo frames. This commit isn't wrong, it just revealed
another issue that has been existing before. According to the vendor
driver the RTL8168e-specific jumbo config doesn't apply for RTL8168evl.

[0] https://lkml.org/lkml/2019/11/30/119

Fixes: 4ebcb113edcc ("r8169: fix jumbo packet handling on resume from suspend")
Reported-by: Alan J. Wylie <alan@wylie.me.uk>
Tested-by: Alan J. Wylie <alan@wylie.me.uk>
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169_main.c

index d47a038..78872ce 100644 (file)
@@ -3872,7 +3872,7 @@ static void rtl_hw_jumbo_enable(struct rtl8169_private *tp)
        case RTL_GIGA_MAC_VER_27 ... RTL_GIGA_MAC_VER_28:
                r8168dp_hw_jumbo_enable(tp);
                break;
-       case RTL_GIGA_MAC_VER_31 ... RTL_GIGA_MAC_VER_34:
+       case RTL_GIGA_MAC_VER_31 ... RTL_GIGA_MAC_VER_33:
                r8168e_hw_jumbo_enable(tp);
                break;
        default: