atp870u: Use pci_request_regions
authorOndrej Zary <linux@rainbow-software.org>
Tue, 17 Nov 2015 18:24:22 +0000 (19:24 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 26 Nov 2015 03:08:51 +0000 (22:08 -0500)
Use pci_request_regions and do it before accessing the I/O ports.
Also add missing pci_disable_device() call to atp870u_remove().

Signed-off-by: Ondrej Zary <linux@rainbow-software.org>
Reviewed-by: Hannes Reinicke <hare@suse.de>
Acked-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/atp870u.c

index 4bb0f4fcd9dfbd539b09940d013879dd74580294..87dd8867106c8dbce6fcd32ea1faa823b2fba45b 100644 (file)
@@ -1276,10 +1276,15 @@ static int atp870u_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
                 goto disable_device;
         }
 
+       err = pci_request_regions(pdev, "atp870u");
+       if (err)
+               goto disable_device;
+       pci_set_master(pdev);
+
         err = -ENOMEM;
        shpnt = scsi_host_alloc(&atp870u_template, sizeof(struct atp_unit));
        if (!shpnt)
-               goto disable_device;
+               goto release_region;
 
        atpdev = shost_priv(shpnt);
 
@@ -1582,10 +1587,6 @@ flash_ok_885:
 
        } 
                spin_unlock_irqrestore(shpnt->host_lock, flags);
-               if (!request_region(shpnt->io_port, shpnt->n_io_port, "atp870u")) {
-                       err = -EBUSY;
-                       goto request_io_fail;
-               }
                err = scsi_add_host(shpnt, &pdev->dev);
                if (err)
                        goto scsi_add_fail;
@@ -1596,16 +1597,13 @@ flash_ok_885:
                return 0;
 
 scsi_add_fail:
-       printk("atp870u_prob:scsi_add_fail\n");
-       release_region(shpnt->io_port, shpnt->n_io_port);
-request_io_fail:
-       printk("atp870u_prob:request_io_fail\n");
        free_irq(shpnt->irq, shpnt);
 free_tables:
-       printk("atp870u_prob:free_table\n");
        atp870u_free_tables(shpnt);
 unregister:
        scsi_host_put(shpnt);
+release_region:
+       pci_release_regions(pdev);
 disable_device:
        pci_disable_device(pdev);
 fail:
@@ -1696,7 +1694,8 @@ static void atp870u_remove (struct pci_dev *pdev)
        
        scsi_remove_host(pshost);
        free_irq(pshost->irq, pshost);
-       release_region(pshost->io_port, pshost->n_io_port);
+       pci_release_regions(pdev);
+       pci_disable_device(pdev);
        atp870u_free_tables(pshost);
        scsi_host_put(pshost);
 }