From: Ondrej Zary <li...@rainbow-software.org>

When a SW-configurable card is specified but not found, the driver
releases wrong region, causing the following message in kernel log:
Trying to free nonexistent resource <0000000000000000-000000000000000f>

Fix it by assigning base earlier.

Signed-off-by: Ondrej Zary <li...@rainbow-software.org>
Fixes: a8cfbcaec0c1 ("scsi: g_NCR5380: Stop using scsi_module.c")
Signed-off-by: Finn Thain <fth...@telegraphics.com.au>

---
 drivers/scsi/g_NCR5380.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/scsi/g_NCR5380.c b/drivers/scsi/g_NCR5380.c
index 6a08d3e..d33e71f 100644
--- a/drivers/scsi/g_NCR5380.c
+++ b/drivers/scsi/g_NCR5380.c
@@ -258,12 +258,12 @@ static int generic_NCR5380_init_one(struct 
scsi_host_template *tpnt,
                if (ports[i]) {
                        /* At this point we have our region reserved */
                        magic_configure(i, 0, magic); /* no IRQ yet */
-                       outb(0xc0, ports[i] + 9);
-                       if (inb(ports[i] + 9) != 0x80) {
+                       base = ports[i];
+                       outb(0xc0, base + 9);
+                       if (inb(base + 9) != 0x80) {
                                ret = -ENODEV;
                                goto out_release;
                        }
-                       base = ports[i];
                        port_idx = i;
                } else
                        return -EINVAL;
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to