diff options
| author | Tejun Heo <htejun@gmail.com> | 2006-04-11 22:16:45 +0900 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-04-11 13:12:46 -0400 |
| commit | 987d2f05b396760517eef7cba66b2f415ac484f5 (patch) | |
| tree | 72d7d4e31f5aa8df3c7988119fa0787632d01460 /drivers/scsi/libata-core.c | |
| parent | 2bf2cb26b2512c6a609bb152982c388329bedff6 (diff) | |
| download | lwn-987d2f05b396760517eef7cba66b2f415ac484f5.tar.gz lwn-987d2f05b396760517eef7cba66b2f415ac484f5.zip | |
[PATCH] libata: make reset methods complain when they fail
Make reset methods complain loud when they fail.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/scsi/libata-core.c')
| -rw-r--r-- | drivers/scsi/libata-core.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c index 92b5077ac052..2d76ce23728f 100644 --- a/drivers/scsi/libata-core.c +++ b/drivers/scsi/libata-core.c @@ -2232,8 +2232,10 @@ static unsigned int ata_bus_softreset(struct ata_port *ap, * the bus shows 0xFF because the odd clown forgets the D7 * pulldown resistor. */ - if (ata_check_status(ap) == 0xFF) + if (ata_check_status(ap) == 0xFF) { + printk(KERN_ERR "ata%u: SRST failed (status 0xFF)\n", ap->id); return AC_ERR_OTHER; + } ata_bus_post_reset(ap, devmask); @@ -2494,8 +2496,8 @@ int sata_std_hardreset(struct ata_port *ap, unsigned int *class) } if (ata_busy_sleep(ap, ATA_TMOUT_BOOT_QUICK, ATA_TMOUT_BOOT)) { - printk(KERN_ERR "ata%u: COMRESET failed " - "(device not ready)\n", ap->id); + printk(KERN_ERR + "ata%u: COMRESET failed (device not ready)\n", ap->id); return -EIO; } |