diff options
author | Auke Kok <auke-jan.h.kok@intel.com> | 2007-10-31 15:22:00 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 15:03:43 -0800 |
commit | 1eae4eb2a1c784bf35ee4f8f158cd21cf8c387c1 (patch) | |
tree | 4e9de1ad1d24b201647376a21862a110e462c831 /drivers/net/e1000e/e1000.h | |
parent | 93ca161027eb6a1761fb674ad7b995aedccf5f6e (diff) | |
download | op-kernel-dev-1eae4eb2a1c784bf35ee4f8f158cd21cf8c387c1.zip op-kernel-dev-1eae4eb2a1c784bf35ee4f8f158cd21cf8c387c1.tar.gz |
e1000e: Disable L1 ASPM power savings for 82573 mobile variants
L1 ASPM link (pci-e link power savings) has significant benefits
(~1W savings when link is active) but unfortunately does not work
correctly on any of the chipsets that have 82573 on mobile platforms
which causes various nuisances:
- eeprom reads return garbage information leading to bad eeprom
checksums
- long ping times (up to 2 seconds)
- complete system hangs (freeze/lockup)
A lot of T60 owners have been plagued by this, but other mobile
solutions also suffer from these symptoms.
Disabling L1 ASPM before we activate the PCI-E link fixes all of
these issues at the cost of some power consumption.
Remove a workaround RDTR adjustment that is no longer needed with
this new one.
Signed-off-by: Auke Kok <auke-jan.h.kok@intel.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net/e1000e/e1000.h')
-rw-r--r-- | drivers/net/e1000e/e1000.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/e1000e/e1000.h b/drivers/net/e1000e/e1000.h index 473f78d..8b88c22 100644 --- a/drivers/net/e1000e/e1000.h +++ b/drivers/net/e1000e/e1000.h @@ -288,7 +288,6 @@ struct e1000_info { #define FLAG_HAS_CTRLEXT_ON_LOAD (1 << 5) #define FLAG_HAS_SWSM_ON_LOAD (1 << 6) #define FLAG_HAS_JUMBO_FRAMES (1 << 7) -#define FLAG_HAS_ASPM (1 << 8) #define FLAG_HAS_STATS_ICR_ICT (1 << 9) #define FLAG_HAS_STATS_PTC_PRC (1 << 10) #define FLAG_HAS_SMART_POWER_DOWN (1 << 11) |