[PATCH] ahci: imx: manage only sata_ref_clk in imx_sata_enable[disable]
From: festevam@gmail.com (Fabio Estevam)
Date: 2014-05-28 15:59:43
Also in:
linux-ide
Hans, On Wed, May 28, 2014 at 12:36 PM, Hans de Goede [off-list ref] wrote:
The proper fix is obvious, if we've shutdown everything due to there not being a device present at the initial probe, don't call ahci_platform_resume_host on resume, as is done in the attached patch.
I tested your attached patch. It does not build as it lacks to add the definition of imxpriv inside the suspend/resume functions. After fixing it I get: root at freescale /$ echo mem > /sys/power/state PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.002 seconds) done. Freezing remaining freezable tasks ... (elapsed 0.002 seconds) done. PM: suspend of devices complete after 91.109 msecs PM: suspend devices took 0.100 seconds PM: late suspend of devices complete after 9.086 msecs PM: noirq suspend of devices complete after 8.780 msecs Disabling non-boot CPUs ... CPU1: shutdown CPU2: shutdown CPU3: shutdown Enabling non-boot CPUs ... CPU1: Booted secondary processor CPU1 is up CPU2: Booted secondary processor CPU2 is up CPU3: Booted secondary processor CPU3 is up PM: noirq resume of devices complete after 6.914 msecs PM: early resume of devices complete after 5.160 msecs PM: resume of devices complete after 167.025 msecs PM: resume devices took 0.180 seconds Restarting tasks ... done. ata1: failed to resume link (SControl 2003) random: nonblocking pool is initialized root at freescale /$ ata1: softreset failed (1st FIS failed) ata1: failed to resume link (SControl 2003) ata1: softreset failed (1st FIS failed) ata1: failed to resume link (SControl 2003) ata1: softreset failed (1st FIS failed) ata1: limiting SATA link speed to 1.5 Gbps ata1: failed to resume link (SControl 2003) ata1: softreset failed (1st FIS failed) ata1: reset failed, giving up With Shawn's version I do not get the suspend error messages: ... CPU1: Booted secondary processor CPU1 is up CPU2: Booted secondary processor CPU2 is up CPU3: Booted secondary processor CPU3 is up PM: noirq resume of devices complete after 5.836 msecs PM: early resume of devices complete after 5.334 msecs PM: resume of devices complete after 176.763 msecs PM: resume devices took 0.190 seconds Restarting tasks ... done. ata1: SATA link down (SStatus 0 SControl 300) random: nonblocking pool is initialized