aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorTejun Heo <htejun@gmail.com>2006-03-25 02:58:13 +0900
committerJeff Garzik <jeff@garzik.org>2006-03-24 23:04:29 -0500
commit298a41ca41b4c933adc7b5a56a0adb0e1df2e556 (patch)
treecb91595ee62afb244bc83b868095ef0c7339947e /drivers
parentaec5c3c1a929d7d79a420e943285cf3ba26a7c0d (diff)
downloadkernel_goldelico_gta04-298a41ca41b4c933adc7b5a56a0adb0e1df2e556.zip
kernel_goldelico_gta04-298a41ca41b4c933adc7b5a56a0adb0e1df2e556.tar.gz
kernel_goldelico_gta04-298a41ca41b4c933adc7b5a56a0adb0e1df2e556.tar.bz2
[PATCH] libata: cosmetic changes in ata_bus_softreset()
ata_bus_softreset() should return AC_ERR_* on failure not arbitrary positive number. While at it, reformat comment above it. Signed-off-by: Tejun Heo <htejun@gmail.com> Acked-by: Alan Cox <alan@redhat.com> Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/scsi/libata-core.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c
index 0aff888..64f71df 100644
--- a/drivers/scsi/libata-core.c
+++ b/drivers/scsi/libata-core.c
@@ -2008,13 +2008,12 @@ static unsigned int ata_bus_softreset(struct ata_port *ap,
*/
msleep(150);
-
/* Before we perform post reset processing we want to see if
- the bus shows 0xFF because the odd clown forgets the D7 pulldown
- resistor */
-
+ * the bus shows 0xFF because the odd clown forgets the D7
+ * pulldown resistor.
+ */
if (ata_check_status(ap) == 0xFF)
- return 1; /* Positive is failure for some reason */
+ return AC_ERR_OTHER;
ata_bus_post_reset(ap, devmask);