| Message ID | 1400330761-2432-1-git-send-email-shawn.guo@freescale.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Sat, May 17, 2014 at 9:46 AM, Shawn Guo <shawn.guo@freescale.com> wrote: > The commit e783c51 (ahci: imx: software workaround for phy reset issue > in resume) calls imx_sata_phy_reset() to reset phy immediately after > SATA MPLL is enabled. It seems working fine mostly, but fails in some > case as below. > > ... > ahci-imx 2200000.sata: failed to reset phy: -110 > ahci-imx: probe of 2200000.sata failed with error -110 > > After talking to the designer, we learnt that when enabling i.MX6Q SATA > MPLL, we need to wait 100us for it to settle down for safety. Add this > required delay to fix above failure. > > Signed-off-by: Shawn Guo <shawn.guo@freescale.com> On my sabresd board that was not connected to any sata disk I was always getting the 'failed to reset phy' error. With this patch the error goes away and the driver probes successfully. Tested-by: Fabio Estevam <fabio.estevam@freescale.com>
On Sat, May 17, 2014 at 10:58:20AM -0300, Fabio Estevam wrote: > On Sat, May 17, 2014 at 9:46 AM, Shawn Guo <shawn.guo@freescale.com> wrote: > > The commit e783c51 (ahci: imx: software workaround for phy reset issue > > in resume) calls imx_sata_phy_reset() to reset phy immediately after > > SATA MPLL is enabled. It seems working fine mostly, but fails in some > > case as below. > > > > ... > > ahci-imx 2200000.sata: failed to reset phy: -110 > > ahci-imx: probe of 2200000.sata failed with error -110 > > > > After talking to the designer, we learnt that when enabling i.MX6Q SATA > > MPLL, we need to wait 100us for it to settle down for safety. Add this > > required delay to fix above failure. > > > > Signed-off-by: Shawn Guo <shawn.guo@freescale.com> > > On my sabresd board that was not connected to any sata disk I was > always getting the 'failed to reset phy' error. Damn. I never noticed that the error is even there when no SATA disk is attached. Sorry. > > With this patch the error goes away and the driver probes successfully. > > Tested-by: Fabio Estevam <fabio.estevam@freescale.com> Thanks. Shawn
On Sat, May 17, 2014 at 08:46:01PM +0800, Shawn Guo wrote: > The commit e783c51 (ahci: imx: software workaround for phy reset issue > in resume) calls imx_sata_phy_reset() to reset phy immediately after > SATA MPLL is enabled. It seems working fine mostly, but fails in some > case as below. > > ... > ahci-imx 2200000.sata: failed to reset phy: -110 > ahci-imx: probe of 2200000.sata failed with error -110 > > After talking to the designer, we learnt that when enabling i.MX6Q SATA > MPLL, we need to wait 100us for it to settle down for safety. Add this > required delay to fix above failure. > > Signed-off-by: Shawn Guo <shawn.guo@freescale.com> Applied to libata/for-3.15-fixes. Thanks.
Shawn, On Sat, May 17, 2014 at 9:46 AM, Shawn Guo <shawn.guo@freescale.com> wrote: > The commit e783c51 (ahci: imx: software workaround for phy reset issue > in resume) calls imx_sata_phy_reset() to reset phy immediately after > SATA MPLL is enabled. It seems working fine mostly, but fails in some > case as below. > > ... > ahci-imx 2200000.sata: failed to reset phy: -110 > ahci-imx: probe of 2200000.sata failed with error -110 > > After talking to the designer, we learnt that when enabling i.MX6Q SATA > MPLL, we need to wait 100us for it to settle down for safety. Add this > required delay to fix above failure. Do you know if other tweaking like this would be necessary to fix suspend/resume of the ahci-imx driver? I see the following issue on mx53 and mx6q: root@freescale /home$ echo mem > /sys/power/state PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.001 seconds) done. Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done. Suspending console(s) (use no_console_suspend to debug) PM: suspend of devices complete after 25.816 msecs PM: suspend devices took 0.030 seconds PM: late suspend of devices complete after 4.361 msecs PM: noirq suspend of devices complete after 4.658 msecs Disabling non-boot CPUs ... PM: noirq resume of devices complete after 2.767 msecs PM: early resume of devices complete after 3.438 msecs ------------[ cut here ]------------ WARNING: CPU: 0 PID: 661 at drivers/ata/libahci.c:224 ahci_enable_ahci+0x74/0x8) Modules linked in: CPU: 0 PID: 661 Comm: sh Tainted: G W 3.15.0-rc5-next-20140521-000027 Backtrace: [<80011c90>] (dump_backtrace) from [<80011e2c>] (show_stack+0x18/0x1c) r6:803a22f4 r5:00000000 r4:00000000 r3:00000000 [<80011e14>] (show_stack) from [<80661e60>] (dump_stack+0x88/0xa4) [<80661dd8>] (dump_stack) from [<80028fdc>] (warn_slowpath_common+0x70/0x94) r5:00000009 r4:00000000 [<80028f6c>] (warn_slowpath_common) from [<80029024>] (warn_slowpath_null+0x24/) r8:808f68c4 r7:00000000 r6:00000000 r5:00000000 r4:e0810004 [<80029000>] (warn_slowpath_null) from [<803a22f4>] (ahci_enable_ahci+0x74/0x80) [<803a2280>] (ahci_enable_ahci) from [<803a2324>] (ahci_reset_controller+0x24/0) r8:ddcd9410 r7:80351178 r6:ddcd9444 r5:dde8b850 r4:e0810000 r3:ddf35e90 [<803a2300>] (ahci_reset_controller) from [<803a2c68>] (ahci_platform_resume_ho) r7:80351178 r6:ddcd9444 r5:dde8b850 r4:ddcd9410 [<803a2c30>] (ahci_platform_resume_host) from [<803a38f0>] (imx_ahci_resume+0x2) r5:00000000 r4:ddcd9410 [<803a38c4>] (imx_ahci_resume) from [<803511ac>] (platform_pm_resume+0x34/0x54) r4:00000000 r3:803a38c4 [<80351178>] (platform_pm_resume) from [<803589fc>] (dpm_run_callback.isra.5+0x) [<803589c8>] (dpm_run_callback.isra.5) from [<80359238>] (device_resume+0xb8/0x) r8:00000000 r7:00000010 r6:ddcd9444 r5:00000001 r4:ddcd9410 [<80359180>] (device_resume) from [<8035a28c>] (dpm_resume+0xf4/0x220) r8:00000010 r7:8115884c r6:80921d7c r5:80921d24 r4:ddcd9410 r3:dd4dce00 [<8035a198>] (dpm_resume) from [<8035a574>] (dpm_resume_end+0x14/0x20) r10:00000000 r9:ddee248c r8:808f761c r7:81102f80 r6:00000000 r5:00000003 r4:00000010 [<8035a560>] (dpm_resume_end) from [<8006b3e8>] (suspend_devices_and_enter+0x14) r4:81102f80 r3:81102f8c [<8006b2a0>] (suspend_devices_and_enter) from [<8006b868>] (pm_suspend+0x218/0x) r10:00000004 r8:dd4a6e80 r7:00000003 r6:807d643c r5:00000003 r4:00000000 [<8006b650>] (pm_suspend) from [<8006a36c>] (state_store+0x70/0xc0) r6:8066c430 r5:807d643c r4:00000003 r3:0000006d [<8006a2fc>] (state_store) from [<8029b27c>] (kobj_attr_store+0x1c/0x28) r10:ddee2480 r8:00000004 r7:dd4a5f78 r6:dd4a6e80 r5:00000004 r4:ddee2480 [<8029b260>] (kobj_attr_store) from [<801459cc>] (sysfs_kf_write+0x54/0x58) [<80145978>] (sysfs_kf_write) from [<80144ddc>] (kernfs_fop_write+0xc8/0x188) r6:dd4a6e80 r5:00000000 r4:00000000 r3:80145978 [<80144d14>] (kernfs_fop_write) from [<800e3450>] (vfs_write+0xa8/0x1b0) r10:00fabdf8 r9:dd4a4000 r8:00000004 r7:dd4a5f78 r6:00fabdf8 r5:00000004 r4:dd581a00 [<800e33a8>] (vfs_write) from [<800e3874>] (SyS_write+0x44/0x90) r10:00fabdf8 r8:00000004 r7:dd581a00 r6:dd581a00 r5:00000000 r4:00000000 [<800e3830>] (SyS_write) from [<8000eae0>] (ret_fast_syscall+0x0/0x48) r10:00000000 r8:8000eca4 r7:00000004 r6:76f6f4d0 r5:00fabdf8 r4:00000004 ---[ end trace f936a22e464842c1 ]--- ------------[ cut here ]------------ WARNING: CPU: 0 PID: 661 at drivers/ata/libahci.c:224 ahci_enable_ahci+0x74/0x8) Modules linked in: CPU: 0 PID: 661 Comm: sh Tainted: G W 3.15.0-rc5-next-20140521-000027 Backtrace: [<80011c90>] (dump_backtrace) from [<80011e2c>] (show_stack+0x18/0x1c) r6:803a22f4 r5:00000000 r4:00000000 r3:00000000 [<80011e14>] (show_stack) from [<80661e60>] (dump_stack+0x88/0xa4) [<80661dd8>] (dump_stack) from [<80028fdc>] (warn_slowpath_common+0x70/0x94) r5:00000009 r4:00000000 [<80028f6c>] (warn_slowpath_common) from [<80029024>] (warn_slowpath_null+0x24/) r8:808f68c4 r7:00000000 r6:00000000 r5:00000000 r4:e0810004 [<80029000>] (warn_slowpath_null) from [<803a22f4>] (ahci_enable_ahci+0x74/0x80) [<803a2280>] (ahci_enable_ahci) from [<803a23a8>] (ahci_reset_controller+0xa8/0) r8:ddcd9410 r7:e0810004 r6:00000000 r5:dde8b850 r4:e0810000 r3:00000001 [<803a2300>] (ahci_reset_controller) from [<803a2c68>] (ahci_platform_resume_ho) r7:80351178 r6:ddcd9444 r5:dde8b850 r4:ddcd9410 [<803a2c30>] (ahci_platform_resume_host) from [<803a38f0>] (imx_ahci_resume+0x2) r5:00000000 r4:ddcd9410 [<803a38c4>] (imx_ahci_resume) from [<803511ac>] (platform_pm_resume+0x34/0x54) r4:00000000 r3:803a38c4 [<80351178>] (platform_pm_resume) from [<803589fc>] (dpm_run_callback.isra.5+0x) [<803589c8>] (dpm_run_callback.isra.5) from [<80359238>] (device_resume+0xb8/0x) r8:00000000 r7:00000010 r6:ddcd9444 r5:00000001 r4:ddcd9410 [<80359180>] (device_resume) from [<8035a28c>] (dpm_resume+0xf4/0x220) r8:00000010 r7:8115884c r6:80921d7c r5:80921d24 r4:ddcd9410 r3:dd4dce00 [<8035a198>] (dpm_resume) from [<8035a574>] (dpm_resume_end+0x14/0x20) r10:00000000 r9:ddee248c r8:808f761c r7:81102f80 r6:00000000 r5:00000003 r4:00000010 [<8035a560>] (dpm_resume_end) from [<8006b3e8>] (suspend_devices_and_enter+0x14) r4:81102f80 r3:81102f8c [<8006b2a0>] (suspend_devices_and_enter) from [<8006b868>] (pm_suspend+0x218/0x) r10:00000004 r8:dd4a6e80 r7:00000003 r6:807d643c r5:00000003 r4:00000000 [<8006b650>] (pm_suspend) from [<8006a36c>] (state_store+0x70/0xc0) r6:8066c430 r5:807d643c r4:00000003 r3:0000006d [<8006a2fc>] (state_store) from [<8029b27c>] (kobj_attr_store+0x1c/0x28) r10:ddee2480 r8:00000004 r7:dd4a5f78 r6:dd4a6e80 r5:00000004 r4:ddee2480 [<8029b260>] (kobj_attr_store) from [<801459cc>] (sysfs_kf_write+0x54/0x58) [<80145978>] (sysfs_kf_write) from [<80144ddc>] (kernfs_fop_write+0xc8/0x188) r6:dd4a6e80 r5:00000000 r4:00000000 r3:80145978 [<80144d14>] (kernfs_fop_write) from [<800e3450>] (vfs_write+0xa8/0x1b0) r10:00fabdf8 r9:dd4a4000 r8:00000004 r7:dd4a5f78 r6:00fabdf8 r5:00000004 r4:dd581a00 [<800e33a8>] (vfs_write) from [<800e3874>] (SyS_write+0x44/0x90) r10:00fabdf8 r8:00000004 r7:dd581a00 r6:dd581a00 r5:00000000 r4:00000000 [<800e3830>] (SyS_write) from [<8000eae0>] (ret_fast_syscall+0x0/0x48) r10:00000000 r8:8000eca4 r7:00000004 r6:76f6f4d0 r5:00fabdf8 r4:00000004 ---[ end trace f936a22e464842c2 ]--- PM: resume of devices complete after 260.366 msecs PM: resume devices took 0.260 seconds Restarting tasks ... done. ata1: failed to resume link (SControl 0) ata1: SATA link down (SStatus 0 SControl 0)
On Sat, May 24, 2014 at 04:35:42PM -0300, Fabio Estevam wrote: > Shawn, > > On Sat, May 17, 2014 at 9:46 AM, Shawn Guo <shawn.guo@freescale.com> wrote: > > The commit e783c51 (ahci: imx: software workaround for phy reset issue > > in resume) calls imx_sata_phy_reset() to reset phy immediately after > > SATA MPLL is enabled. It seems working fine mostly, but fails in some > > case as below. > > > > ... > > ahci-imx 2200000.sata: failed to reset phy: -110 > > ahci-imx: probe of 2200000.sata failed with error -110 > > > > After talking to the designer, we learnt that when enabling i.MX6Q SATA > > MPLL, we need to wait 100us for it to settle down for safety. Add this > > required delay to fix above failure. > > Do you know if other tweaking like this would be necessary to fix > suspend/resume of the ahci-imx driver? > > I see the following issue on mx53 and mx6q: Hmm, I haven't seen this? How do I reproduce it on imx6q? Shawn > > root@freescale /home$ echo mem > /sys/power/state > PM: Syncing filesystems ... done. > Freezing user space processes ... (elapsed 0.001 seconds) done. > Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done. > Suspending console(s) (use no_console_suspend to debug) > PM: suspend of devices complete after 25.816 msecs > PM: suspend devices took 0.030 seconds > PM: late suspend of devices complete after 4.361 msecs > PM: noirq suspend of devices complete after 4.658 msecs > Disabling non-boot CPUs ... > PM: noirq resume of devices complete after 2.767 msecs > PM: early resume of devices complete after 3.438 msecs > ------------[ cut here ]------------ > WARNING: CPU: 0 PID: 661 at drivers/ata/libahci.c:224 ahci_enable_ahci+0x74/0x8) > Modules linked in: > CPU: 0 PID: 661 Comm: sh Tainted: G W 3.15.0-rc5-next-20140521-000027 > Backtrace: > [<80011c90>] (dump_backtrace) from [<80011e2c>] (show_stack+0x18/0x1c) > r6:803a22f4 r5:00000000 r4:00000000 r3:00000000 > [<80011e14>] (show_stack) from [<80661e60>] (dump_stack+0x88/0xa4) > [<80661dd8>] (dump_stack) from [<80028fdc>] (warn_slowpath_common+0x70/0x94) > r5:00000009 r4:00000000 > [<80028f6c>] (warn_slowpath_common) from [<80029024>] (warn_slowpath_null+0x24/) > r8:808f68c4 r7:00000000 r6:00000000 r5:00000000 r4:e0810004 > [<80029000>] (warn_slowpath_null) from [<803a22f4>] (ahci_enable_ahci+0x74/0x80) > [<803a2280>] (ahci_enable_ahci) from [<803a2324>] (ahci_reset_controller+0x24/0) > r8:ddcd9410 r7:80351178 r6:ddcd9444 r5:dde8b850 r4:e0810000 r3:ddf35e90 > [<803a2300>] (ahci_reset_controller) from [<803a2c68>] (ahci_platform_resume_ho) > r7:80351178 r6:ddcd9444 r5:dde8b850 r4:ddcd9410 > [<803a2c30>] (ahci_platform_resume_host) from [<803a38f0>] (imx_ahci_resume+0x2) > r5:00000000 r4:ddcd9410 > [<803a38c4>] (imx_ahci_resume) from [<803511ac>] (platform_pm_resume+0x34/0x54) > r4:00000000 r3:803a38c4 > [<80351178>] (platform_pm_resume) from [<803589fc>] (dpm_run_callback.isra.5+0x) > [<803589c8>] (dpm_run_callback.isra.5) from [<80359238>] (device_resume+0xb8/0x) > r8:00000000 r7:00000010 r6:ddcd9444 r5:00000001 r4:ddcd9410 > [<80359180>] (device_resume) from [<8035a28c>] (dpm_resume+0xf4/0x220) > r8:00000010 r7:8115884c r6:80921d7c r5:80921d24 r4:ddcd9410 r3:dd4dce00 > [<8035a198>] (dpm_resume) from [<8035a574>] (dpm_resume_end+0x14/0x20) > r10:00000000 r9:ddee248c r8:808f761c r7:81102f80 r6:00000000 r5:00000003 > r4:00000010 > [<8035a560>] (dpm_resume_end) from [<8006b3e8>] (suspend_devices_and_enter+0x14) > r4:81102f80 r3:81102f8c > [<8006b2a0>] (suspend_devices_and_enter) from [<8006b868>] (pm_suspend+0x218/0x) > r10:00000004 r8:dd4a6e80 r7:00000003 r6:807d643c r5:00000003 r4:00000000 > [<8006b650>] (pm_suspend) from [<8006a36c>] (state_store+0x70/0xc0) > r6:8066c430 r5:807d643c r4:00000003 r3:0000006d > [<8006a2fc>] (state_store) from [<8029b27c>] (kobj_attr_store+0x1c/0x28) > r10:ddee2480 r8:00000004 r7:dd4a5f78 r6:dd4a6e80 r5:00000004 r4:ddee2480 > [<8029b260>] (kobj_attr_store) from [<801459cc>] (sysfs_kf_write+0x54/0x58) > [<80145978>] (sysfs_kf_write) from [<80144ddc>] (kernfs_fop_write+0xc8/0x188) > r6:dd4a6e80 r5:00000000 r4:00000000 r3:80145978 > [<80144d14>] (kernfs_fop_write) from [<800e3450>] (vfs_write+0xa8/0x1b0) > r10:00fabdf8 r9:dd4a4000 r8:00000004 r7:dd4a5f78 r6:00fabdf8 r5:00000004 > r4:dd581a00 > [<800e33a8>] (vfs_write) from [<800e3874>] (SyS_write+0x44/0x90) > r10:00fabdf8 r8:00000004 r7:dd581a00 r6:dd581a00 r5:00000000 r4:00000000 > [<800e3830>] (SyS_write) from [<8000eae0>] (ret_fast_syscall+0x0/0x48) > r10:00000000 r8:8000eca4 r7:00000004 r6:76f6f4d0 r5:00fabdf8 r4:00000004 > ---[ end trace f936a22e464842c1 ]--- > ------------[ cut here ]------------ > WARNING: CPU: 0 PID: 661 at drivers/ata/libahci.c:224 ahci_enable_ahci+0x74/0x8) > Modules linked in: > CPU: 0 PID: 661 Comm: sh Tainted: G W 3.15.0-rc5-next-20140521-000027 > Backtrace: > [<80011c90>] (dump_backtrace) from [<80011e2c>] (show_stack+0x18/0x1c) > r6:803a22f4 r5:00000000 r4:00000000 r3:00000000 > [<80011e14>] (show_stack) from [<80661e60>] (dump_stack+0x88/0xa4) > [<80661dd8>] (dump_stack) from [<80028fdc>] (warn_slowpath_common+0x70/0x94) > r5:00000009 r4:00000000 > [<80028f6c>] (warn_slowpath_common) from [<80029024>] (warn_slowpath_null+0x24/) > r8:808f68c4 r7:00000000 r6:00000000 r5:00000000 r4:e0810004 > [<80029000>] (warn_slowpath_null) from [<803a22f4>] (ahci_enable_ahci+0x74/0x80) > [<803a2280>] (ahci_enable_ahci) from [<803a23a8>] (ahci_reset_controller+0xa8/0) > r8:ddcd9410 r7:e0810004 r6:00000000 r5:dde8b850 r4:e0810000 r3:00000001 > [<803a2300>] (ahci_reset_controller) from [<803a2c68>] (ahci_platform_resume_ho) > r7:80351178 r6:ddcd9444 r5:dde8b850 r4:ddcd9410 > [<803a2c30>] (ahci_platform_resume_host) from [<803a38f0>] (imx_ahci_resume+0x2) > r5:00000000 r4:ddcd9410 > [<803a38c4>] (imx_ahci_resume) from [<803511ac>] (platform_pm_resume+0x34/0x54) > r4:00000000 r3:803a38c4 > [<80351178>] (platform_pm_resume) from [<803589fc>] (dpm_run_callback.isra.5+0x) > [<803589c8>] (dpm_run_callback.isra.5) from [<80359238>] (device_resume+0xb8/0x) > r8:00000000 r7:00000010 r6:ddcd9444 r5:00000001 r4:ddcd9410 > [<80359180>] (device_resume) from [<8035a28c>] (dpm_resume+0xf4/0x220) > r8:00000010 r7:8115884c r6:80921d7c r5:80921d24 r4:ddcd9410 r3:dd4dce00 > [<8035a198>] (dpm_resume) from [<8035a574>] (dpm_resume_end+0x14/0x20) > r10:00000000 r9:ddee248c r8:808f761c r7:81102f80 r6:00000000 r5:00000003 > r4:00000010 > [<8035a560>] (dpm_resume_end) from [<8006b3e8>] (suspend_devices_and_enter+0x14) > r4:81102f80 r3:81102f8c > [<8006b2a0>] (suspend_devices_and_enter) from [<8006b868>] (pm_suspend+0x218/0x) > r10:00000004 r8:dd4a6e80 r7:00000003 r6:807d643c r5:00000003 r4:00000000 > [<8006b650>] (pm_suspend) from [<8006a36c>] (state_store+0x70/0xc0) > r6:8066c430 r5:807d643c r4:00000003 r3:0000006d > [<8006a2fc>] (state_store) from [<8029b27c>] (kobj_attr_store+0x1c/0x28) > r10:ddee2480 r8:00000004 r7:dd4a5f78 r6:dd4a6e80 r5:00000004 r4:ddee2480 > [<8029b260>] (kobj_attr_store) from [<801459cc>] (sysfs_kf_write+0x54/0x58) > [<80145978>] (sysfs_kf_write) from [<80144ddc>] (kernfs_fop_write+0xc8/0x188) > r6:dd4a6e80 r5:00000000 r4:00000000 r3:80145978 > [<80144d14>] (kernfs_fop_write) from [<800e3450>] (vfs_write+0xa8/0x1b0) > r10:00fabdf8 r9:dd4a4000 r8:00000004 r7:dd4a5f78 r6:00fabdf8 r5:00000004 > r4:dd581a00 > [<800e33a8>] (vfs_write) from [<800e3874>] (SyS_write+0x44/0x90) > r10:00fabdf8 r8:00000004 r7:dd581a00 r6:dd581a00 r5:00000000 r4:00000000 > [<800e3830>] (SyS_write) from [<8000eae0>] (ret_fast_syscall+0x0/0x48) > r10:00000000 r8:8000eca4 r7:00000004 r6:76f6f4d0 r5:00fabdf8 r4:00000004 > ---[ end trace f936a22e464842c2 ]--- > PM: resume of devices complete after 260.366 msecs > PM: resume devices took 0.260 seconds > Restarting tasks ... done. > ata1: failed to resume link (SControl 0) > ata1: SATA link down (SStatus 0 SControl 0)
On Sun, May 25, 2014 at 3:40 AM, Shawn Guo <shawn.guo@freescale.com> wrote:
> Hmm, I haven't seen this? How do I reproduce it on imx6q?
Run linux-next on a mx6qsabresd with no SATA disk connected and then:
$ echo enabled > /sys/class/tty/ttymxc0/power/wakeup
$ echo mem > /sys/power/state
Hit any key in console and then you will get the warning from ahci_imx suspend.
diff --git a/drivers/ata/ahci_imx.c b/drivers/ata/ahci_imx.c index 5824d99..8befeb6 100644 --- a/drivers/ata/ahci_imx.c +++ b/drivers/ata/ahci_imx.c @@ -258,6 +258,8 @@ static int imx_sata_enable(struct ahci_host_priv *hpriv) IMX6Q_GPR13_SATA_MPLL_CLK_EN, IMX6Q_GPR13_SATA_MPLL_CLK_EN); + usleep_range(100, 200); + ret = imx_sata_phy_reset(hpriv); if (ret) { dev_err(dev, "failed to reset phy: %d\n", ret);
The commit e783c51 (ahci: imx: software workaround for phy reset issue in resume) calls imx_sata_phy_reset() to reset phy immediately after SATA MPLL is enabled. It seems working fine mostly, but fails in some case as below. ... ahci-imx 2200000.sata: failed to reset phy: -110 ahci-imx: probe of 2200000.sata failed with error -110 After talking to the designer, we learnt that when enabling i.MX6Q SATA MPLL, we need to wait 100us for it to settle down for safety. Add this required delay to fix above failure. Signed-off-by: Shawn Guo <shawn.guo@freescale.com> --- Tejun, Sorry for that I did not catch it when I was working on the phy reset issue in the first place. I used to test the driver with a 32GiB and 64GiB SanDisk SSDs, and did not run into this issue. But it gets exposed on a 8GiB SSD I got to play recently. Shawn drivers/ata/ahci_imx.c | 2 ++ 1 file changed, 2 insertions(+)