Commit a3906892 authored by Hauke Mehrtens's avatar Hauke Mehrtens Committed by Wim Van Sebroeck

watchdog: bcm47xx_wdt.c: rename ops methods

Rename the methods registered to struct watchdog_ops bcm47xx_wdt_ops in
order to add an other struct watchdog_ops using different ops in the
next patch.
Also rename WDT_MAX_TIME to WDT_SOFTTIMER_MAX.
Signed-off-by: default avatarHauke Mehrtens <hauke@hauke-m.de>
Signed-off-by: default avatarWim Van Sebroeck <wim@iguana.be>
parent f82dedf8
...@@ -30,7 +30,7 @@ ...@@ -30,7 +30,7 @@
#define DRV_NAME "bcm47xx_wdt" #define DRV_NAME "bcm47xx_wdt"
#define WDT_DEFAULT_TIME 30 /* seconds */ #define WDT_DEFAULT_TIME 30 /* seconds */
#define WDT_MAX_TIME 255 /* seconds */ #define WDT_SOFTTIMER_MAX 255 /* seconds */
static int wdt_time = WDT_DEFAULT_TIME; static int wdt_time = WDT_DEFAULT_TIME;
static bool nowayout = WATCHDOG_NOWAYOUT; static bool nowayout = WATCHDOG_NOWAYOUT;
...@@ -49,7 +49,7 @@ static inline struct bcm47xx_wdt *bcm47xx_wdt_get(struct watchdog_device *wdd) ...@@ -49,7 +49,7 @@ static inline struct bcm47xx_wdt *bcm47xx_wdt_get(struct watchdog_device *wdd)
return container_of(wdd, struct bcm47xx_wdt, wdd); return container_of(wdd, struct bcm47xx_wdt, wdd);
} }
static void bcm47xx_timer_tick(unsigned long data) static void bcm47xx_wdt_soft_timer_tick(unsigned long data)
{ {
struct bcm47xx_wdt *wdt = (struct bcm47xx_wdt *)data; struct bcm47xx_wdt *wdt = (struct bcm47xx_wdt *)data;
u32 next_tick = min(wdt->wdd.timeout * 1000, wdt->max_timer_ms); u32 next_tick = min(wdt->wdd.timeout * 1000, wdt->max_timer_ms);
...@@ -62,7 +62,7 @@ static void bcm47xx_timer_tick(unsigned long data) ...@@ -62,7 +62,7 @@ static void bcm47xx_timer_tick(unsigned long data)
} }
} }
static int bcm47xx_wdt_keepalive(struct watchdog_device *wdd) static int bcm47xx_wdt_soft_keepalive(struct watchdog_device *wdd)
{ {
struct bcm47xx_wdt *wdt = bcm47xx_wdt_get(wdd); struct bcm47xx_wdt *wdt = bcm47xx_wdt_get(wdd);
...@@ -71,17 +71,17 @@ static int bcm47xx_wdt_keepalive(struct watchdog_device *wdd) ...@@ -71,17 +71,17 @@ static int bcm47xx_wdt_keepalive(struct watchdog_device *wdd)
return 0; return 0;
} }
static int bcm47xx_wdt_start(struct watchdog_device *wdd) static int bcm47xx_wdt_soft_start(struct watchdog_device *wdd)
{ {
struct bcm47xx_wdt *wdt = bcm47xx_wdt_get(wdd); struct bcm47xx_wdt *wdt = bcm47xx_wdt_get(wdd);
bcm47xx_wdt_keepalive(wdd); bcm47xx_wdt_soft_keepalive(wdd);
bcm47xx_timer_tick((unsigned long)wdt); bcm47xx_wdt_soft_timer_tick((unsigned long)wdt);
return 0; return 0;
} }
static int bcm47xx_wdt_stop(struct watchdog_device *wdd) static int bcm47xx_wdt_soft_stop(struct watchdog_device *wdd)
{ {
struct bcm47xx_wdt *wdt = bcm47xx_wdt_get(wdd); struct bcm47xx_wdt *wdt = bcm47xx_wdt_get(wdd);
...@@ -91,12 +91,12 @@ static int bcm47xx_wdt_stop(struct watchdog_device *wdd) ...@@ -91,12 +91,12 @@ static int bcm47xx_wdt_stop(struct watchdog_device *wdd)
return 0; return 0;
} }
static int bcm47xx_wdt_set_timeout(struct watchdog_device *wdd, static int bcm47xx_wdt_soft_set_timeout(struct watchdog_device *wdd,
unsigned int new_time) unsigned int new_time)
{ {
if (new_time < 1 || new_time > WDT_MAX_TIME) { if (new_time < 1 || new_time > WDT_SOFTTIMER_MAX) {
pr_warn("timeout value must be 1<=x<=%d, using %d\n", pr_warn("timeout value must be 1<=x<=%d, using %d\n",
WDT_MAX_TIME, new_time); WDT_SOFTTIMER_MAX, new_time);
return -EINVAL; return -EINVAL;
} }
...@@ -122,12 +122,12 @@ static int bcm47xx_wdt_notify_sys(struct notifier_block *this, ...@@ -122,12 +122,12 @@ static int bcm47xx_wdt_notify_sys(struct notifier_block *this,
return NOTIFY_DONE; return NOTIFY_DONE;
} }
static struct watchdog_ops bcm47xx_wdt_ops = { static struct watchdog_ops bcm47xx_wdt_soft_ops = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
.start = bcm47xx_wdt_start, .start = bcm47xx_wdt_soft_start,
.stop = bcm47xx_wdt_stop, .stop = bcm47xx_wdt_soft_stop,
.ping = bcm47xx_wdt_keepalive, .ping = bcm47xx_wdt_soft_keepalive,
.set_timeout = bcm47xx_wdt_set_timeout, .set_timeout = bcm47xx_wdt_soft_set_timeout,
}; };
static int bcm47xx_wdt_probe(struct platform_device *pdev) static int bcm47xx_wdt_probe(struct platform_device *pdev)
...@@ -138,10 +138,10 @@ static int bcm47xx_wdt_probe(struct platform_device *pdev) ...@@ -138,10 +138,10 @@ static int bcm47xx_wdt_probe(struct platform_device *pdev)
if (!wdt) if (!wdt)
return -ENXIO; return -ENXIO;
setup_timer(&wdt->soft_timer, bcm47xx_timer_tick, setup_timer(&wdt->soft_timer, bcm47xx_wdt_soft_timer_tick,
(long unsigned int)wdt); (long unsigned int)wdt);
wdt->wdd.ops = &bcm47xx_wdt_ops; wdt->wdd.ops = &bcm47xx_wdt_soft_ops;
wdt->wdd.info = &bcm47xx_wdt_info; wdt->wdd.info = &bcm47xx_wdt_info;
wdt->wdd.timeout = WDT_DEFAULT_TIME; wdt->wdd.timeout = WDT_DEFAULT_TIME;
ret = wdt->wdd.ops->set_timeout(&wdt->wdd, timeout); ret = wdt->wdd.ops->set_timeout(&wdt->wdd, timeout);
......
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