[WATCHDOG] wdt_pci - use pci_request_region
authorWim Van Sebroeck <wim@iguana.be>
Thu, 6 Aug 2009 18:57:49 +0000 (18:57 +0000)
committerWim Van Sebroeck <wim@iguana.be>
Fri, 18 Sep 2009 08:38:49 +0000 (08:38 +0000)
Use pci_request_region instead of request_region for this pci_driver.

Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/wdt_pci.c

index 7a1bdc7c95a98fdbcc7a897190b4b7634b2e2e85..02b6bdaa2deef648e2f36aa156b152e1c42d547a 100644 (file)
@@ -647,14 +647,15 @@ static int __devinit wdtpci_init_one(struct pci_dev *dev,
                goto out_pci;
        }
 
-       irq = dev->irq;
-       io = pci_resource_start(dev, 2);
-
-       if (request_region(io, 16, "wdt_pci") == NULL) {
-               printk(KERN_ERR PFX "I/O address 0x%04x already in use\n", io);
+       if (pci_request_region(dev, 2, "wdt_pci")) {
+               printk(KERN_ERR PFX "I/O address 0x%04x already in use\n",
+                       pci_resource_start(dev, 2));
                goto out_pci;
        }
 
+       irq = dev->irq;
+       io = pci_resource_start(dev, 2);
+
        if (request_irq(irq, wdtpci_interrupt, IRQF_DISABLED | IRQF_SHARED,
                         "wdt_pci", &wdtpci_miscdev)) {
                printk(KERN_ERR PFX "IRQ %d is not free\n", irq);
@@ -717,7 +718,7 @@ out_rbt:
 out_irq:
        free_irq(irq, &wdtpci_miscdev);
 out_reg:
-       release_region(io, 16);
+       pci_release_region(dev, 2);
 out_pci:
        pci_disable_device(dev);
        goto out;
@@ -733,7 +734,7 @@ static void __devexit wdtpci_remove_one(struct pci_dev *pdev)
                misc_deregister(&temp_miscdev);
        unregister_reboot_notifier(&wdtpci_notifier);
        free_irq(irq, &wdtpci_miscdev);
-       release_region(io, 16);
+       pci_release_region(pdev, 2);
        pci_disable_device(pdev);
        dev_count--;
 }