Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1736648 > unrolled thread

[PATCH 1/1] net: ti: netcp: use setup_timer

Started byAllen Pais <allen.lkml@gmail.com>
First post2017-09-21 15:10 +0200
Last post2017-09-22 00:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/1] net: ti: netcp: use setup_timer Allen Pais <allen.lkml@gmail.com> - 2017-09-21 15:10 +0200
    Re: [PATCH 1/1] net: ti: netcp: use setup_timer David Miller <davem@davemloft.net> - 2017-09-22 00:50 +0200

#1736648 — [PATCH 1/1] net: ti: netcp: use setup_timer

FromAllen Pais <allen.lkml@gmail.com>
Date2017-09-21 15:10 +0200
Subject[PATCH 1/1] net: ti: netcp: use setup_timer
Message-ID<us9oS-1pB-11@gated-at.bofh.it>
    Use setup_timer function instead of initializing timer with the
    function and data fields.

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

diff --git a/drivers/net/ethernet/ti/netcp_ethss.c b/drivers/net/ethernet/ti/netcp_ethss.c
index 28cb38a..4ad8216 100644
--- a/drivers/net/ethernet/ti/netcp_ethss.c
+++ b/drivers/net/ethernet/ti/netcp_ethss.c
@@ -3616,9 +3616,8 @@ static int gbe_probe(struct netcp_device *netcp_device, struct device *dev,
 	}
 	spin_unlock_bh(&gbe_dev->hw_stats_lock);
 
-	init_timer(&gbe_dev->timer);
-	gbe_dev->timer.data	 = (unsigned long)gbe_dev;
-	gbe_dev->timer.function = netcp_ethss_timer;
+	setup_timer(&gbe_dev->timer, netcp_ethss_timer,
+		    (unsigned long)gbe_dev);
 	gbe_dev->timer.expires	 = jiffies + GBE_TIMER_INTERVAL;
 	add_timer(&gbe_dev->timer);
 	*inst_priv = gbe_dev;
-- 
2.7.4

[toc] | [next] | [standalone]


#1737088

FromDavid Miller <davem@davemloft.net>
Date2017-09-22 00:50 +0200
Message-ID<usisa-6Mo-25@gated-at.bofh.it>
In reply to#1736648
From: Allen Pais <allen.lkml@gmail.com>
Date: Thu, 21 Sep 2017 18:32:58 +0530

>     Use setup_timer function instead of initializing timer with the
>     function and data fields.
> 
> Signed-off-by: Allen Pais <allen.lkml@gmail.com>

Applied.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web