Commit d20a1d90 authored by Guenter Roeck's avatar Guenter Roeck

watchdog: sunxi: register restart handler with kernel restart handler

The kernel core now provides an API to trigger a system restart.  Register
with it instead of setting arm_pm_restart.
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
Acked-by: default avatarHeiko Stuebner <heiko@sntech.de>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jonas Jensen <jonas.jensen@gmail.com>
Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
Cc: Randy Dunlap <rdunlap@infradead.org>
Cc: Russell King <linux@arm.linux.org.uk>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Tomasz Figa <t.figa@samsung.com>
Cc: Will Deacon <will.deacon@arm.com>
Cc: Wim Van Sebroeck <wim@iguana.be>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 87ffc69e
...@@ -21,14 +21,13 @@ ...@@ -21,14 +21,13 @@
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/moduleparam.h> #include <linux/moduleparam.h>
#include <linux/notifier.h>
#include <linux/of.h> #include <linux/of.h>
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/reboot.h> #include <linux/reboot.h>
#include <linux/types.h> #include <linux/types.h>
#include <linux/watchdog.h> #include <linux/watchdog.h>
#include <asm/system_misc.h>
#define WDT_MAX_TIMEOUT 16 #define WDT_MAX_TIMEOUT 16
#define WDT_MIN_TIMEOUT 1 #define WDT_MIN_TIMEOUT 1
#define WDT_MODE_TIMEOUT(n) ((n) << 3) #define WDT_MODE_TIMEOUT(n) ((n) << 3)
...@@ -50,6 +49,7 @@ static unsigned int timeout = WDT_MAX_TIMEOUT; ...@@ -50,6 +49,7 @@ static unsigned int timeout = WDT_MAX_TIMEOUT;
struct sunxi_wdt_dev { struct sunxi_wdt_dev {
struct watchdog_device wdt_dev; struct watchdog_device wdt_dev;
void __iomem *wdt_base; void __iomem *wdt_base;
struct notifier_block restart_handler;
}; };
/* /*
...@@ -74,24 +74,29 @@ static const int wdt_timeout_map[] = { ...@@ -74,24 +74,29 @@ static const int wdt_timeout_map[] = {
[16] = 0xB, /* 16s */ [16] = 0xB, /* 16s */
}; };
static void __iomem *reboot_wdt_base;
static void sun4i_wdt_restart(enum reboot_mode mode, const char *cmd) static int sunxi_restart_handle(struct notifier_block *this, unsigned long mode,
void *cmd)
{ {
struct sunxi_wdt_dev *sunxi_wdt = container_of(this,
struct sunxi_wdt_dev,
restart_handler);
void __iomem *wdt_base = sunxi_wdt->wdt_base;
/* Enable timer and set reset bit in the watchdog */ /* Enable timer and set reset bit in the watchdog */
writel(WDT_MODE_EN | WDT_MODE_RST_EN, reboot_wdt_base + WDT_MODE); writel(WDT_MODE_EN | WDT_MODE_RST_EN, wdt_base + WDT_MODE);
/* /*
* Restart the watchdog. The default (and lowest) interval * Restart the watchdog. The default (and lowest) interval
* value for the watchdog is 0.5s. * value for the watchdog is 0.5s.
*/ */
writel(WDT_CTRL_RELOAD, reboot_wdt_base + WDT_CTRL); writel(WDT_CTRL_RELOAD, wdt_base + WDT_CTRL);
while (1) { while (1) {
mdelay(5); mdelay(5);
writel(WDT_MODE_EN | WDT_MODE_RST_EN, writel(WDT_MODE_EN | WDT_MODE_RST_EN, wdt_base + WDT_MODE);
reboot_wdt_base + WDT_MODE);
} }
return NOTIFY_DONE;
} }
static int sunxi_wdt_ping(struct watchdog_device *wdt_dev) static int sunxi_wdt_ping(struct watchdog_device *wdt_dev)
...@@ -205,8 +210,12 @@ static int sunxi_wdt_probe(struct platform_device *pdev) ...@@ -205,8 +210,12 @@ static int sunxi_wdt_probe(struct platform_device *pdev)
if (unlikely(err)) if (unlikely(err))
return err; return err;
reboot_wdt_base = sunxi_wdt->wdt_base; sunxi_wdt->restart_handler.notifier_call = sunxi_restart_handle;
arm_pm_restart = sun4i_wdt_restart; sunxi_wdt->restart_handler.priority = 128;
err = register_restart_handler(&sunxi_wdt->restart_handler);
if (err)
dev_err(&pdev->dev,
"cannot register restart handler (err=%d)\n", err);
dev_info(&pdev->dev, "Watchdog enabled (timeout=%d sec, nowayout=%d)", dev_info(&pdev->dev, "Watchdog enabled (timeout=%d sec, nowayout=%d)",
sunxi_wdt->wdt_dev.timeout, nowayout); sunxi_wdt->wdt_dev.timeout, nowayout);
...@@ -218,7 +227,7 @@ static int sunxi_wdt_remove(struct platform_device *pdev) ...@@ -218,7 +227,7 @@ static int sunxi_wdt_remove(struct platform_device *pdev)
{ {
struct sunxi_wdt_dev *sunxi_wdt = platform_get_drvdata(pdev); struct sunxi_wdt_dev *sunxi_wdt = platform_get_drvdata(pdev);
arm_pm_restart = NULL; unregister_restart_handler(&sunxi_wdt->restart_handler);
watchdog_unregister_device(&sunxi_wdt->wdt_dev); watchdog_unregister_device(&sunxi_wdt->wdt_dev);
watchdog_set_drvdata(&sunxi_wdt->wdt_dev, NULL); watchdog_set_drvdata(&sunxi_wdt->wdt_dev, NULL);
......
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