1 files changed, 10 insertions, 11 deletions

diff --git a/drivers/net/ethernet/intel/e1000e/netdev.c b/drivers/net/ethernet/intel/e1000e/netdev.c
index 40e6232bed27..99596a63b6d0 100644
--- a/drivers/net/ethernet/intel/e1000e/netdev.c
+++ b/drivers/net/ethernet/intel/e1000e/netdev.c
@@ -2979,17 +2979,10 @@ static void e1000_setup_rctl(struct e1000_adapter *adapter)
         u32 pages = 0;
 
         /* Workaround Si errata on PCHx - configure jumbo frame flow */
-        if (hw->mac.type >= e1000_pch2lan) {
-                s32 ret_val;
-
-                if (adapter->netdev->mtu > ETH_DATA_LEN)
-                        ret_val = e1000_lv_jumbo_workaround_ich8lan(hw, true);
-                else
-                        ret_val = e1000_lv_jumbo_workaround_ich8lan(hw, false);
-
-                if (ret_val)
-                        e_dbg("failed to enable jumbo frame workaround mode\n");
-        }
+        if ((hw->mac.type >= e1000_pch2lan) &&
+            (adapter->netdev->mtu > ETH_DATA_LEN) &&
+            e1000_lv_jumbo_workaround_ich8lan(hw, true))
+                e_dbg("failed to enable jumbo frame workaround mode\n");
 
         /* Program MC offset vector base */
         rctl = er32(RCTL);
@@ -4034,6 +4027,12 @@ void e1000e_down(struct e1000_adapter *adapter)
         adapter->link_speed = 0;
         adapter->link_duplex = 0;
 
+        /* Disable Si errata workaround on PCHx for jumbo frame flow */
+        if ((hw->mac.type >= e1000_pch2lan) &&
+            (adapter->netdev->mtu > ETH_DATA_LEN) &&
+            e1000_lv_jumbo_workaround_ich8lan(hw, false))
+                e_dbg("failed to disable jumbo frame workaround mode\n");
+
         if (!pci_channel_offline(adapter->pdev))
                 e1000e_reset(adapter);