Use setup_timer function instead of initializing timer with the
    function and data fields.

Signed-off-by: Allen Pais <allen.l...@gmail.com>
---
 drivers/net/wan/hdlc_fr.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wan/hdlc_fr.c b/drivers/net/wan/hdlc_fr.c
index 78596e4..425a47f 100644
--- a/drivers/net/wan/hdlc_fr.c
+++ b/drivers/net/wan/hdlc_fr.c
@@ -1003,11 +1003,10 @@ static void fr_start(struct net_device *dev)
                state(hdlc)->n391cnt = 0;
                state(hdlc)->txseq = state(hdlc)->rxseq = 0;
 
-               init_timer(&state(hdlc)->timer);
+               setup_timer(&state(hdlc)->timer, fr_timer,
+                           (unsigned long)dev);
                /* First poll after 1 s */
                state(hdlc)->timer.expires = jiffies + HZ;
-               state(hdlc)->timer.function = fr_timer;
-               state(hdlc)->timer.data = (unsigned long)dev;
                add_timer(&state(hdlc)->timer);
        } else
                fr_set_link_state(1, dev);
-- 
2.7.4

Reply via email to