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

diff --git a/drivers/net/ethernet/sun/sungem.c 
b/drivers/net/ethernet/sun/sungem.c
index fa607d0..b75ab8f 100644
--- a/drivers/net/ethernet/sun/sungem.c
+++ b/drivers/net/ethernet/sun/sungem.c
@@ -2910,9 +2910,7 @@ static int gem_init_one(struct pci_dev *pdev, const 
struct pci_device_id *ent)
 
        gp->msg_enable = DEFAULT_MSG;
 
-       init_timer(&gp->link_timer);
-       gp->link_timer.function = gem_link_timer;
-       gp->link_timer.data = (unsigned long) gp;
+       setup_timer(&gp->link_timer, gem_link_timer, (unsigned long)gp);
 
        INIT_WORK(&gp->reset_task, gem_reset_task);
 
-- 
2.7.4

Reply via email to