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_cisco.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/net/wan/hdlc_cisco.c b/drivers/net/wan/hdlc_cisco.c
index a408abc..c696d42 100644
--- a/drivers/net/wan/hdlc_cisco.c
+++ b/drivers/net/wan/hdlc_cisco.c
@@ -293,10 +293,8 @@ static void cisco_start(struct net_device *dev)
        st->up = st->txseq = st->rxseq = 0;
        spin_unlock_irqrestore(&st->lock, flags);
 
-       init_timer(&st->timer);
+       setup_timer(&st->timer, cisco_timer, (unsigned long)dev);
        st->timer.expires = jiffies + HZ; /* First poll after 1 s */
-       st->timer.function = cisco_timer;
-       st->timer.data = (unsigned long)dev;
        add_timer(&st->timer);
 }
 
-- 
2.7.4

Reply via email to