Staging: media: lirc: Use setup_timer
authorVaishali Thakkar <vthakkar1994@gmail.com>
Wed, 18 Feb 2015 16:52:29 +0000 (22:22 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 26 Feb 2015 20:23:59 +0000 (12:23 -0800)
This patch introduces the use of function setup_timer
instead of structure assignments as it is the preferred
way to setup and set the timer.

This is done using Coccinelle and semantic patch used is
as follows:

@@
expression x,y,z;
@@

- init_timer (&x);
+ setup_timer (&x, y, z);
- x.function = y;
- x.data = z;

Signed-off-by: Vaishali Thakkar <vthakkar1994@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/media/lirc/lirc_sir.c

index 39f4733fb1ee25fea623120441d989c6624bc235..d1ada8c72f0d186524fe23b0dbdb4c41dabf6eb1 100644 (file)
@@ -683,9 +683,7 @@ static int init_port(void)
        }
        pr_info("I/O port 0x%.4x, IRQ %d.\n", io, irq);
 
-       init_timer(&timerlist);
-       timerlist.function = sir_timeout;
-       timerlist.data = 0xabadcafe;
+       setup_timer(&timerlist, sir_timeout, 0xabadcafe);
 
        return 0;
 }