watchdog: bcm2835_wdt: remove redundant ->set_timeout callback
authorRasmus Villemoes <rasmus.villemoes@prevas.dk>
Fri, 15 Jul 2016 08:15:22 +0000 (10:15 +0200)
committerWim Van Sebroeck <wim@iguana.be>
Sun, 17 Jul 2016 19:03:50 +0000 (21:03 +0200)
bcm2835_wdt_set_timeout does exactly what the watchdog framework does
in the absence of a ->set_timeout callback (see watchdog_set_timeout
in watchdog_dev.c), so remove it.

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/bcm2835_wdt.c

index 733e402..4dddd82 100644 (file)
@@ -82,12 +82,6 @@ static int bcm2835_wdt_stop(struct watchdog_device *wdog)
        return 0;
 }
 
-static int bcm2835_wdt_set_timeout(struct watchdog_device *wdog, unsigned int t)
-{
-       wdog->timeout = t;
-       return 0;
-}
-
 static unsigned int bcm2835_wdt_get_timeleft(struct watchdog_device *wdog)
 {
        struct bcm2835_wdt *wdt = watchdog_get_drvdata(wdog);
@@ -100,7 +94,6 @@ static const struct watchdog_ops bcm2835_wdt_ops = {
        .owner =        THIS_MODULE,
        .start =        bcm2835_wdt_start,
        .stop =         bcm2835_wdt_stop,
-       .set_timeout =  bcm2835_wdt_set_timeout,
        .get_timeleft = bcm2835_wdt_get_timeleft,
 };