Re: [PATCH][RESUBMIT] drivers/ide/ide-probe.c, kernel 2.6.23.1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jonas Stare wrote:

diff -u linux-2.6.23.1-orig/drivers/ide/ide-probe.c linux-2.6.23.1/drivers/ide/ide-probe.c --- linux-2.6.23.1-orig/drivers/ide/ide-probe.c 2007-10-12 18:43:44.000000000 +0200
+++ linux-2.6.23.1/drivers/ide/ide-probe.c      2007-11-16 10:26:23.000000000 +0100
@@ -643,6 +643,7 @@
 static int wait_hwif_ready(ide_hwif_t *hwif)
 {
        int rc;
+       int unit;
printk(KERN_DEBUG "Probing IDE interface %s...\n", hwif->name); @@ -659,16 +660,25 @@
                return rc;
/* Now make sure both master & slave are ready */
-       SELECT_DRIVE(&hwif->drives[0]);
-       hwif->OUTB(8, hwif->io_ports[IDE_CONTROL_OFFSET]);
-       mdelay(2);
-       rc = ide_wait_not_busy(hwif, 35000);
-       if (rc)
-               return rc;
-       SELECT_DRIVE(&hwif->drives[1]);
-       hwif->OUTB(8, hwif->io_ports[IDE_CONTROL_OFFSET]);
-       mdelay(2);
-       rc = ide_wait_not_busy(hwif, 35000);
+       for (unit = 0; unit < MAX_DRIVES; ++unit) {
+               /* Ignore disks that we will not probe for later. */
+               if (!hwif->drives[unit].noprobe ||
+                   hwif->drives[unit].present) {

   Why not:

		if (hwif->drives[unit].noprobe || !hwif->drives[unit].present) {
                	printk(KERN_DEBUG "Skip ide_wait_not_busy for %s:%d\n",
                               hwif->name, unit);
			continue;
		}
                SELECT_DRIVE(&hwif->drives[unit]);
                hwif->OUTB(8, hwif->io_ports[IDE_CONTROL_OFFSET]);
                mdelay(2);
                rc = ide_wait_not_busy(hwif, 35000);
                if (rc) {
			/* Exit function with master reselected */
                        if (unit != 0)
                        	SELECT_DRIVE(&hwif->drives[0]);
                        return rc;
        	}
	}
/* Exit function with master reselected (let's be sane) */
        SELECT_DRIVE(&hwif->drives[0]);

MBR, Sergei
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux