e1000e: Remove special case for 82573/82574 ASPM L1 disablement
authorChris Boot <bootc@bootc.net>
Tue, 24 Apr 2012 07:24:58 +0000 (07:24 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Thu, 3 May 2012 09:32:36 +0000 (02:32 -0700)
For the 82573, ASPM L1 gets disabled wholesale so this special-case code
is not required. For the 82574 the previous patch does the same as for
the 82573, disabling L1 on the adapter. Thus, this code is no longer
required and can be removed.

Signed-off-by: Chris Boot <bootc@bootc.net>
Tested-by: Jeff Pieper <jeffrey.e.pieper@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/e1000e/netdev.c

index ab4505c99e672b59b09a2414e773a7f99a230112..9c4576ec36b4d56ca7945c645c1b02037233a016 100644 (file)
@@ -5272,14 +5272,6 @@ static int e1000_change_mtu(struct net_device *netdev, int new_mtu)
                return -EINVAL;
        }
 
-       /* 82573 Errata 17 */
-       if (((adapter->hw.mac.type == e1000_82573) ||
-            (adapter->hw.mac.type == e1000_82574)) &&
-           (max_frame > ETH_FRAME_LEN + ETH_FCS_LEN)) {
-               adapter->flags2 |= FLAG2_DISABLE_ASPM_L1;
-               e1000e_disable_aspm(adapter->pdev, PCIE_LINK_STATE_L1);
-       }
-
        while (test_and_set_bit(__E1000_RESETTING, &adapter->state))
                usleep_range(1000, 2000);
        /* e1000e_down -> e1000e_reset dependent on max_frame_size & mtu */