watchdog: sbsa: Drop status function
authorGuenter Roeck <linux@roeck-us.net>
Sun, 17 Jul 2016 22:04:12 +0000 (15:04 -0700)
committerWim Van Sebroeck <wim@iguana.be>
Tue, 19 Jul 2016 08:09:12 +0000 (10:09 +0200)
The watchdog status function is supposed to return WDIOF_ flags,
not internal status flags. The available WDIOF_ flags are now
returned by the watchdog core, so the status function in this driver
is unnecessary.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/sbsa_gwdt.c

index cc885f1..ce0c38b 100644 (file)
@@ -180,15 +180,6 @@ static int sbsa_gwdt_keepalive(struct watchdog_device *wdd)
        return 0;
 }
 
-static unsigned int sbsa_gwdt_status(struct watchdog_device *wdd)
-{
-       struct sbsa_gwdt *gwdt = watchdog_get_drvdata(wdd);
-       u32 status = readl(gwdt->control_base + SBSA_GWDT_WCS);
-
-       /* is the watchdog timer running? */
-       return (status & SBSA_GWDT_WCS_EN) << WDOG_ACTIVE;
-}
-
 static int sbsa_gwdt_start(struct watchdog_device *wdd)
 {
        struct sbsa_gwdt *gwdt = watchdog_get_drvdata(wdd);
@@ -228,7 +219,6 @@ static struct watchdog_ops sbsa_gwdt_ops = {
        .owner          = THIS_MODULE,
        .start          = sbsa_gwdt_start,
        .stop           = sbsa_gwdt_stop,
-       .status         = sbsa_gwdt_status,
        .ping           = sbsa_gwdt_keepalive,
        .set_timeout    = sbsa_gwdt_set_timeout,
        .get_timeleft   = sbsa_gwdt_get_timeleft,