e1000e: Disable L1 ASPM power savings for 82573 mobile variants
authorAuke Kok <auke-jan.h.kok@intel.com>
Wed, 31 Oct 2007 22:22:00 +0000 (15:22 -0700)
committerDavid S. Miller <davem@davemloft.net>
Mon, 28 Jan 2008 23:03:43 +0000 (15:03 -0800)
commit1eae4eb2a1c784bf35ee4f8f158cd21cf8c387c1
tree4e9de1ad1d24b201647376a21862a110e462c831
parent93ca161027eb6a1761fb674ad7b995aedccf5f6e
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>
drivers/net/e1000e/82571.c
drivers/net/e1000e/e1000.h
drivers/net/e1000e/netdev.c
drivers/net/e1000e/param.c