summaryrefslogtreecommitdiff
path: root/drivers/ide/ide-probe.c
diff options
context:
space:
mode:
authorBartlomiej Zolnierkiewicz <bzolnier@gmail.com>2008-02-02 01:09:36 +0300
committerBartlomiej Zolnierkiewicz <bzolnier@gmail.com>2008-02-02 01:09:36 +0300
commit9d501529b688dada3fd90d91b1517e48d5ac3e05 (patch)
tree455d4f52979dd11abc1ef6dbd3c01128e03118d1 /drivers/ide/ide-probe.c
parentfbd130887a45341259cde02e134581fb3dac6e14 (diff)
downloadlinux-9d501529b688dada3fd90d91b1517e48d5ac3e05.tar.xz
ide: make probe_hwif() return an error value
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>
Diffstat (limited to 'drivers/ide/ide-probe.c')
-rw-r--r--drivers/ide/ide-probe.c16
1 files changed, 7 insertions, 9 deletions
diff --git a/drivers/ide/ide-probe.c b/drivers/ide/ide-probe.c
index 3acb1b692f8c..2395498fbc7b 100644
--- a/drivers/ide/ide-probe.c
+++ b/drivers/ide/ide-probe.c
@@ -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++) {