watchdog: cpwd: Convert timers to use timer_setup()
PATCH 1/1 watchdog: cpwd: Convert timers to use timer_setup()
In preparation for unconditionally passing the struct timer_list pointer to all timer callbacks, switch to using the new timer_setup() and from_timer() to pass the timer pointer explicitly. Switches to using the global that is used everywhere else.
Cc Wim Van Sebroeck <[email protected]>
Cc Guenter Roeck <[email protected]>
Signed-off-by Kees Cook <[email protected]>
--- drivers/watchdog/cpwd.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/watchdog/cpwd.c +4 -4
@@ -230,9 +230,9 @@ static void cpwd_resetbrokentimer(struct cpwd *p, int index)* interrupts within the PLD so me must continually * reset the timers ad infinitum. */ -static void cpwd_brokentimer (unsigned long data) +static void cpwd_brokentimer (struct timer_list *unused) { -struct cpwd *p = (struct cpwd *) data; +struct cpwd *p = cpwd_device;int id, tripped =0 ;/* kill a running timer instance, in case we
@@ -275,7 +275,7 @@ static void cpwd_stoptimer(struct cpwd *p, int index)if (p->broken) { p->devs[index].runstatus |=WD_STAT_BSTOP ; -cpwd_brokentimer ((unsigned long ) p); +cpwd_brokentimer (NULL ); } } }
@@ -608,7 +608,7 @@ static int cpwd_probe(struct platform_device *op) }if (p->broken) { -setup_timer (&cpwd_timer, cpwd_brokentimer, (unsigned long )p); +timer_setup (&cpwd_timer, cpwd_brokentimer,0 ); cpwd_timer.expires =WD_BTIMEOUT ; pr_info("PLD defect workaround enabled for model %s\n",
Signature · 6 lines
2.7.4 -- Kees Cook Pixel Security
Quoted 9 lines · Kees Cook
In preparation for unconditionally passing the struct
timer_listpointer to all timer callbacks, switch to using the newtimer_setup()andfrom_timer()to pass the timer pointer explicitly. Switches to using the global that is used everywhere else.Cc Wim Van Sebroeck <[email protected]>Cc Guenter Roeck <[email protected]>Signed-off-by Kees Cook <[email protected]>
Reviewed-by Guenter Roeck <[email protected]>
Quoted 39 lines
--- drivers/watchdog/cpwd.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)drivers/watchdog/cpwd.c +4
-4@@ -230,9 +230,9 @@ static void cpwd_resetbrokentimer(struct cpwd *p, int index)* interrupts within the PLD so me must continually * reset the timers ad infinitum. */ -static void cpwd_brokentimer (unsigned long data) +static void cpwd_brokentimer (struct timer_list *unused) { -struct cpwd *p = (struct cpwd *) data; +struct cpwd *p = cpwd_device;int id, tripped =0 ;/* kill a running timer instance, in case we @@ -275,7 +275,7 @@ static void cpwd_stoptimer(struct cpwd *p, int index)if (p->broken) { p->devs[index].runstatus |=WD_STAT_BSTOP ; -cpwd_brokentimer ((unsigned long ) p); +cpwd_brokentimer (NULL ); } } }@@ -608,7 +608,7 @@ static int cpwd_probe(struct platform_device *op) }if (p->broken) { -setup_timer (&cpwd_timer, cpwd_brokentimer, (unsigned long )p); +timer_setup (&cpwd_timer, cpwd_brokentimer,0 ); cpwd_timer.expires =WD_BTIMEOUT ; pr_info("PLD defect workaround enabled for model %s\n",