diff options
author | Wim Van Sebroeck <wim@iguana.be> | 2009-08-06 14:57:49 -0400 |
---|---|---|
committer | Wim Van Sebroeck <wim@iguana.be> | 2009-09-18 04:38:49 -0400 |
commit | 119d3e56e7c82a73d27b5dd010c52dab1bc9f846 (patch) | |
tree | b4e8bed69dd451d64a1e052a1282dedb8015b641 /drivers | |
parent | d7e9791bc1cbf635f13859216a825af5199a2061 (diff) |
[WATCHDOG] wdt_pci - use pci_request_region
Use pci_request_region instead of request_region for this pci_driver.
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/watchdog/wdt_pci.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/drivers/watchdog/wdt_pci.c b/drivers/watchdog/wdt_pci.c index 7a1bdc7c95a9..02b6bdaa2dee 100644 --- a/drivers/watchdog/wdt_pci.c +++ b/drivers/watchdog/wdt_pci.c | |||
@@ -647,14 +647,15 @@ static int __devinit wdtpci_init_one(struct pci_dev *dev, | |||
647 | goto out_pci; | 647 | goto out_pci; |
648 | } | 648 | } |
649 | 649 | ||
650 | irq = dev->irq; | 650 | if (pci_request_region(dev, 2, "wdt_pci")) { |
651 | io = pci_resource_start(dev, 2); | 651 | printk(KERN_ERR PFX "I/O address 0x%04x already in use\n", |
652 | 652 | pci_resource_start(dev, 2)); | |
653 | if (request_region(io, 16, "wdt_pci") == NULL) { | ||
654 | printk(KERN_ERR PFX "I/O address 0x%04x already in use\n", io); | ||
655 | goto out_pci; | 653 | goto out_pci; |
656 | } | 654 | } |
657 | 655 | ||
656 | irq = dev->irq; | ||
657 | io = pci_resource_start(dev, 2); | ||
658 | |||
658 | if (request_irq(irq, wdtpci_interrupt, IRQF_DISABLED | IRQF_SHARED, | 659 | if (request_irq(irq, wdtpci_interrupt, IRQF_DISABLED | IRQF_SHARED, |
659 | "wdt_pci", &wdtpci_miscdev)) { | 660 | "wdt_pci", &wdtpci_miscdev)) { |
660 | printk(KERN_ERR PFX "IRQ %d is not free\n", irq); | 661 | printk(KERN_ERR PFX "IRQ %d is not free\n", irq); |
@@ -717,7 +718,7 @@ out_rbt: | |||
717 | out_irq: | 718 | out_irq: |
718 | free_irq(irq, &wdtpci_miscdev); | 719 | free_irq(irq, &wdtpci_miscdev); |
719 | out_reg: | 720 | out_reg: |
720 | release_region(io, 16); | 721 | pci_release_region(dev, 2); |
721 | out_pci: | 722 | out_pci: |
722 | pci_disable_device(dev); | 723 | pci_disable_device(dev); |
723 | goto out; | 724 | goto out; |
@@ -733,7 +734,7 @@ static void __devexit wdtpci_remove_one(struct pci_dev *pdev) | |||
733 | misc_deregister(&temp_miscdev); | 734 | misc_deregister(&temp_miscdev); |
734 | unregister_reboot_notifier(&wdtpci_notifier); | 735 | unregister_reboot_notifier(&wdtpci_notifier); |
735 | free_irq(irq, &wdtpci_miscdev); | 736 | free_irq(irq, &wdtpci_miscdev); |
736 | release_region(io, 16); | 737 | pci_release_region(pdev, 2); |
737 | pci_disable_device(pdev); | 738 | pci_disable_device(pdev); |
738 | dev_count--; | 739 | dev_count--; |
739 | } | 740 | } |