Merge master.kernel.org:/pub/scm/linux/kernel/git/wim/linux-2.6-watchdog
[pandora-kernel.git] / drivers / char / watchdog / rm9k_wdt.c
index 8cd12ab..b467883 100644 (file)
@@ -26,6 +26,7 @@
 #include <linux/interrupt.h>
 #include <linux/fs.h>
 #include <linux/reboot.h>
+#include <linux/notifier.h>
 #include <linux/miscdevice.h>
 #include <linux/watchdog.h>
 #include <asm/io.h>
 #define CPGIG1ER               0x0054
 
 
-
 /* Function prototypes */
-static int __init wdt_gpi_probe(struct device *);
-static int __exit wdt_gpi_remove(struct device *);
+static irqreturn_t wdt_gpi_irqhdl(int, void *);
+static void wdt_gpi_start(void);
+static void wdt_gpi_stop(void);
 static void wdt_gpi_set_timeout(unsigned int);
 static int wdt_gpi_open(struct inode *, struct file *);
 static int wdt_gpi_release(struct inode *, struct file *);
 static ssize_t wdt_gpi_write(struct file *, const char __user *, size_t, loff_t *);
 static long wdt_gpi_ioctl(struct file *, unsigned int, unsigned long);
-static const struct resource *wdt_gpi_get_resource(struct platform_device *, const char *, unsigned int);
 static int wdt_gpi_notify(struct notifier_block *, unsigned long, void *);
-static irqreturn_t wdt_gpi_irqhdl(int, void *, struct pt_regs *);
-
-
+static const struct resource *wdt_gpi_get_resource(struct platform_device *, const char *, unsigned int);
+static int __init wdt_gpi_probe(struct device *);
+static int __exit wdt_gpi_remove(struct device *);
 
 
 static const char wdt_gpi_name[] = "wdt_gpi";
 static atomic_t opencnt;
 static int expect_close;
-static int locked = 0;
-
+static int locked;
 
 
 /* These are set from device resources */
@@ -73,23 +72,30 @@ static void __iomem * wd_regs;
 static unsigned int wd_irq, wd_ctr;
 
 
-
 /* Module arguments */
 static int timeout = MAX_TIMEOUT_SECONDS;
 module_param(timeout, int, 0444);
+MODULE_PARM_DESC(timeout, "Watchdog timeout in seconds");
+
 static unsigned long resetaddr = 0xbffdc200;
 module_param(resetaddr, ulong, 0444);
+MODULE_PARM_DESC(resetaddr, "Address to write to to force a reset");
+
 static unsigned long flagaddr = 0xbffdc104;
 module_param(flagaddr, ulong, 0444);
-static int powercycle = 0;
+MODULE_PARM_DESC(flagaddr, "Address to write to boot flags to");
+
+static int powercycle;
 module_param(powercycle, bool, 0444);
+MODULE_PARM_DESC(powercycle, "Cycle power if watchdog expires");
 
 static int nowayout = WATCHDOG_NOWAYOUT;
 module_param(nowayout, bool, 0444);
