ncr5380: Cleanup bogus {request,release}_region() calls
authorFinn Thain <fthain@telegraphics.com.au>
Sun, 3 Jan 2016 05:05:20 +0000 (16:05 +1100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 7 Jan 2016 02:42:55 +0000 (21:42 -0500)
Commit 8b801ead3d7a ("[ARM] rpc: update Acorn SCSI drivers to modern ecard
interfaces") neglected to remove a request_region() call in cumana_1.c.

Commit eda32612f7b2 ("[PATCH] give all LLDD driver a ->release method") in
history/history.git added some pointless release_region() calls in dtc.c,
pas16.c and t128.c.

Fix these issues.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/arm/cumana_1.c
drivers/scsi/dtc.c
drivers/scsi/pas16.c
drivers/scsi/t128.c

index c7dc65e39cdbcc9d19168675ce88424e001ef959..3b5ff105f1d66a3a2cae37f8687ae15c96871b87 100644 (file)
@@ -245,12 +245,6 @@ static int cumanascsi1_probe(struct expansion_card *ec,
         priv(host)->ctrl = 0;
         writeb(0, priv(host)->base + CTRL);
 
-       host->n_io_port = 255;
-       if (!(request_region(host->io_port, host->n_io_port, "CumanaSCSI-1"))) {
-               ret = -EBUSY;
-               goto out_unmap;
-       }
-
        ret = request_irq(host->irq, cumanascsi_intr, 0,
                          "CumanaSCSI-1", host);
        if (ret) {
index 02a5532f4267e7f932b8a4e7fef753aae51054fb..698bea067af9438f597bd2101f63ce00f24c05d0 100644 (file)
@@ -423,8 +423,6 @@ static int dtc_release(struct Scsi_Host *shost)
        if (shost->irq != NO_IRQ)
                free_irq(shost->irq, shost);
        NCR5380_exit(shost);
-       if (shost->io_port && shost->n_io_port)
-               release_region(shost->io_port, shost->n_io_port);
        scsi_unregister(shost);
        iounmap(hostdata->base);
        return 0;
index c316ff7ffef6dc49dfafe62df23db313852afe1a..e5c76eefa5e08829a7694a701ec5bef5b59eaa40 100644 (file)
@@ -540,8 +540,6 @@ static int pas16_release(struct Scsi_Host *shost)
        if (shost->irq != NO_IRQ)
                free_irq(shost->irq, shost);
        NCR5380_exit(shost);
-       if (shost->io_port && shost->n_io_port)
-               release_region(shost->io_port, shost->n_io_port);
        scsi_unregister(shost);
        return 0;
 }
index d5e6b676d75e49d60d3ed302142a7250dcef5b97..3adcae1150d28bd2c3f96d653c1a145e94461514 100644 (file)
@@ -255,8 +255,6 @@ static int t128_release(struct Scsi_Host *shost)
        if (shost->irq != NO_IRQ)
                free_irq(shost->irq, shost);
        NCR5380_exit(shost);
-       if (shost->io_port && shost->n_io_port)
-               release_region(shost->io_port, shost->n_io_port);
        scsi_unregister(shost);
        iounmap(hostdata->base);
        return 0;