From: Simon Arlott <simon@fire.lp0.eu> To: Guenter Roeck <linux@roeck-us.net>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Ralf Baechle <ralf@linux-mips.org>, Thomas Gleixner <tglx@linutronix.de>, Jason Cooper <jason@lakedaemon.net>, Marc Zyngier <marc.zyngier@arm.com>, Kevin Cernekee <cernekee@gmail.com>, Florian Fainelli <f.fainelli@gmail.com>, Wim Van Sebroeck <wim@iguana.be>, Maxime Bizon <mbizon@freebox.fr>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, linux-mips@linux-mips.org, linux-watchdog@vger.kernel.org Cc: Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, Jonas Gorski <jogo@openwrt.org> Subject: [PATCH (v2) 4/10] watchdog: bcm63xx_wdt: Handle hardware interrupt and remove software timer Date: Wed, 25 Nov 2015 22:33:54 +0000 [thread overview] Message-ID: <56563752.50807@simon.arlott.org.uk> (raw) In-Reply-To: <5651CB9C.4090005@simon.arlott.org.uk> There is a level triggered interrupt for the watchdog timer as part of the bcm63xx_timer device. The interrupt occurs when the hardware watchdog timer reaches 50% of the remaining time. It is not possible to mask the interrupt within the bcm63xx_timer device. To get around this limitation, handle the interrupt by restarting the watchdog with the current remaining time (which will be half the previous timeout) so that the interrupt occurs again at 1/4th, 1/8th, etc. of the original timeout value until the watchdog forces a reboot. The software timer was restarting the hardware watchdog with a 85 second timeout until the software timer expired, and then causing a panic() about 42.5 seconds later when the hardware interrupt occurred. The hardware watchdog would not reboot until a further 42.5 seconds had passed. Remove the software timer and rely on the hardware timer directly, reducing the maximum timeout from 256 seconds to 85 seconds (2^32 / WDT_HZ). Signed-off-by: Simon Arlott <simon@fire.lp0.eu> --- Initialise bcm63xx_wdt_device.running to false. drivers/watchdog/bcm63xx_wdt.c | 125 ++++++++++++++++++++++++----------------- 1 file changed, 73 insertions(+), 52 deletions(-) diff --git a/drivers/watchdog/bcm63xx_wdt.c b/drivers/watchdog/bcm63xx_wdt.c index ab26fd9..3f55cba 100644 --- a/drivers/watchdog/bcm63xx_wdt.c +++ b/drivers/watchdog/bcm63xx_wdt.c @@ -3,6 +3,7 @@ * * Copyright (C) 2007, Miguel Gaio <miguel.gaio@efixo.com> * Copyright (C) 2008, Florian Fainelli <florian@openwrt.org> + * Copyright 2015 Simon Arlott * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License @@ -20,11 +21,10 @@ #include <linux/miscdevice.h> #include <linux/module.h> #include <linux/moduleparam.h> +#include <linux/spinlock.h> #include <linux/types.h> #include <linux/uaccess.h> #include <linux/watchdog.h> -#include <linux/timer.h> -#include <linux/jiffies.h> #include <linux/interrupt.h> #include <linux/ptrace.h> #include <linux/resource.h> @@ -37,16 +37,17 @@ #define PFX KBUILD_MODNAME -#define WDT_HZ 50000000 /* Fclk */ -#define WDT_DEFAULT_TIME 30 /* seconds */ -#define WDT_MAX_TIME 256 /* seconds */ +#define WDT_HZ 50000000 /* Fclk */ +#define WDT_DEFAULT_TIME 30 /* seconds */ +#define WDT_MAX_TIME (0xffffffff / WDT_HZ) /* seconds */ -static struct { +struct bcm63xx_wdt_hw { + raw_spinlock_t lock; void __iomem *regs; - struct timer_list timer; unsigned long inuse; - atomic_t ticks; -} bcm63xx_wdt_device; + bool running; +}; +static struct bcm63xx_wdt_hw bcm63xx_wdt_device; static int expect_close; @@ -59,48 +60,67 @@ MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started (default=" /* HW functions */ static void bcm63xx_wdt_hw_start(void) { - bcm_writel(0xfffffffe, bcm63xx_wdt_device.regs + WDT_DEFVAL_REG); + unsigned long flags; + + raw_spin_lock_irqsave(&bcm63xx_wdt_device.lock, flags); + bcm_writel(wdt_time * WDT_HZ, bcm63xx_wdt_device.regs + WDT_DEFVAL_REG); bcm_writel(WDT_START_1, bcm63xx_wdt_device.regs + WDT_CTL_REG); bcm_writel(WDT_START_2, bcm63xx_wdt_device.regs + WDT_CTL_REG); + bcm63xx_wdt_device.running = true; + raw_spin_unlock_irqrestore(&bcm63xx_wdt_device.lock, flags); } static void bcm63xx_wdt_hw_stop(void) { + unsigned long flags; + + raw_spin_lock_irqsave(&bcm63xx_wdt_device.lock, flags); bcm_writel(WDT_STOP_1, bcm63xx_wdt_device.regs + WDT_CTL_REG); bcm_writel(WDT_STOP_2, bcm63xx_wdt_device.regs + WDT_CTL_REG); + bcm63xx_wdt_device.running = false; + raw_spin_unlock_irqrestore(&bcm63xx_wdt_device.lock, flags); } +/* The watchdog interrupt occurs when half the timeout is remaining */ static void bcm63xx_wdt_isr(void *data) { - struct pt_regs *regs = get_irq_regs(); - - die(PFX " fire", regs); -} - -static void bcm63xx_timer_tick(unsigned long unused) -{ - if (!atomic_dec_and_test(&bcm63xx_wdt_device.ticks)) { - bcm63xx_wdt_hw_start(); - mod_timer(&bcm63xx_wdt_device.timer, jiffies + HZ); - } else - pr_crit("watchdog will restart system\n"); -} - -static void bcm63xx_wdt_pet(void) -{ - atomic_set(&bcm63xx_wdt_device.ticks, wdt_time); -} - -static void bcm63xx_wdt_start(void) -{ - bcm63xx_wdt_pet(); - bcm63xx_timer_tick(0); -} + struct bcm63xx_wdt_hw *hw = &bcm63xx_wdt_device; + unsigned long flags; + + raw_spin_lock_irqsave(&hw->lock, flags); + if (!hw->running) { + /* Stop the watchdog as it shouldn't be running */ + bcm_writel(WDT_STOP_1, hw->regs + WDT_CTL_REG); + bcm_writel(WDT_STOP_2, hw->regs + WDT_CTL_REG); + } else { + u32 timeleft = bcm_readl(hw->regs + WDT_CTL_REG); + u32 ms; + + if (timeleft >= 2) { + /* The only way to clear this level triggered interrupt + * without disrupting the normal running of the watchdog + * is to restart the watchdog with the current remaining + * time value (which will be half the previous timeout) + * so the interrupt occurs again at 1/4th, 1/8th, etc. + * of the original timeout value until we reboot. + * + * This is done with a lock held in case userspace is + * trying to restart the watchdog on another CPU. + */ + bcm_writel(timeleft, hw->regs + WDT_DEFVAL_REG); + bcm_writel(WDT_START_1, hw->regs + WDT_CTL_REG); + bcm_writel(WDT_START_2, hw->regs + WDT_CTL_REG); + } else { + /* The watchdog cannot be started with a time of less + * than 2 ticks (it won't fire). + */ + die(PFX ": watchdog timer expired\n", get_irq_regs()); + } -static void bcm63xx_wdt_pause(void) -{ - del_timer_sync(&bcm63xx_wdt_device.timer); - bcm63xx_wdt_hw_stop(); + ms = timeleft / (WDT_HZ / 1000); + pr_alert("warning timer fired, reboot in %ums\n", ms); + } + raw_spin_unlock_irqrestore(&hw->lock, flags); } static int bcm63xx_wdt_settimeout(int new_time) @@ -118,17 +138,17 @@ static int bcm63xx_wdt_open(struct inode *inode, struct file *file) if (test_and_set_bit(0, &bcm63xx_wdt_device.inuse)) return -EBUSY; - bcm63xx_wdt_start(); + bcm63xx_wdt_hw_start(); return nonseekable_open(inode, file); } static int bcm63xx_wdt_release(struct inode *inode, struct file *file) { if (expect_close == 42) - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); else { pr_crit("Unexpected close, not stopping watchdog!\n"); - bcm63xx_wdt_start(); + bcm63xx_wdt_hw_start(); } clear_bit(0, &bcm63xx_wdt_device.inuse); expect_close = 0; @@ -153,7 +173,7 @@ static ssize_t bcm63xx_wdt_write(struct file *file, const char *data, expect_close = 42; } } - bcm63xx_wdt_pet(); + bcm63xx_wdt_hw_start(); } return len; } @@ -187,18 +207,18 @@ static long bcm63xx_wdt_ioctl(struct file *file, unsigned int cmd, return -EFAULT; if (new_value & WDIOS_DISABLECARD) { - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); retval = 0; } if (new_value & WDIOS_ENABLECARD) { - bcm63xx_wdt_start(); + bcm63xx_wdt_hw_start(); retval = 0; } return retval; case WDIOC_KEEPALIVE: - bcm63xx_wdt_pet(); + bcm63xx_wdt_hw_start(); return 0; case WDIOC_SETTIMEOUT: @@ -208,7 +228,7 @@ static long bcm63xx_wdt_ioctl(struct file *file, unsigned int cmd, if (bcm63xx_wdt_settimeout(new_value)) return -EINVAL; - bcm63xx_wdt_pet(); + bcm63xx_wdt_hw_start(); case WDIOC_GETTIMEOUT: return put_user(wdt_time, p); @@ -240,8 +260,6 @@ static int bcm63xx_wdt_probe(struct platform_device *pdev) int ret; struct resource *r; - setup_timer(&bcm63xx_wdt_device.timer, bcm63xx_timer_tick, 0L); - r = platform_get_resource(pdev, IORESOURCE_MEM, 0); if (!r) { dev_err(&pdev->dev, "failed to get resources\n"); @@ -255,6 +273,9 @@ static int bcm63xx_wdt_probe(struct platform_device *pdev) return -ENXIO; } + raw_spin_lock_init(&bcm63xx_wdt_device.lock); + bcm63xx_wdt_device.running = false; + ret = bcm63xx_timer_register(TIMER_WDT_ID, bcm63xx_wdt_isr, NULL); if (ret < 0) { dev_err(&pdev->dev, "failed to register wdt timer isr\n"); @@ -264,8 +285,8 @@ static int bcm63xx_wdt_probe(struct platform_device *pdev) if (bcm63xx_wdt_settimeout(wdt_time)) { bcm63xx_wdt_settimeout(WDT_DEFAULT_TIME); dev_info(&pdev->dev, - ": wdt_time value must be 1 <= wdt_time <= 256, using %d\n", - wdt_time); + ": wdt_time value must be 1 <= wdt_time <= %d, using %d\n", + WDT_MAX_TIME, wdt_time); } ret = misc_register(&bcm63xx_wdt_miscdev); @@ -287,7 +308,7 @@ unregister_timer: static int bcm63xx_wdt_remove(struct platform_device *pdev) { if (!nowayout) - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); misc_deregister(&bcm63xx_wdt_miscdev); bcm63xx_timer_unregister(TIMER_WDT_ID); @@ -296,7 +317,7 @@ static int bcm63xx_wdt_remove(struct platform_device *pdev) static void bcm63xx_wdt_shutdown(struct platform_device *pdev) { - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); } static struct platform_driver bcm63xx_wdt_driver = { -- 2.1.4 -- Simon Arlott
WARNING: multiple messages have this Message-ID (diff)
From: Simon Arlott <simon-A6De1vDTPLDsq35pWSNszA@public.gmane.org> To: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>, "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, Ralf Baechle <ralf-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org>, Thomas Gleixner <tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org>, Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>, Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org>, Kevin Cernekee <cernekee-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Wim Van Sebroeck <wim-IQzOog9fTRqzQB+pC5nmwQ@public.gmane.org>, Maxime Bizon <mbizon-MmRyKUhfbQ9GWvitb5QawA@public.gmane.org>, Linux Kernel Mailing List <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, linux-mips-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org, linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>, Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>, Ian Campbell <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>, Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>, Jonas Gorski <jogo-p3rKhJxN3npAfugRpC6u6w@public.gmane.org> Subject: [PATCH (v2) 4/10] watchdog: bcm63xx_wdt: Handle hardware interrupt and remove software timer Date: Wed, 25 Nov 2015 22:33:54 +0000 [thread overview] Message-ID: <56563752.50807@simon.arlott.org.uk> (raw) In-Reply-To: <5651CB9C.4090005-qdVf85lJwsCyrPCCpiK2c/XRex20P6io@public.gmane.org> There is a level triggered interrupt for the watchdog timer as part of the bcm63xx_timer device. The interrupt occurs when the hardware watchdog timer reaches 50% of the remaining time. It is not possible to mask the interrupt within the bcm63xx_timer device. To get around this limitation, handle the interrupt by restarting the watchdog with the current remaining time (which will be half the previous timeout) so that the interrupt occurs again at 1/4th, 1/8th, etc. of the original timeout value until the watchdog forces a reboot. The software timer was restarting the hardware watchdog with a 85 second timeout until the software timer expired, and then causing a panic() about 42.5 seconds later when the hardware interrupt occurred. The hardware watchdog would not reboot until a further 42.5 seconds had passed. Remove the software timer and rely on the hardware timer directly, reducing the maximum timeout from 256 seconds to 85 seconds (2^32 / WDT_HZ). Signed-off-by: Simon Arlott <simon-A6De1vDTPLDsq35pWSNszA@public.gmane.org> --- Initialise bcm63xx_wdt_device.running to false. drivers/watchdog/bcm63xx_wdt.c | 125 ++++++++++++++++++++++++----------------- 1 file changed, 73 insertions(+), 52 deletions(-) diff --git a/drivers/watchdog/bcm63xx_wdt.c b/drivers/watchdog/bcm63xx_wdt.c index ab26fd9..3f55cba 100644 --- a/drivers/watchdog/bcm63xx_wdt.c +++ b/drivers/watchdog/bcm63xx_wdt.c @@ -3,6 +3,7 @@ * * Copyright (C) 2007, Miguel Gaio <miguel.gaio-HH44TBFINEIAvxtiuMwx3w@public.gmane.org> * Copyright (C) 2008, Florian Fainelli <florian-p3rKhJxN3npAfugRpC6u6w@public.gmane.org> + * Copyright 2015 Simon Arlott * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License @@ -20,11 +21,10 @@ #include <linux/miscdevice.h> #include <linux/module.h> #include <linux/moduleparam.h> +#include <linux/spinlock.h> #include <linux/types.h> #include <linux/uaccess.h> #include <linux/watchdog.h> -#include <linux/timer.h> -#include <linux/jiffies.h> #include <linux/interrupt.h> #include <linux/ptrace.h> #include <linux/resource.h> @@ -37,16 +37,17 @@ #define PFX KBUILD_MODNAME -#define WDT_HZ 50000000 /* Fclk */ -#define WDT_DEFAULT_TIME 30 /* seconds */ -#define WDT_MAX_TIME 256 /* seconds */ +#define WDT_HZ 50000000 /* Fclk */ +#define WDT_DEFAULT_TIME 30 /* seconds */ +#define WDT_MAX_TIME (0xffffffff / WDT_HZ) /* seconds */ -static struct { +struct bcm63xx_wdt_hw { + raw_spinlock_t lock; void __iomem *regs; - struct timer_list timer; unsigned long inuse; - atomic_t ticks; -} bcm63xx_wdt_device; + bool running; +}; +static struct bcm63xx_wdt_hw bcm63xx_wdt_device; static int expect_close; @@ -59,48 +60,67 @@ MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started (default=" /* HW functions */ static void bcm63xx_wdt_hw_start(void) { - bcm_writel(0xfffffffe, bcm63xx_wdt_device.regs + WDT_DEFVAL_REG); + unsigned long flags; + + raw_spin_lock_irqsave(&bcm63xx_wdt_device.lock, flags); + bcm_writel(wdt_time * WDT_HZ, bcm63xx_wdt_device.regs + WDT_DEFVAL_REG); bcm_writel(WDT_START_1, bcm63xx_wdt_device.regs + WDT_CTL_REG); bcm_writel(WDT_START_2, bcm63xx_wdt_device.regs + WDT_CTL_REG); + bcm63xx_wdt_device.running = true; + raw_spin_unlock_irqrestore(&bcm63xx_wdt_device.lock, flags); } static void bcm63xx_wdt_hw_stop(void) { + unsigned long flags; + + raw_spin_lock_irqsave(&bcm63xx_wdt_device.lock, flags); bcm_writel(WDT_STOP_1, bcm63xx_wdt_device.regs + WDT_CTL_REG); bcm_writel(WDT_STOP_2, bcm63xx_wdt_device.regs + WDT_CTL_REG); + bcm63xx_wdt_device.running = false; + raw_spin_unlock_irqrestore(&bcm63xx_wdt_device.lock, flags); } +/* The watchdog interrupt occurs when half the timeout is remaining */ static void bcm63xx_wdt_isr(void *data) { - struct pt_regs *regs = get_irq_regs(); - - die(PFX " fire", regs); -} - -static void bcm63xx_timer_tick(unsigned long unused) -{ - if (!atomic_dec_and_test(&bcm63xx_wdt_device.ticks)) { - bcm63xx_wdt_hw_start(); - mod_timer(&bcm63xx_wdt_device.timer, jiffies + HZ); - } else - pr_crit("watchdog will restart system\n"); -} - -static void bcm63xx_wdt_pet(void) -{ - atomic_set(&bcm63xx_wdt_device.ticks, wdt_time); -} - -static void bcm63xx_wdt_start(void) -{ - bcm63xx_wdt_pet(); - bcm63xx_timer_tick(0); -} + struct bcm63xx_wdt_hw *hw = &bcm63xx_wdt_device; + unsigned long flags; + + raw_spin_lock_irqsave(&hw->lock, flags); + if (!hw->running) { + /* Stop the watchdog as it shouldn't be running */ + bcm_writel(WDT_STOP_1, hw->regs + WDT_CTL_REG); + bcm_writel(WDT_STOP_2, hw->regs + WDT_CTL_REG); + } else { + u32 timeleft = bcm_readl(hw->regs + WDT_CTL_REG); + u32 ms; + + if (timeleft >= 2) { + /* The only way to clear this level triggered interrupt + * without disrupting the normal running of the watchdog + * is to restart the watchdog with the current remaining + * time value (which will be half the previous timeout) + * so the interrupt occurs again at 1/4th, 1/8th, etc. + * of the original timeout value until we reboot. + * + * This is done with a lock held in case userspace is + * trying to restart the watchdog on another CPU. + */ + bcm_writel(timeleft, hw->regs + WDT_DEFVAL_REG); + bcm_writel(WDT_START_1, hw->regs + WDT_CTL_REG); + bcm_writel(WDT_START_2, hw->regs + WDT_CTL_REG); + } else { + /* The watchdog cannot be started with a time of less + * than 2 ticks (it won't fire). + */ + die(PFX ": watchdog timer expired\n", get_irq_regs()); + } -static void bcm63xx_wdt_pause(void) -{ - del_timer_sync(&bcm63xx_wdt_device.timer); - bcm63xx_wdt_hw_stop(); + ms = timeleft / (WDT_HZ / 1000); + pr_alert("warning timer fired, reboot in %ums\n", ms); + } + raw_spin_unlock_irqrestore(&hw->lock, flags); } static int bcm63xx_wdt_settimeout(int new_time) @@ -118,17 +138,17 @@ static int bcm63xx_wdt_open(struct inode *inode, struct file *file) if (test_and_set_bit(0, &bcm63xx_wdt_device.inuse)) return -EBUSY; - bcm63xx_wdt_start(); + bcm63xx_wdt_hw_start(); return nonseekable_open(inode, file); } static int bcm63xx_wdt_release(struct inode *inode, struct file *file) { if (expect_close == 42) - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); else { pr_crit("Unexpected close, not stopping watchdog!\n"); - bcm63xx_wdt_start(); + bcm63xx_wdt_hw_start(); } clear_bit(0, &bcm63xx_wdt_device.inuse); expect_close = 0; @@ -153,7 +173,7 @@ static ssize_t bcm63xx_wdt_write(struct file *file, const char *data, expect_close = 42; } } - bcm63xx_wdt_pet(); + bcm63xx_wdt_hw_start(); } return len; } @@ -187,18 +207,18 @@ static long bcm63xx_wdt_ioctl(struct file *file, unsigned int cmd, return -EFAULT; if (new_value & WDIOS_DISABLECARD) { - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); retval = 0; } if (new_value & WDIOS_ENABLECARD) { - bcm63xx_wdt_start(); + bcm63xx_wdt_hw_start(); retval = 0; } return retval; case WDIOC_KEEPALIVE: - bcm63xx_wdt_pet(); + bcm63xx_wdt_hw_start(); return 0; case WDIOC_SETTIMEOUT: @@ -208,7 +228,7 @@ static long bcm63xx_wdt_ioctl(struct file *file, unsigned int cmd, if (bcm63xx_wdt_settimeout(new_value)) return -EINVAL; - bcm63xx_wdt_pet(); + bcm63xx_wdt_hw_start(); case WDIOC_GETTIMEOUT: return put_user(wdt_time, p); @@ -240,8 +260,6 @@ static int bcm63xx_wdt_probe(struct platform_device *pdev) int ret; struct resource *r; - setup_timer(&bcm63xx_wdt_device.timer, bcm63xx_timer_tick, 0L); - r = platform_get_resource(pdev, IORESOURCE_MEM, 0); if (!r) { dev_err(&pdev->dev, "failed to get resources\n"); @@ -255,6 +273,9 @@ static int bcm63xx_wdt_probe(struct platform_device *pdev) return -ENXIO; } + raw_spin_lock_init(&bcm63xx_wdt_device.lock); + bcm63xx_wdt_device.running = false; + ret = bcm63xx_timer_register(TIMER_WDT_ID, bcm63xx_wdt_isr, NULL); if (ret < 0) { dev_err(&pdev->dev, "failed to register wdt timer isr\n"); @@ -264,8 +285,8 @@ static int bcm63xx_wdt_probe(struct platform_device *pdev) if (bcm63xx_wdt_settimeout(wdt_time)) { bcm63xx_wdt_settimeout(WDT_DEFAULT_TIME); dev_info(&pdev->dev, - ": wdt_time value must be 1 <= wdt_time <= 256, using %d\n", - wdt_time); + ": wdt_time value must be 1 <= wdt_time <= %d, using %d\n", + WDT_MAX_TIME, wdt_time); } ret = misc_register(&bcm63xx_wdt_miscdev); @@ -287,7 +308,7 @@ unregister_timer: static int bcm63xx_wdt_remove(struct platform_device *pdev) { if (!nowayout) - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); misc_deregister(&bcm63xx_wdt_miscdev); bcm63xx_timer_unregister(TIMER_WDT_ID); @@ -296,7 +317,7 @@ static int bcm63xx_wdt_remove(struct platform_device *pdev) static void bcm63xx_wdt_shutdown(struct platform_device *pdev) { - bcm63xx_wdt_pause(); + bcm63xx_wdt_hw_stop(); } static struct platform_driver bcm63xx_wdt_driver = { -- 2.1.4 -- Simon Arlott -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-11-25 22:34 UTC|newest] Thread overview: 95+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-21 19:02 [PATCH 1/4] clocksource: Add brcm,bcm6345-timer device tree binding Simon Arlott 2015-11-21 19:02 ` Simon Arlott 2015-11-21 19:03 ` [PATCH 2/4] MIPS: bmips: Add bcm6345-l2-timer interrupt controller Simon Arlott 2015-11-21 19:04 ` [PATCH 3/4] watchdog: Add brcm,bcm6345-wdt device tree binding Simon Arlott 2015-11-22 22:13 ` Rob Herring 2015-11-22 22:13 ` Rob Herring 2015-11-21 19:05 ` [PATCH 4/4] MIPS: bmips: Convert bcm63xx_wdt to use WATCHDOG_CORE Simon Arlott 2015-11-21 19:05 ` Simon Arlott 2015-11-21 21:32 ` Guenter Roeck 2015-11-21 21:44 ` Simon Arlott 2015-11-21 21:44 ` Simon Arlott 2015-11-22 2:32 ` Guenter Roeck 2015-11-22 14:02 ` [PATCH 4/10] (Was: [PATCH 4/4]) " Simon Arlott 2015-11-22 14:05 ` [PATCH 4/10] watchdog: bcm63xx_wdt: Handle hardware interrupt and remove software timer Simon Arlott 2015-11-24 18:21 ` Guenter Roeck 2015-11-24 18:21 ` Guenter Roeck 2015-11-24 18:21 ` Guenter Roeck 2015-11-25 20:14 ` Jonas Gorski 2015-11-25 20:14 ` Jonas Gorski 2015-11-25 20:28 ` Simon Arlott 2015-11-25 20:28 ` Simon Arlott 2015-11-25 22:33 ` Simon Arlott [this message] 2015-11-25 22:33 ` [PATCH (v2) " Simon Arlott 2015-11-22 14:06 ` [PATCH 5/10] watchdog: bcm63xx_wdt: Use WATCHDOG_CORE Simon Arlott 2015-11-22 14:06 ` Simon Arlott 2015-11-25 2:44 ` Guenter Roeck 2015-11-25 2:44 ` Guenter Roeck 2015-11-25 13:02 ` Simon Arlott 2015-11-25 14:10 ` Guenter Roeck 2015-11-25 14:10 ` Guenter Roeck 2015-11-25 19:43 ` Simon Arlott 2015-11-25 19:43 ` Simon Arlott 2015-11-25 22:40 ` [PATCH (v3) 5/11] " Simon Arlott 2015-11-25 22:40 ` Simon Arlott 2015-11-22 14:07 ` [PATCH 6/10] watchdog: bcm63xx_wdt: Obtain watchdog clock HZ from "periph" clk Simon Arlott 2015-11-22 14:07 ` Simon Arlott 2015-11-23 15:02 ` Jonas Gorski 2015-11-23 15:02 ` Jonas Gorski 2015-11-23 18:19 ` Florian Fainelli 2015-11-23 18:19 ` Florian Fainelli 2015-11-23 19:00 ` Simon Arlott 2015-11-23 19:00 ` Simon Arlott 2015-11-24 22:12 ` [PATCH (v2) " Simon Arlott 2015-11-24 22:42 ` Florian Fainelli 2015-11-24 22:42 ` Florian Fainelli 2015-11-25 22:47 ` [PATCH (v3) 6/11] " Simon Arlott 2015-11-25 22:47 ` Simon Arlott 2015-11-22 14:09 ` [PATCH 7/10] watchdog: bcm63xx_wdt: Add get_timeleft function Simon Arlott 2015-11-22 14:09 ` Simon Arlott 2015-11-24 22:15 ` [PATCH (v2) " Simon Arlott 2015-11-24 22:15 ` Simon Arlott 2015-11-24 22:43 ` Florian Fainelli 2015-11-24 22:43 ` Florian Fainelli 2015-11-25 2:51 ` Guenter Roeck 2015-11-25 2:51 ` Guenter Roeck 2015-11-25 8:17 ` Simon Arlott 2015-11-25 8:17 ` Simon Arlott 2015-11-25 22:50 ` [PATCH (v3) 7/11] " Simon Arlott 2015-11-25 22:50 ` Simon Arlott 2015-11-25 22:54 ` [PATCH (v4) " Simon Arlott 2015-11-25 22:54 ` Simon Arlott 2015-11-25 22:57 ` [PATCH (v4) 8/11] watchdog: bcm63xx_wdt: Warn if the watchdog is currently running Simon Arlott 2015-11-25 22:57 ` Simon Arlott 2015-11-22 14:11 ` [PATCH 8/10] watchdog: bcm63xx_wdt: Remove dependency on mach-bcm63xx functions/defines Simon Arlott 2015-11-22 14:11 ` Simon Arlott 2015-11-22 14:12 ` [PATCH 9/10] watchdog: bcm63xx_wdt: Use bcm63xx_timer interrupt directly Simon Arlott 2015-11-22 14:12 ` Simon Arlott 2015-11-25 23:03 ` [PATCH (v2) 10/11] " Simon Arlott 2015-11-25 23:03 ` Simon Arlott 2015-11-22 14:14 ` [PATCH 10/10] watchdog: bcm63xx_wdt: Use brcm,bcm6345-wdt device tree binding Simon Arlott 2015-11-22 14:14 ` Simon Arlott 2015-11-25 23:09 ` [PATCH (v2) 11/11] " Simon Arlott 2015-11-25 23:09 ` Simon Arlott 2015-11-22 22:12 ` [PATCH 1/4] clocksource: Add brcm,bcm6345-timer " Rob Herring 2015-11-23 15:33 ` Jonas Gorski 2015-11-23 18:55 ` [PATCH (v2) 1/10] " Simon Arlott 2015-11-23 18:55 ` Simon Arlott 2015-11-23 18:57 ` [PATCH (v2) 2/10] MIPS: bmips: Add bcm6345-l2-timer interrupt controller Simon Arlott 2015-11-23 18:57 ` Simon Arlott 2015-11-24 22:10 ` [PATCH (v3) " Simon Arlott 2015-11-24 22:10 ` Simon Arlott 2015-11-24 22:36 ` Florian Fainelli 2015-11-24 22:36 ` Florian Fainelli 2015-11-26 22:32 ` [PATCH (v4) 2/11] " Simon Arlott 2015-11-27 8:37 ` Thomas Gleixner 2015-11-27 8:37 ` Thomas Gleixner 2015-11-28 12:26 ` [PATCH (v5) 3/11] " Simon Arlott 2015-11-28 12:26 ` Simon Arlott 2015-12-01 0:22 ` Guenter Roeck 2015-12-01 0:22 ` Guenter Roeck 2016-05-09 12:01 ` Álvaro Fernández Rojas 2016-05-09 13:06 ` Guenter Roeck 2016-05-09 13:06 ` Guenter Roeck 2016-05-11 6:40 ` [PATCH 2/4] " Álvaro Fernández Rojas 2015-11-25 3:05 ` [PATCH (v2) 1/10] clocksource: Add brcm,bcm6345-timer device tree binding Rob Herring
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=56563752.50807@simon.arlott.org.uk \ --to=simon@fire.lp0.eu \ --cc=cernekee@gmail.com \ --cc=devicetree@vger.kernel.org \ --cc=f.fainelli@gmail.com \ --cc=galak@codeaurora.org \ --cc=ijc+devicetree@hellion.org.uk \ --cc=jason@lakedaemon.net \ --cc=jogo@openwrt.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mips@linux-mips.org \ --cc=linux-watchdog@vger.kernel.org \ --cc=linux@roeck-us.net \ --cc=marc.zyngier@arm.com \ --cc=mark.rutland@arm.com \ --cc=mbizon@freebox.fr \ --cc=pawel.moll@arm.com \ --cc=ralf@linux-mips.org \ --cc=robh+dt@kernel.org \ --cc=tglx@linutronix.de \ --cc=wim@iguana.be \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.