+MODULE_PARM_DESC(nowayout, "Watchdog cannot be disabled once started");
 
 
-
-static struct file_operations fops = {
+/* Kernel interfaces */
+static const struct file_operations fops = {
        .owner          = THIS_MODULE,
        .open           = wdt_gpi_open,
        .release        = wdt_gpi_release,
@@ -103,77 +109,54 @@ static struct miscdevice miscdev = {
        .fops           = &fops,
 };
 
-static struct device_driver wdt_gpi_driver = {
-       .name           = (char *) wdt_gpi_name,
-       .bus            = &platform_bus_type,
-       .owner          = THIS_MODULE,
-       .probe          = wdt_gpi_probe,
-       .remove         = __exit_p(wdt_gpi_remove),
-       .shutdown       = NULL,
-       .suspend        = NULL,
-       .resume         = NULL,
-};
-
 static struct notifier_block wdt_gpi_shutdown = {
        .notifier_call  = wdt_gpi_notify,
 };
 
 
-
-static const struct resource *
-wdt_gpi_get_resource(struct platform_device *pdv, const char *name,
-                     unsigned int type)
+/* Interrupt handler */
+static irqreturn_t wdt_gpi_irqhdl(int irq, void *ctxt)
 {
-       char buf[80];
-       if (snprintf(buf, sizeof buf, "%s_0", name) >= sizeof buf)
-               return NULL;
-       return platform_get_resource_byname(pdv, type, buf);
-}
-
+       if (!unlikely(__raw_readl(wd_regs + 0x0008) & 0x1))
+               return IRQ_NONE;
+       __raw_writel(0x1, wd_regs + 0x0008);
 
 
-/* No hotplugging on the platform bus - use __init */
-static int __init wdt_gpi_probe(struct device *dev)
-{
-       int res;
-       struct platform_device * const pdv = to_platform_device(dev);
-       const struct resource
-               * const rr = wdt_gpi_get_resource(pdv, WDT_RESOURCE_REGS,
-                                                 IORESOURCE_MEM),
-               * const ri = wdt_gpi_get_resource(pdv, WDT_RESOURCE_IRQ,
-                                                 IORESOURCE_IRQ),
-               * const rc = wdt_gpi_get_resource(pdv, WDT_RESOURCE_COUNTER,
-                                                 0);
-
-       if (unlikely(!rr || !ri || !rc))
-               return -ENXIO;
+       printk(KERN_CRIT "%s: watchdog expired - resetting system\n",
+               wdt_gpi_name);
 
-       wd_regs = ioremap_nocache(rr->start, rr->end + 1 - rr->start);
-       if (unlikely(!wd_regs))
-               return -ENOMEM;
-       wd_irq = ri->start;
-       wd_ctr = rc->start;
-       res = misc_register(&miscdev);
-       if (res)
-               iounmap(wd_regs);
-       else
-               register_reboot_notifier(&wdt_gpi_shutdown);
-       return res;
+       *(volatile char *) flagaddr |= 0x01;
+       *(volatile char *) resetaddr = powercycle ? 0x01 : 0x2;
+       iob();
+       while (1)
+               cpu_relax();
 }
 
 
-
-static int __exit wdt_gpi_remove(struct device *dev)
+/* Watchdog functions */
+static void wdt_gpi_start(void)
 {
-       int res;
+       u32 reg;
 
-       unregister_reboot_notifier(&wdt_gpi_shutdown);
-       res = misc_deregister(&miscdev);
-       iounmap(wd_regs);
-       wd_regs = NULL;
-       return res;
+       lock_titan_regs();
+       reg = titan_readl(CPGIG1ER);
+       titan_writel(reg | (0x100 << wd_ctr), CPGIG1ER);
+       iob();
+       unlock_titan_regs();
 }
 
+static void wdt_gpi_stop(void)
+{
+       u32 reg;
+
+       lock_titan_regs();
+       reg = titan_readl(CPCCR) & ~(0xf << (wd_ctr * 4));
+       titan_writel(reg, CPCCR);
+       reg = titan_readl(CPGIG1ER);
+       titan_writel(reg & ~(0x100 << wd_ctr), CPGIG1ER);
+       iob();
+       unlock_titan_regs();
+}
 
 static void wdt_gpi_set_timeout(unsigned int to)
 {
@@ -194,13 +177,12 @@ static void wdt_gpi_set_timeout(unsigned int to)
 }
 
 
-
-static int wdt_gpi_open(struct inode *i, struct file *f)
+/* /dev/watchdog operations */
+static int wdt_gpi_open(struct inode *inode, struct file *file)
 {
        int res;
-       u32 reg;
 
-       if (unlikely(0 > atomic_dec_if_positive(&opencnt)))
+       if (unlikely(atomic_dec_if_positive(&opencnt) < 0))
                return -EBUSY;
 
        expect_close = 0;
@@ -216,42 +198,27 @@ static int wdt_gpi_open(struct inode *i, struct file *f)
                return res;
 
        wdt_gpi_set_timeout(timeout);
-
-       lock_titan_regs();
-       reg = titan_readl(CPGIG1ER);
-       titan_writel(reg | (0x100 << wd_ctr), CPGIG1ER);
-       iob();
-       unlock_titan_regs();
+       wdt_gpi_start();
 
        printk(KERN_INFO "%s: watchdog started, timeout = %u seconds\n",
                wdt_gpi_name, timeout);
-       return 0;
+       return nonseekable_open(inode, file);
 }
 
-
-
-static int wdt_gpi_release(struct inode *i, struct file *f)
+static int wdt_gpi_release(struct inode *inode, struct file *file)
 {
        if (nowayout) {
-               printk(KERN_NOTICE "%s: no way out - watchdog left running\n",
+               printk(KERN_INFO "%s: no way out - watchdog left running\n",
                        wdt_gpi_name);
                __module_get(THIS_MODULE);
                locked = 1;
        } else {
                if (expect_close) {
-                       u32 reg;
-
-                       lock_titan_regs();
-                       reg = titan_readl(CPCCR) & ~(0xf << (wd_ctr * 4));
-                       titan_writel(reg, CPCCR);
-                       reg = titan_readl(CPGIG1ER);
-                       titan_writel(reg & ~(0x100 << wd_ctr), CPGIG1ER);
-                       iob();
-                       unlock_titan_regs();
+                       wdt_gpi_stop();
                        free_irq(wd_irq, &miscdev);
                        printk(KERN_INFO "%s: watchdog stopped\n", wdt_gpi_name);
                } else {
-                       printk(KERN_NOTICE "%s: unexpected close() -"
+                       printk(KERN_CRIT "%s: unexpected close() -"
                                " watchdog left running\n",
                                wdt_gpi_name);
                        wdt_gpi_set_timeout(timeout);
@@ -264,8 +231,6 @@ static int wdt_gpi_release(struct inode *i, struct file *f)
        return 0;
 }
 
-
-
 static ssize_t
 wdt_gpi_write(struct file *f, const char __user *d, size_t s, loff_t *o)
 {
@@ -276,14 +241,13 @@ wdt_gpi_write(struct file *f, const char __user *d, size_t s, loff_t *o)
        return s ? 1 : 0;
 }
 
-
-
 static long
 wdt_gpi_ioctl(struct file *f, unsigned int cmd, unsigned long arg)
 {
        long res = -ENOTTY;
        const long size = _IOC_SIZE(cmd);
        int stat;
+       void __user *argp = (void __user *)arg;
        static struct watchdog_info wdinfo = {
                .identity               = "RM9xxx/GPI watchdog",
                .firmware_version       = 0,
@@ -308,8 +272,7 @@ wdt_gpi_ioctl(struct file *f, unsigned int cmd, unsigned long arg)
                wdinfo.options = nowayout ?
                        WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING :
                        WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | WDIOF_MAGICCLOSE;
-               res = __copy_to_user((void __user *)arg, &wdinfo, size) ?
-                       -EFAULT : size;
+               res = __copy_to_user(argp, &wdinfo, size) ?  -EFAULT : size;
                break;
 
        case WDIOC_GETSTATUS:
@@ -318,7 +281,7 @@ wdt_gpi_ioctl(struct file *f, unsigned int cmd, unsigned long arg)
        case WDIOC_GETBOOTSTATUS:
                stat = (*(volatile char *) flagaddr & 0x01)
                        ? WDIOF_CARDRESET : 0;
-               res = __copy_to_user((void __user *)arg, &stat, size) ?
+               res = __copy_to_user(argp, &stat, size) ?
                        -EFAULT : size;
                break;
 
@@ -333,24 +296,23 @@ wdt_gpi_ioctl(struct file *f, unsigned int cmd, unsigned long arg)
        case WDIOC_SETTIMEOUT:
                {
                        int val;
-                       if (unlikely(__copy_from_user(&val, (const void __user *) arg,
-                                       size))) {
+                       if (unlikely(__copy_from_user(&val, argp, size))) {
                                res = -EFAULT;
                                break;
                        }
 
-                       if (val > 32)
-                               val = 32;
+                       if (val > MAX_TIMEOUT_SECONDS)
+                               val = MAX_TIMEOUT_SECONDS;
                        timeout = val;
                        wdt_gpi_set_timeout(val);
                        res = size;
-                       printk("%s: timeout set to %u seconds\n",
+                       printk(KERN_INFO "%s: timeout set to %u seconds\n",
                                wdt_gpi_name, timeout);
                }
                break;
 
        case WDIOC_GETTIMEOUT:
-               res = __copy_to_user((void __user *) arg, &timeout, size) ?
+               res = __copy_to_user(argp, &timeout, size) ?
                        -EFAULT : size;
                break;
        }
@@ -359,45 +321,80 @@ wdt_gpi_ioctl(struct file *f, unsigned int cmd, unsigned long arg)
 }
 
 
-
-
-static irqreturn_t wdt_gpi_irqhdl(int irq, void *ctxt, struct pt_regs *regs)
+/* Shutdown notifier */
+static int
+wdt_gpi_notify(struct notifier_block *this, unsigned long code, void *unused)
 {
-       if (!unlikely(__raw_readl(wd_regs + 0x0008) & 0x1))
-               return IRQ_NONE;
-       __raw_writel(0x1, wd_regs + 0x0008);
+       if (code == SYS_DOWN || code == SYS_HALT)
+               wdt_gpi_stop();
 
+       return NOTIFY_DONE;
+}
 
-       printk(KERN_WARNING "%s: watchdog expired - resetting system\n",
-               wdt_gpi_name);
 
-       *(volatile char *) flagaddr |= 0x01;
-       *(volatile char *) resetaddr = powercycle ? 0x01 : 0x2;
-       iob();
-       while (1)
-               cpu_relax();
+/* Init & exit procedures */
+static const struct resource *
+wdt_gpi_get_resource(struct platform_device *pdv, const char *name,
+                     unsigned int type)
+{
+       char buf[80];
+       if (snprintf(buf, sizeof buf, "%s_0", name) >= sizeof buf)
+               return NULL;
+       return platform_get_resource_byname(pdv, type, buf);
 }
 
+/* No hotplugging on the platform bus - use __init */
+static int __init wdt_gpi_probe(struct device *dev)
+{
+       int res;
+       struct platform_device * const pdv = to_platform_device(dev);
+       const struct resource
+               * const rr = wdt_gpi_get_resource(pdv, WDT_RESOURCE_REGS,
+                                                 IORESOURCE_MEM),
+               * const ri = wdt_gpi_get_resource(pdv, WDT_RESOURCE_IRQ,
+                                                 IORESOURCE_IRQ),
+               * const rc = wdt_gpi_get_resource(pdv, WDT_RESOURCE_COUNTER,
+                                                 0);
 
+       if (unlikely(!rr || !ri || !rc))
+               return -ENXIO;
 
-static int
-wdt_gpi_notify(struct notifier_block *this, unsigned long code, void *unused)
+       wd_regs = ioremap_nocache(rr->start, rr->end + 1 - rr->start);
+       if (unlikely(!wd_regs))
+               return -ENOMEM;
+       wd_irq = ri->start;
+       wd_ctr = rc->start;
+       res = misc_register(&miscdev);
+       if (res)
+               iounmap(wd_regs);
+       else
+               register_reboot_notifier(&wdt_gpi_shutdown);
+       return res;
+}
+
+static int __exit wdt_gpi_remove(struct device *dev)
 {
-       if(code == SYS_DOWN || code == SYS_HALT) {
-               u32 reg;
-
-               lock_titan_regs();
-               reg = titan_readl(CPCCR) & ~(0xf << (wd_ctr * 4));
-               titan_writel(reg, CPCCR);
-               reg = titan_readl(CPGIG1ER);
-               titan_writel(reg & ~(0x100 << wd_ctr), CPGIG1ER);
-               iob();
-               unlock_titan_regs();
-       }
-       return NOTIFY_DONE;
+       int res;
+
+       unregister_reboot_notifier(&wdt_gpi_shutdown);
+       res = misc_deregister(&miscdev);
+       iounmap(wd_regs);
+       wd_regs = NULL;
+       return res;
 }
 
 
+/* Device driver init & exit */
+static struct device_driver wdt_gpi_driver = {
+       .name           = (char *) wdt_gpi_name,
+       .bus            = &platform_bus_type,
+       .owner          = THIS_MODULE,
+       .probe          = wdt_gpi_probe,
+       .remove         = __exit_p(wdt_gpi_remove),
+       .shutdown       = NULL,
+       .suspend        = NULL,
+       .resume         = NULL,
+};
 
 static int __init wdt_gpi_init_module(void)
 {
@@ -407,8 +404,6 @@ static int __init wdt_gpi_init_module(void)
        return driver_register(&wdt_gpi_driver);
 }
 
-
-
 static void __exit wdt_gpi_cleanup_module(void)
 {
        driver_unregister(&wdt_gpi_driver);
@@ -417,15 +412,9 @@ static void __exit wdt_gpi_cleanup_module(void)
 module_init(wdt_gpi_init_module);
 module_exit(wdt_gpi_cleanup_module);
 
-
-
 MODULE_AUTHOR("Thomas Koeller <thomas.koeller@baslerweb.com>");
 MODULE_DESCRIPTION("Basler eXcite watchdog driver for gpi devices");
 MODULE_VERSION("0.1");
 MODULE_LICENSE("GPL");
 MODULE_ALIAS_MISCDEV(WATCHDOG_MINOR);
-MODULE_PARM_DESC(timeout, "Watchdog timeout in seconds");
-MODULE_PARM_DESC(resetaddr, "Address to write to to force a reset");
-MODULE_PARM_DESC(flagaddr, "Address to write to boot flags to");
-MODULE_PARM_DESC(nowayout, "Watchdog cannot be disabled once started");
-MODULE_PARM_DESC(powercycle, "Cycle power if watchdog expires");
+