[PATCH] libata: cosmetic changes in ata_bus_softreset()
authorTejun Heo <htejun@gmail.com>
Fri, 24 Mar 2006 17:58:13 +0000 (02:58 +0900)
committerJeff Garzik <jeff@garzik.org>
Sat, 25 Mar 2006 04:04:29 +0000 (23:04 -0500)
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>
drivers/scsi/libata-core.c

index 0aff888d9ecdd42b9756f17fb983336c946de86d..64f71df19ed19d78b614227cb5e430ee3828035a 100644 (file)
@@ -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);