diff options
| author |   Tejun Heo <htejun@gmail.com> | 2007-02-02 15:29:27 +0900 |
|---|---|---|
| committer |   Jeff Garzik <jeff@garzik.org> | 2007-02-09 17:39:38 -0500 |
| commit | 34fee227dd13af593be599b19683464ac4dd4c8b (patch) | |
| tree | 2c246bdc4e573fef2df2c67b83d6b3f43c63503e | |
| parent | libata: rearrange dmesg info to add full ATA revision (diff) | |
| download | linux-dev-34fee227dd13af593be599b19683464ac4dd4c8b.tar.xz linux-dev-34fee227dd13af593be599b19683464ac4dd4c8b.zip | |
libata: add 150ms between completion of hardreset and status checking
Follow the old SRST rule and delay 150ms between completion of
hardreset and status checking. Debouncing delay should usually cover
this but debounce duration could be shorter than 150ms under certain
circumstances.
Usefulness depends on host controller implementation but it can't hurt
and serves as a reminder that 2s delay for GoVault should also be
added here.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
| -rw-r--r-- | drivers/ata/libata-core.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c index 5c2e581bf990..6a2083a6d175 100644 --- a/drivers/ata/libata-core.c +++ b/drivers/ata/libata-core.c @@ -3023,6 +3023,9 @@ int sata_std_hardreset(struct ata_port *ap, unsigned int *class) return 0; } + /* wait a while before checking status, see SRST for more info */ + msleep(150); + if (ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT)) { ata_port_printk(ap, KERN_ERR, "COMRESET failed (device not ready)\n"); |