1 files changed, 224 insertions, 113 deletions

diff --git a/drivers/watchdog/omap_wdt.c b/drivers/watchdog/omap_wdt.c
index 3a11dadfd8e7..7bcbb7f4745f 100644
--- a/drivers/watchdog/omap_wdt.c
+++ b/drivers/watchdog/omap_wdt.c
@@ -1,7 +1,7 @@
 /*
- * linux/drivers/char/watchdog/omap_wdt.c
+ * omap_wdt.c
  *
- * Watchdog driver for the TI OMAP 16xx & 24xx 32KHz (non-secure) watchdog
+ * Watchdog driver for the TI OMAP 16xx & 24xx/34xx 32KHz (non-secure) watchdog
  *
  * Author: MontaVista Software, Inc.
  *       <gdavis@mvista.com> or <source@mvista.com>
@@ -47,50 +47,68 @@
 
 #include "omap_wdt.h"
 
+static struct platform_device *omap_wdt_dev;
+
 static unsigned timer_margin;
 module_param(timer_margin, uint, 0);
 MODULE_PARM_DESC(timer_margin, "initial watchdog timeout (in seconds)");
 
-static int omap_wdt_users;
-static struct clk *armwdt_ck;
-static struct clk *mpu_wdt_ick;
-static struct clk *mpu_wdt_fck;
-
 static unsigned int wdt_trgr_pattern = 0x1234;
 static spinlock_t wdt_lock;
 
-static void omap_wdt_ping(void)
+struct omap_wdt_dev {
+        void __iomem    *base;          /* physical */
+        struct device   *dev;
+        int             omap_wdt_users;
+        struct clk      *armwdt_ck;
+        struct clk      *mpu_wdt_ick;
+        struct clk      *mpu_wdt_fck;
+        struct resource *mem;
+        struct miscdevice omap_wdt_miscdev;
+};
+
+static void omap_wdt_ping(struct omap_wdt_dev *wdev)
 {
+        void __iomem    *base = wdev->base;
+
         /* wait for posted write to complete */
-        while ((omap_readl(OMAP_WATCHDOG_WPS)) & 0x08)
+        while ((__raw_readl(base + OMAP_WATCHDOG_WPS)) & 0x08)
                 cpu_relax();
+
         wdt_trgr_pattern = ~wdt_trgr_pattern;
-        omap_writel(wdt_trgr_pattern, (OMAP_WATCHDOG_TGR));
+        __raw_writel(wdt_trgr_pattern, (base + OMAP_WATCHDOG_TGR));
+
         /* wait for posted write to complete */
-        while ((omap_readl(OMAP_WATCHDOG_WPS)) & 0x08)
+        while ((__raw_readl(base + OMAP_WATCHDOG_WPS)) & 0x08)
                 cpu_relax();
         /* reloaded WCRR from WLDR */
 }
 
-static void omap_wdt_enable(void)
+static void omap_wdt_enable(struct omap_wdt_dev *wdev)
 {
+        void __iomem *base = wdev->base;
+
         /* Sequence to enable the watchdog */
-        omap_writel(0xBBBB, OMAP_WATCHDOG_SPR);
-        while ((omap_readl(OMAP_WATCHDOG_WPS)) & 0x10)
+        __raw_writel(0xBBBB, base + OMAP_WATCHDOG_SPR);
+        while ((__raw_readl(base + OMAP_WATCHDOG_WPS)) & 0x10)
                 cpu_relax();
-        omap_writel(0x4444, OMAP_WATCHDOG_SPR);
-        while ((omap_readl(OMAP_WATCHDOG_WPS)) & 0x10)
+
+        __raw_writel(0x4444, base + OMAP_WATCHDOG_SPR);
+        while ((__raw_readl(base + OMAP_WATCHDOG_WPS)) & 0x10)
                 cpu_relax();
 }
 
