ahci: don't enter slumber on power down
authorTejun Heo <htejun@gmail.com>
Sat, 20 Jan 2007 17:10:11 +0000 (02:10 +0900)
committerJeff Garzik <jeff@garzik.org>
Wed, 24 Jan 2007 07:03:45 +0000 (02:03 -0500)
commit07c53dac4904206a50dd7c87adabbb1acff903fb
tree09fb7d1b53b4e6b5aabf05139ca8fff866e7e8d0
parentf740d1689d91415cfc749d17138a11ed03b7d38b
ahci: don't enter slumber on power down

Some ATA/ATAPI devices act weirdly after the link is put into slumber
mode.  Some hang completely requiring physical power removal while
others fail to wake up till the link is hardreset a couple of times.

The addition of slumber on power down was never driven by real need.
It just followed what ahci spec said literally.  The spec itself seems
faulty in that it doesn't consider devices (not controllers) which
don't support link powersaving mode.

Theory never matches reality when it comes to dark allys of cheap
ATA/ATAPI world.  It's just unrealistic to expect vendors to test
rarely used link powersaving feature rigorously.  This patch makes
ahci more friendly to the coldness of reality.

This shouldn't have any negative effect - when suspend operation
succeeds, we power off the whole machine; otherwise, we wake up
everything.  I can't see any reason to be so elaborate with powering
down the link in the first place.

Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/ata/ahci.c