]> pilppa.com Git - linux-2.6-omap-h63xx.git/commit
[PATCH] ahci: do not powerdown during initialization
authorTejun Heo <htejun@gmail.com>
Mon, 20 Nov 2006 06:42:36 +0000 (15:42 +0900)
committerTejun Heo <htejun@gmail.com>
Sun, 3 Dec 2006 08:56:29 +0000 (17:56 +0900)
commit8e16f941226f15622fbbc416a1f3d8705001a191
tree97bf87ebff9e3e69caa0a0ebdd9ee9291c5dfac5
parent70e6ad0c6d1e6cb9ee3c036a85ca2561eb1fd766
[PATCH] ahci: do not powerdown during initialization

ahci_init_controller() calls ahci_deinit_port() to make sure the
controller is stopped before initializing the controller.  In turn,
ahci_deinit_port() invokes ahci_power_down() to power down the port.
If the controller supports slumber mode, the link is put into it.

Unfortunately, some devices don't implement link powersaving mode
properly and show erratic behavior after link is put into slumber
mode.  For example, HL-DT-ST DVD-RAM GSA-H30N completely locks up on
slumber transition and can only be recovered with the *REAL* hard
reset - power removal and reapply.

Note that this makes the first probing reset different from all
others.  If the above dvd-ram is hotplugged after ahci is initialized,
no problem occurs because ahci is already fully initialized with phy
powered up.  So, this might also be the reason for other weird AHCI
initial probing abnormalities.

This patch moves power up/down out of port init/deinit and call them
only when needed.

Power down is now called only when suspending.  As system suspend
usually involves powering down 12v for storage devices, this shouldn't
cause problem even if the attached device doesn't support slumber
mode.  However, in partial power management and suspend failure cases,
devices might lock up after suspend attempt.  I thought about removing
transition to slumber mode altogether but ahci spec mandates it before
HBA D3 state transition.  Blacklisting such devices might be the
solution.

Signed-off-by: Tejun Heo <htejun@gmail.com>
drivers/ata/ahci.c