Commit faea9111 authored by Dave Jones's avatar Dave Jones

[WATCHDOG] PNP API conversion.

Done by Adam Belay.
parent 498b9cb8
...@@ -23,6 +23,7 @@ ...@@ -23,6 +23,7 @@
* Add WDIOC_GETBOOTSTATUS and WDIOC_SETOPTIONS ioctls * Add WDIOC_GETBOOTSTATUS and WDIOC_SETOPTIONS ioctls
* Fix CONFIG_WATCHDOG_NOWAYOUT * Fix CONFIG_WATCHDOG_NOWAYOUT
* 20020530 Joel Becker Add Matt Domsch's nowayout module option * 20020530 Joel Becker Add Matt Domsch's nowayout module option
* 20030116 Adam Belay Updated to the latest pnp code
* *
*/ */
...@@ -35,7 +36,7 @@ ...@@ -35,7 +36,7 @@
#include <linux/notifier.h> #include <linux/notifier.h>
#include <linux/reboot.h> #include <linux/reboot.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/isapnp.h> #include <linux/pnp.h>
#include <linux/pci.h> #include <linux/pci.h>
#include <asm/semaphore.h> #include <asm/semaphore.h>
...@@ -75,9 +76,9 @@ struct semaphore open_sem; ...@@ -75,9 +76,9 @@ struct semaphore open_sem;
static char expect_close; static char expect_close;
spinlock_t sc1200wdt_lock; /* io port access serialisation */ spinlock_t sc1200wdt_lock; /* io port access serialisation */
#if defined CONFIG_ISAPNP || defined CONFIG_ISAPNP_MODULE #if defined CONFIG_PNP
static int isapnp = 1; static int isapnp = 1;
static struct pci_dev *wdt_dev; static struct pnp_dev *wdt_dev;
MODULE_PARM(isapnp, "i"); MODULE_PARM(isapnp, "i");
MODULE_PARM_DESC(isapnp, "When set to 0 driver ISA PnP support will be disabled"); MODULE_PARM_DESC(isapnp, "When set to 0 driver ISA PnP support will be disabled");
...@@ -328,40 +329,49 @@ static int __init sc1200wdt_probe(void) ...@@ -328,40 +329,49 @@ static int __init sc1200wdt_probe(void)
} }
#if defined CONFIG_ISAPNP || defined CONFIG_ISAPNP_MODULE #if defined CONFIG_PNP
static int __init sc1200wdt_isapnp_probe(void) struct pnp_device_id scl200wdt_pnp_devices[] = {
/* National Semiconductor PC87307/PC97307 watchdog component */
{.id = "NSC0800", .driver_data = 0},
{.id = ""}
};
static int scl200wdt_pnp_probe(struct pnp_dev * dev, const struct pnp_device_id *dev_id)
{ {
int ret; /* this driver only supports one card at a time */
if (wdt_dev || !isapnp)
return -EBUSY;
/* The WDT is logical device 8 on the main device */ wdt_dev = dev;
wdt_dev = isapnp_find_dev(NULL, ISAPNP_VENDOR('N','S','C'), ISAPNP_FUNCTION(0x08), NULL); io = pnp_port_start(wdt_dev, 0);
if (!wdt_dev) io_len = pnp_port_len(wdt_dev, 0);
return -ENODEV;
if (wdt_dev->prepare(wdt_dev) < 0) {
printk(KERN_ERR PFX "ISA PnP found device that could not be autoconfigured\n");
return -EAGAIN;
}
if (!(pci_resource_flags(wdt_dev, 0) & IORESOURCE_IO)) { if (!request_region(io, io_len, SC1200_MODULE_NAME)) {
printk(KERN_ERR PFX "ISA PnP could not find io ports\n"); printk(KERN_ERR PFX "Unable to register IO port %#x\n", io);
return -ENODEV; return -EBUSY;
} }
ret = wdt_dev->activate(wdt_dev); printk(KERN_INFO "scl200wdt: PnP device found at io port %#x/%d\n", io, io_len);
if (ret && (ret != -EBUSY))
return -ENOMEM;
/* io port resource overriding support? */
io = pci_resource_start(wdt_dev, 0);
io_len = pci_resource_len(wdt_dev, 0);
printk(KERN_DEBUG PFX "ISA PnP found device at io port %#x/%d\n", io, io_len);
return 0; return 0;
} }
#endif /* CONFIG_ISAPNP */ static void scl200wdt_pnp_remove(struct pnp_dev * dev)
{
if (wdt_dev){
release_region(io, io_len);
wdt_dev = NULL;
}
}
static struct pnp_driver scl200wdt_pnp_driver = {
.name = "scl200wdt",
.id_table = scl200wdt_pnp_devices,
.probe = scl200wdt_pnp_probe,
.remove = scl200wdt_pnp_remove,
};
#endif /* CONFIG_PNP */
static int __init sc1200wdt_init(void) static int __init sc1200wdt_init(void)
...@@ -373,9 +383,9 @@ static int __init sc1200wdt_init(void) ...@@ -373,9 +383,9 @@ static int __init sc1200wdt_init(void)
spin_lock_init(&sc1200wdt_lock); spin_lock_init(&sc1200wdt_lock);
sema_init(&open_sem, 1); sema_init(&open_sem, 1);
#if defined CONFIG_ISAPNP || defined CONFIG_ISAPNP_MODULE #if defined CONFIG_PNP
if (isapnp) { if (isapnp) {
ret = sc1200wdt_isapnp_probe(); ret = pnp_register_driver(&scl200wdt_pnp_driver);
if (ret) if (ret)
goto out_clean; goto out_clean;
} }
...@@ -387,12 +397,17 @@ static int __init sc1200wdt_init(void) ...@@ -387,12 +397,17 @@ static int __init sc1200wdt_init(void)
goto out_clean; goto out_clean;
} }
/* now that the user has specified an IO port and we haven't detected
* any devices, disable pnp support */
isapnp = 0;
pnp_unregister_driver(&scl200wdt_pnp_driver);
if (!request_region(io, io_len, SC1200_MODULE_NAME)) { if (!request_region(io, io_len, SC1200_MODULE_NAME)) {
printk(KERN_ERR PFX "Unable to register IO port %#x\n", io); printk(KERN_ERR PFX "Unable to register IO port %#x\n", io);
ret = -EBUSY; ret = -EBUSY;
goto out_pnp; goto out_clean;
} }
ret = sc1200wdt_probe(); ret = sc1200wdt_probe();
if (ret) if (ret)
goto out_io; goto out_io;
...@@ -420,11 +435,6 @@ static int __init sc1200wdt_init(void) ...@@ -420,11 +435,6 @@ static int __init sc1200wdt_init(void)
out_io: out_io:
release_region(io, io_len); release_region(io, io_len);
out_pnp:
#if defined CONFIG_ISAPNP || defined CONFIG_ISAPNP_MODULE
if (isapnp && wdt_dev)
wdt_dev->deactivate(wdt_dev);
#endif
goto out_clean; goto out_clean;
} }
...@@ -434,11 +444,11 @@ static void __exit sc1200wdt_exit(void) ...@@ -434,11 +444,11 @@ static void __exit sc1200wdt_exit(void)
misc_deregister(&sc1200wdt_miscdev); misc_deregister(&sc1200wdt_miscdev);
unregister_reboot_notifier(&sc1200wdt_notifier); unregister_reboot_notifier(&sc1200wdt_notifier);
#if defined CONFIG_ISAPNP || defined CONFIG_ISAPNP_MODULE #if defined CONFIG_PNP
if(isapnp && wdt_dev) if(isapnp)
wdt_dev->deactivate(wdt_dev); pnp_unregister_driver(&scl200wdt_pnp_driver);
else
#endif #endif
release_region(io, io_len); release_region(io, io_len);
} }
...@@ -455,7 +465,7 @@ static int __init sc1200wdt_setup(char *str) ...@@ -455,7 +465,7 @@ static int __init sc1200wdt_setup(char *str)
if (ints[0] > 1) if (ints[0] > 1)
timeout = ints[2]; timeout = ints[2];
#if defined CONFIG_ISAPNP || defined CONFIG_ISAPNP_MODULE #if defined CONFIG_PNP
if (ints[0] > 2) if (ints[0] > 2)
isapnp = ints[3]; isapnp = ints[3];
#endif #endif
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment