watchdog: bcm47xx_wdt: use core reboot notifier
authorDamien Riegel <damien.riegel@savoirfairelinux.com>
Fri, 20 Nov 2015 21:54:52 +0000 (16:54 -0500)
committerWim Van Sebroeck <wim@iguana.be>
Sun, 13 Dec 2015 14:55:55 +0000 (15:55 +0100)
Get rid of the custom reboot notifier block registration and use the one
provided by the watchdog core.

Signed-off-by: Damien Riegel <damien.riegel@savoirfairelinux.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Vivien Didelot <vivien.didelot@savoirlinux.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/bcm47xx_wdt.c
include/linux/bcm47xx_wdt.h

index 01bffe174ff9df333e09a13e33738ae848367493..df1c2a4b01650a182fac16cfe2fef1e2a2c778ea 100644 (file)
@@ -20,7 +20,6 @@
 #include <linux/module.h>
 #include <linux/moduleparam.h>
 #include <linux/platform_device.h>
-#include <linux/reboot.h>
 #include <linux/types.h>
 #include <linux/watchdog.h>
 #include <linux/timer.h>
@@ -168,17 +167,6 @@ static const struct watchdog_info bcm47xx_wdt_info = {
                                WDIOF_MAGICCLOSE,
 };
 
-static int bcm47xx_wdt_notify_sys(struct notifier_block *this,
-                                 unsigned long code, void *unused)
-{
-       struct bcm47xx_wdt *wdt;
-
-       wdt = container_of(this, struct bcm47xx_wdt, notifier);
-       if (code == SYS_DOWN || code == SYS_HALT)
-               wdt->wdd.ops->stop(&wdt->wdd);
-       return NOTIFY_DONE;
-}
-
 static struct watchdog_ops bcm47xx_wdt_soft_ops = {
        .owner          = THIS_MODULE,
        .start          = bcm47xx_wdt_soft_start,
@@ -215,24 +203,17 @@ static int bcm47xx_wdt_probe(struct platform_device *pdev)
                goto err_timer;
        watchdog_set_nowayout(&wdt->wdd, nowayout);
        watchdog_set_restart_priority(&wdt->wdd, 64);
-
-       wdt->notifier.notifier_call = &bcm47xx_wdt_notify_sys;
-
-       ret = register_reboot_notifier(&wdt->notifier);
-       if (ret)
-               goto err_timer;
+       watchdog_stop_on_reboot(&wdt->wdd);
 
        ret = watchdog_register_device(&wdt->wdd);
        if (ret)
-               goto err_notifier;
+               goto err_timer;
 
        dev_info(&pdev->dev, "BCM47xx Watchdog Timer enabled (%d seconds%s%s)\n",
                timeout, nowayout ? ", nowayout" : "",
                soft ? ", Software Timer" : "");
        return 0;
 
-err_notifier:
-       unregister_reboot_notifier(&wdt->notifier);
 err_timer:
        if (soft)
                del_timer_sync(&wdt->soft_timer);
@@ -248,7 +229,6 @@ static int bcm47xx_wdt_remove(struct platform_device *pdev)
                return -ENXIO;
 
        watchdog_unregister_device(&wdt->wdd);
-       unregister_reboot_notifier(&wdt->notifier);
 
        return 0;
 }
index b708786d4cbf6d374bdff697eac61e623d427723..8d9d07ec22a57938bb188ff9be036ca31a7e881a 100644 (file)
@@ -1,7 +1,6 @@
 #ifndef LINUX_BCM47XX_WDT_H_
 #define LINUX_BCM47XX_WDT_H_
 
-#include <linux/notifier.h>
 #include <linux/timer.h>
 #include <linux/types.h>
 #include <linux/watchdog.h>
@@ -15,7 +14,6 @@ struct bcm47xx_wdt {
        void *driver_data;
 
        struct watchdog_device wdd;
-       struct notifier_block notifier;
 
        struct timer_list soft_timer;
        atomic_t soft_ticks;