ide: make probe_hwif() return an error value
authorBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Fri, 1 Feb 2008 22:09:36 +0000 (23:09 +0100)
committerBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Fri, 1 Feb 2008 22:09:36 +0000 (23:09 +0100)
Make probe_hwif() return an error value.

While at it:

* Remove comment about MAX_DRIVES == 2 limitation (it is not special to
  probe_hwif(), it is a general assumption taken by a lot of IDE code).

* Rename probe_hwif() to ide_probe_port().

Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
drivers/ide/ide-iops.c
drivers/ide/ide-probe.c
drivers/ide/pci/ns87415.c

index 495e3878e42e8f870b24314c63ca192ee3616c2d..16b1f6e12781a7c53a1e5ea1cb3b8bc786b971df 100644 (file)
@@ -1166,7 +1166,7 @@ EXPORT_SYMBOL(ide_do_reset);
 
 /*
  * ide_wait_not_busy() waits for the currently selected device on the hwif
- * to report a non-busy status, see comments in probe_hwif().
+ * to report a non-busy status, see comments in ide_probe_port().
  */
 int ide_wait_not_busy(ide_hwif_t *hwif, unsigned long timeout)
 {
index 3acb1b692f8ce6870d9b7698b945136e39878dd0..2395498fbc7b6bb86651a4de33ee236aee31747e 100644 (file)
@@ -738,24 +738,20 @@ void ide_undecoded_slave(ide_drive_t *drive1)
 
 EXPORT_SYMBOL_GPL(ide_undecoded_slave);
 
-/*
- * This routine only knows how to look for drive units 0 and 1
- * on an interface, so any setting of MAX_DRIVES > 2 won't work here.
- */
-static void probe_hwif(ide_hwif_t *hwif)
+static int ide_probe_port(ide_hwif_t *hwif)
 {
        unsigned long flags;
        unsigned int irqd;
        int unit;
 
        if (hwif->noprobe)
-               return;
+               return -EACCES;
 
        if ((hwif->chipset != ide_4drives || !hwif->mate || !hwif->mate->present) &&
            (ide_hwif_request_regions(hwif))) {
                printk(KERN_ERR "%s: ports already in use, skipping probe\n",
                                hwif->name);
-               return
+               return -EBUSY;
        }
 
        /*
@@ -804,7 +800,7 @@ static void probe_hwif(ide_hwif_t *hwif)
 
        if (!hwif->present) {
                ide_hwif_release_regions(hwif);
-               return;
+               return -ENODEV;
        }
 
        for (unit = 0; unit < MAX_DRIVES; unit++) {
@@ -840,6 +836,8 @@ static void probe_hwif(ide_hwif_t *hwif)
                else
                        drive->no_io_32bit = drive->id->dword_io ? 1 : 0;
        }
+
+       return 0;
 }
 
 #if MAX_HWIFS > 1
@@ -1311,7 +1309,7 @@ int ide_device_add_all(u8 *idx)
                if (idx[i] == 0xff)
                        continue;
 
-               probe_hwif(&ide_hwifs[idx[i]]);
+               (void)ide_probe_port(&ide_hwifs[idx[i]]);
        }
 
        for (i = 0; i < MAX_HWIFS; i++) {
index d50caa082d504115ebd8f38b347c43185ba99f7c..fc9eee9ccac30d3ef5f7540d7ec23b4d7a24157f 100644 (file)
@@ -229,8 +229,8 @@ static void __devinit init_hwif_ns87415 (ide_hwif_t *hwif)
 
 #ifdef __sparc_v9__
                /*
-                * XXX: Reset the device, if we don't it will not respond
-                *      to SELECT_DRIVE() properly during first probe_hwif().
+                * XXX: Reset the device, if we don't it will not respond to
+                *      SELECT_DRIVE() properly during first ide_probe_port().
                 */
                timeout = 10000;
                outb(12, hwif->io_ports[IDE_CONTROL_OFFSET]);