-static void omap_wdt_disable(void)
+static void omap_wdt_disable(struct omap_wdt_dev *wdev)
 {
+        void __iomem *base = wdev->base;
+
         /* sequence required to disable watchdog */
-        omap_writel(0xAAAA, OMAP_WATCHDOG_SPR); /* TIMER_MODE */
-        while (omap_readl(OMAP_WATCHDOG_WPS) & 0x10)
+        __raw_writel(0xAAAA, base + OMAP_WATCHDOG_SPR); /* TIMER_MODE */
+        while (__raw_readl(base + OMAP_WATCHDOG_WPS) & 0x10)
                 cpu_relax();
-        omap_writel(0x5555, OMAP_WATCHDOG_SPR); /* TIMER_MODE */
-        while (omap_readl(OMAP_WATCHDOG_WPS) & 0x10)
+
+        __raw_writel(0x5555, base + OMAP_WATCHDOG_SPR); /* TIMER_MODE */
+        while (__raw_readl(base + OMAP_WATCHDOG_WPS) & 0x10)
                 cpu_relax();
 }
 
@@ -103,83 +121,90 @@ static void omap_wdt_adjust_timeout(unsigned new_timeout)
         timer_margin = new_timeout;
 }
 
-static void omap_wdt_set_timeout(void)
+static void omap_wdt_set_timeout(struct omap_wdt_dev *wdev)
 {
         u32 pre_margin = GET_WLDR_VAL(timer_margin);
+        void __iomem *base = wdev->base;
 
         /* just count up at 32 KHz */
-        while (omap_readl(OMAP_WATCHDOG_WPS) & 0x04)
+        while (__raw_readl(base + OMAP_WATCHDOG_WPS) & 0x04)
                 cpu_relax();
-        omap_writel(pre_margin, OMAP_WATCHDOG_LDR);
-        while (omap_readl(OMAP_WATCHDOG_WPS) & 0x04)
+
+        __raw_writel(pre_margin, base + OMAP_WATCHDOG_LDR);
+        while (__raw_readl(base + OMAP_WATCHDOG_WPS) & 0x04)
                 cpu_relax();
 }
 
 /*
  *      Allow only one task to hold it open
  */
