linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* VIA VT6420 SATA suspend/resume problem in 2.6.20?
@ 2007-05-03 12:42 Jeroen Janssen
  2007-05-03 14:45 ` Tejun Heo
  0 siblings, 1 reply; 10+ messages in thread
From: Jeroen Janssen @ 2007-05-03 12:42 UTC (permalink / raw)
  To: linux-ide

Hello,

I have a board with a VIA 6420 chipset and a SATA harddisk running on
2.6.20 (Fedora Core 6).

When I suspend the system, it goes to sleep instantly, but when I
wakeup the system, it takes about 2 x 30 seconds for the SATA disk to
be accessable again.

I searched the mailinglist and could not really find if this is a
known problem (although I noticed that there's some work going on in
the acpi power management).

Does anyone if this is a known problem? (and if so, is there a
solution available?)
How can I pinpoint the exact problem?

If you need any specific logging, just let me know and I'll provide them.

Best regards,

Jeroen Janssen

^ permalink raw reply	[flat|nested] 10+ messages in thread
* [PATCH] sata_via: add missing PM hooks
@ 2007-05-04 13:30 Tejun Heo
  2007-05-08  9:54 ` Jeroen Janssen
  2007-05-10  0:17 ` Jeff Garzik
  0 siblings, 2 replies; 10+ messages in thread
From: Tejun Heo @ 2007-05-04 13:30 UTC (permalink / raw)
  To: Jeff Garzik, IDE/ATA development list, stable; +Cc: Jeroen Janssen

For some reason, sata_via is missing PM hooks.  Add them.  Spotted by
Jeroen Janssen <jeroen.janssen@gmail.com>.

Signed-off-by: Tejun Heo <htejun@gmail.com>
Cc: Jeroen Janssen <jeroen.janssen@gmail.com>
---
I have no idea how we missed this.  This is for both libata-dev &&
-stable.  Thanks.

Index: tree0/drivers/ata/sata_via.c
===================================================================
--- tree0.orig/drivers/ata/sata_via.c
+++ tree0/drivers/ata/sata_via.c
@@ -97,6 +97,10 @@ static struct pci_driver svia_pci_driver
 	.name			= DRV_NAME,
 	.id_table		= svia_pci_tbl,
 	.probe			= svia_init_one,
+#ifdef CONFIG_PM
+	.suspend		= ata_pci_device_suspend,
+	.resume			= ata_pci_device_resume,
+#endif
 	.remove			= ata_pci_remove_one,
 };
 
@@ -116,6 +120,10 @@ static struct scsi_host_template svia_sh
 	.slave_configure	= ata_scsi_slave_config,
 	.slave_destroy		= ata_scsi_slave_destroy,
 	.bios_param		= ata_std_bios_param,
+#ifdef CONFIG_PM
+	.suspend		= ata_scsi_device_suspend,
+	.resume			= ata_scsi_device_resume,
+#endif
 };
 
 static const struct ata_port_operations vt6420_sata_ops = {

^ permalink raw reply	[flat|nested] 10+ messages in thread
end of thread, other threads:[~2007-05-10  0:17 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-05-03 12:42 VIA VT6420 SATA suspend/resume problem in 2.6.20? Jeroen Janssen
2007-05-03 14:45 ` Tejun Heo
2007-05-04  8:22   ` Jeroen Janssen
2007-05-04  9:03     ` [PATCH] sata_via: add missing PM hooks Tejun Heo
2007-05-04 13:21       ` Jeroen Janssen
2007-05-04 13:25         ` Tejun Heo
  -- strict thread matches above, loose matches on Subject: below --
2007-05-04 13:30 Tejun Heo
2007-05-08  9:54 ` Jeroen Janssen
2007-05-08 10:27   ` Tejun Heo
2007-05-10  0:17 ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).