-
 static int omap_wdt_open(struct inode *inode, struct file *file)
 {
-        if (test_and_set_bit(1, (unsigned long *)&omap_wdt_users))
+        struct omap_wdt_dev *wdev = platform_get_drvdata(omap_wdt_dev);
+        void __iomem *base = wdev->base;
+
+        if (test_and_set_bit(1, (unsigned long *)&(wdev->omap_wdt_users)))
                 return -EBUSY;
 
         if (cpu_is_omap16xx())
-                clk_enable(armwdt_ck);  /* Enable the clock */
+                clk_enable(wdev->armwdt_ck);    /* Enable the clock */
 
-        if (cpu_is_omap24xx()) {
-                clk_enable(mpu_wdt_ick);    /* Enable the interface clock */
-                clk_enable(mpu_wdt_fck);    /* Enable the functional clock */
+        if (cpu_is_omap24xx() || cpu_is_omap34xx()) {
+                clk_enable(wdev->mpu_wdt_ick);    /* Enable the interface clock */
+                clk_enable(wdev->mpu_wdt_fck);    /* Enable the functional clock */
         }
 
         /* initialize prescaler */
-        while (omap_readl(OMAP_WATCHDOG_WPS) & 0x01)
+        while (__raw_readl(base + OMAP_WATCHDOG_WPS) & 0x01)
                 cpu_relax();
-        omap_writel((1 << 5) | (PTV << 2), OMAP_WATCHDOG_CNTRL);
-        while (omap_readl(OMAP_WATCHDOG_WPS) & 0x01)
+
+        __raw_writel((1 << 5) | (PTV << 2), base + OMAP_WATCHDOG_CNTRL);
+        while (__raw_readl(base + OMAP_WATCHDOG_WPS) & 0x01)
                 cpu_relax();
 
-        omap_wdt_set_timeout();
-        omap_wdt_enable();
+        file->private_data = (void *) wdev;
+
+        omap_wdt_set_timeout(wdev);
+        omap_wdt_enable(wdev);
+
         return nonseekable_open(inode, file);
 }
 
 static int omap_wdt_release(struct inode *inode, struct file *file)
 {
+        struct omap_wdt_dev *wdev = file->private_data;
+
         /*
          *      Shut off the timer unless NOWAYOUT is defined.
          */
 #ifndef CONFIG_WATCHDOG_NOWAYOUT
-        omap_wdt_disable();
 
-        if (cpu_is_omap16xx()) {
-                clk_disable(armwdt_ck); /* Disable the clock */
-                clk_put(armwdt_ck);
-                armwdt_ck = NULL;
-        }
+        omap_wdt_disable(wdev);
 
-        if (cpu_is_omap24xx()) {
-                clk_disable(mpu_wdt_ick);       /* Disable the clock */
-                clk_disable(mpu_wdt_fck);       /* Disable the clock */
-                clk_put(mpu_wdt_ick);
-                clk_put(mpu_wdt_fck);
-                mpu_wdt_ick = NULL;
-                mpu_wdt_fck = NULL;
+        if (cpu_is_omap16xx())
+                clk_disable(wdev->armwdt_ck);   /* Disable the clock */
+
+        if (cpu_is_omap24xx() || cpu_is_omap34xx()) {
+                clk_disable(wdev->mpu_wdt_ick); /* Disable the clock */
+                clk_disable(wdev->mpu_wdt_fck); /* Disable the clock */
         }
 #else
         printk(KERN_CRIT "omap_wdt: Unexpected close, not stopping!\n");
 #endif
-        omap_wdt_users = 0;
+        wdev->omap_wdt_users = 0;
+
         return 0;
 }
 
 static ssize_t omap_wdt_write(struct file *file, const char __user *data,
                 size_t len, loff_t *ppos)
 {
+        struct omap_wdt_dev *wdev = file->private_data;
+
         /* Refresh LOAD_TIME. */
         if (len) {
                 spin_lock(&wdt_lock);
-                omap_wdt_ping();
+                omap_wdt_ping(wdev);
                 spin_unlock(&wdt_lock);
         }
         return len;
@@ -188,6 +213,7 @@ static ssize_t omap_wdt_write(struct file *file, const char __user *data,
 static long omap_wdt_ioctl(struct file *file, unsigned int cmd,
                                                 unsigned long arg)
 {
+        struct omap_wdt_dev *wdev;
         int new_margin;
         static const struct watchdog_info ident = {
                 .identity = "OMAP Watchdog",
@@ -195,6 +221,8 @@ static long omap_wdt_ioctl(struct file *file, unsigned int cmd,
                 .firmware_version = 0,
         };
 
+        wdev = file->private_data;
+
         switch (cmd) {
         case WDIOC_GETSUPPORT:
                 return copy_to_user((struct watchdog_info __user *)arg, &ident,
@@ -203,14 +231,14 @@ static long omap_wdt_ioctl(struct file *file, unsigned int cmd,
                 return put_user(0, (int __user *)arg);
         case WDIOC_GETBOOTSTATUS:
                 if (cpu_is_omap16xx())
-                        return put_user(omap_readw(ARM_SYSST),
+                        return put_user(__raw_readw(ARM_SYSST),
                                         (int __user *)arg);
                 if (cpu_is_omap24xx())
                         return put_user(omap_prcm_get_reset_sources(),
                                         (int __user *)arg);
         case WDIOC_KEEPALIVE:
                 spin_lock(&wdt_lock);
-                omap_wdt_ping();
+                omap_wdt_ping(wdev);
                 spin_unlock(&wdt_lock);
                 return 0;
         case WDIOC_SETTIMEOUT:
@@ -219,11 +247,11 @@ static long omap_wdt_ioctl(struct file *file, unsigned int cmd,
                 omap_wdt_adjust_timeout(new_margin);
 
                 spin_lock(&wdt_lock);
-                omap_wdt_disable();
-                omap_wdt_set_timeout();
-                omap_wdt_enable();
+                omap_wdt_disable(wdev);
+                omap_wdt_set_timeout(wdev);
+                omap_wdt_enable(wdev);
 
-                omap_wdt_ping();
+                omap_wdt_ping(wdev);
                 spin_unlock(&wdt_lock);
                 /* Fall */
         case WDIOC_GETTIMEOUT:
@@ -241,96 +269,173 @@ static const struct file_operations omap_wdt_fops = {
         .release = omap_wdt_release,
 };
 
-static struct miscdevice omap_wdt_miscdev = {
-        .minor = WATCHDOG_MINOR,
-        .name = "watchdog",
-        .fops = &omap_wdt_fops,
-};
-
 static int __init omap_wdt_probe(struct platform_device *pdev)
 {
         struct resource *res, *mem;
+        struct omap_wdt_dev *wdev;
         int ret;
 
         /* reserve static register mappings */
         res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-        if (!res)
-                return -ENOENT;
+        if (!res) {
+                ret = -ENOENT;
+                goto err_get_resource;
+        }
+
+        if (omap_wdt_dev) {
+                ret = -EBUSY;
+                goto err_busy;
+        }
 
         mem = request_mem_region(res->start, res->end - res->start + 1,
                                  pdev->name);
-        if (mem == NULL)
-                return -EBUSY;
+        if (!mem) {
+                ret = -EBUSY;
+                goto err_busy;
+        }
 
-        platform_set_drvdata(pdev, mem);
+        wdev = kzalloc(sizeof(struct omap_wdt_dev), GFP_KERNEL);
+        if (!wdev) {
+                ret = -ENOMEM;
+                goto err_kzalloc;
+        }
 
-        omap_wdt_users = 0;
+        wdev->omap_wdt_users = 0;
+        wdev->mem = mem;
 
         if (cpu_is_omap16xx()) {
-                armwdt_ck = clk_get(&pdev->dev, "armwdt_ck");
-                if (IS_ERR(armwdt_ck)) {
-                        ret = PTR_ERR(armwdt_ck);
-                        armwdt_ck = NULL;
-                        goto fail;
+                wdev->armwdt_ck = clk_get(&pdev->dev, "armwdt_ck");
+                if (IS_ERR(wdev->armwdt_ck)) {
+                        ret = PTR_ERR(wdev->armwdt_ck);
+                        wdev->armwdt_ck = NULL;
+                        goto err_clk;
                 }
         }
 
         if (cpu_is_omap24xx()) {
-                mpu_wdt_ick = clk_get(&pdev->dev, "mpu_wdt_ick");
-                if (IS_ERR(mpu_wdt_ick)) {
-                        ret = PTR_ERR(mpu_wdt_ick);
-                        mpu_wdt_ick = NULL;
-                        goto fail;
+                wdev->mpu_wdt_ick = clk_get(&pdev->dev, "mpu_wdt_ick");
+                if (IS_ERR(wdev->mpu_wdt_ick)) {
+                        ret = PTR_ERR(wdev->mpu_wdt_ick);
+                        wdev->mpu_wdt_ick = NULL;
+                        goto err_clk;
                 }
-                mpu_wdt_fck = clk_get(&pdev->dev, "mpu_wdt_fck");
-                if (IS_ERR(mpu_wdt_fck)) {
-                        ret = PTR_ERR(mpu_wdt_fck);
-                        mpu_wdt_fck = NULL;
-                        goto fail;
+                wdev->mpu_wdt_fck = clk_get(&pdev->dev, "mpu_wdt_fck");
+                if (IS_ERR(wdev->mpu_wdt_fck)) {
+                        ret = PTR_ERR(wdev->mpu_wdt_fck);
+                        wdev->mpu_wdt_fck = NULL;
+                        goto err_clk;
                 }
         }
 
-        omap_wdt_disable();
+        if (cpu_is_omap34xx()) {
+                wdev->mpu_wdt_ick = clk_get(&pdev->dev, "wdt2_ick");
+                if (IS_ERR(wdev->mpu_wdt_ick)) {
+                        ret = PTR_ERR(wdev->mpu_wdt_ick);
+                        wdev->mpu_wdt_ick = NULL;
+                        goto err_clk;
+                }
+                wdev->mpu_wdt_fck = clk_get(&pdev->dev, "wdt2_fck");
+                if (IS_ERR(wdev->mpu_wdt_fck)) {
+                        ret = PTR_ERR(wdev->mpu_wdt_fck);
+                        wdev->mpu_wdt_fck = NULL;
+                        goto err_clk;
+                }
+        }
+        wdev->base = ioremap(res->start, res->end - res->start + 1);
+        if (!wdev->base) {
+                ret = -ENOMEM;
+                goto err_ioremap;
+        }
+
+        platform_set_drvdata(pdev, wdev);
+
+        omap_wdt_disable(wdev);
         omap_wdt_adjust_timeout(timer_margin);
 
-        omap_wdt_miscdev.parent = &pdev->dev;
-        ret = misc_register(&omap_wdt_miscdev);
+        wdev->omap_wdt_miscdev.parent = &pdev->dev;
+        wdev->omap_wdt_miscdev.minor = WATCHDOG_MINOR;
+        wdev->omap_wdt_miscdev.name = "watchdog";
+        wdev->omap_wdt_miscdev.fops = &omap_wdt_fops;
+
+        ret = misc_register(&(wdev->omap_wdt_miscdev));
         if (ret)
-                goto fail;
+                goto err_misc;
 
-        pr_info("OMAP Watchdog Timer: initial timeout %d sec\n", timer_margin);
+        pr_info("OMAP Watchdog Timer Rev 0x%02x: initial timeout %d sec\n",
+                __raw_readl(wdev->base + OMAP_WATCHDOG_REV) & 0xFF,
+                timer_margin);
 
         /* autogate OCP interface clock */
-        omap_writel(0x01, OMAP_WATCHDOG_SYS_CONFIG);
+        __raw_writel(0x01, wdev->base + OMAP_WATCHDOG_SYS_CONFIG);
+
+        omap_wdt_dev = pdev;
+
         return 0;
 
-fail:
-        if (armwdt_ck)
-                clk_put(armwdt_ck);
-        if (mpu_wdt_ick)
-                clk_put(mpu_wdt_ick);
-        if (mpu_wdt_fck)
-                clk_put(mpu_wdt_fck);
-        release_resource(mem);
+err_misc:
+        platform_set_drvdata(pdev, NULL);
+        iounmap(wdev->base);
+
+err_ioremap:
+        wdev->base = NULL;
+
+err_clk:
+        if (wdev->armwdt_ck)
+                clk_put(wdev->armwdt_ck);
+        if (wdev->mpu_wdt_ick)
+                clk_put(wdev->mpu_wdt_ick);
+        if (wdev->mpu_wdt_fck)
+                clk_put(wdev->mpu_wdt_fck);
+        kfree(wdev);
+
+err_kzalloc:
+        release_mem_region(res->start, res->end - res->start + 1);
+
+err_busy:
+err_get_resource:
+
         return ret;
 }
 
 static void omap_wdt_shutdown(struct platform_device *pdev)
 {
-        omap_wdt_disable();
+        struct omap_wdt_dev *wdev = platform_get_drvdata(pdev);
+
+        if (wdev->omap_wdt_users)
+                omap_wdt_disable(wdev);
 }
 
 static int omap_wdt_remove(struct platform_device *pdev)
 {
-        struct resource *mem = platform_get_drvdata(pdev);
-        misc_deregister(&omap_wdt_miscdev);
-        release_resource(mem);
-        if (armwdt_ck)
-                clk_put(armwdt_ck);
-        if (mpu_wdt_ick)
-                clk_put(mpu_wdt_ick);
-        if (mpu_wdt_fck)
-                clk_put(mpu_wdt_fck);
+        struct omap_wdt_dev *wdev = platform_get_drvdata(pdev);
+        struct resource *res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+
+        if (!res)
+                return -ENOENT;
+
+        misc_deregister(&(wdev->omap_wdt_miscdev));
+        release_mem_region(res->start, res->end - res->start + 1);
+        platform_set_drvdata(pdev, NULL);
+
+        if (wdev->armwdt_ck) {
+                clk_put(wdev->armwdt_ck);
+                wdev->armwdt_ck = NULL;
+        }
+
+        if (wdev->mpu_wdt_ick) {
+                clk_put(wdev->mpu_wdt_ick);
+                wdev->mpu_wdt_ick = NULL;
+        }
+
+        if (wdev->mpu_wdt_fck) {
+                clk_put(wdev->mpu_wdt_fck);
+                wdev->mpu_wdt_fck = NULL;
+        }
+        iounmap(wdev->base);
+
+        kfree(wdev);
+        omap_wdt_dev = NULL;
+
         return 0;
 }
 
@@ -344,17 +449,23 @@ static int omap_wdt_remove(struct platform_device *pdev)
 
 static int omap_wdt_suspend(struct platform_device *pdev, pm_message_t state)
 {
-        if (omap_wdt_users)
-                omap_wdt_disable();
+        struct omap_wdt_dev *wdev = platform_get_drvdata(pdev);
+
+        if (wdev->omap_wdt_users)
+                omap_wdt_disable(wdev);
+
         return 0;
 }
 
 static int omap_wdt_resume(struct platform_device *pdev)
 {
-        if (omap_wdt_users) {
-                omap_wdt_enable();
-                omap_wdt_ping();
+        struct omap_wdt_dev *wdev = platform_get_drvdata(pdev);
+
+        if (wdev->omap_wdt_users) {
+                omap_wdt_enable(wdev);
+                omap_wdt_ping(wdev);
         }
+
         return 0;
 }