drivers/net: fealnx: Convert timers to use timer_setup()
In preparation for unconditionally passing the struct timer_list pointer to all timer callbacks, switch to using the new timer_setup() and from_timer() to pass the timer pointer explicitly. Cc: "David S. Miller" <davem@davemloft.net> Cc: "yuval.shaia@oracle.com" <yuval.shaia@oracle.com> Cc: Allen Pais <allen.lkml@gmail.com> Cc: Stephen Hemminger <stephen@networkplumber.org> Cc: Philippe Reynes <tremyfr@gmail.com> Cc: Johannes Berg <johannes.berg@intel.com> Cc: netdev@vger.kernel.org Signed-off-by: Kees Cook <keescook@chromium.org> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
9cb618c295
commit
8b3718dc2c
|
@ -426,8 +426,8 @@ static void mdio_write(struct net_device *dev, int phy_id, int location, int val
|
||||||
static int netdev_open(struct net_device *dev);
|
static int netdev_open(struct net_device *dev);
|
||||||
static void getlinktype(struct net_device *dev);
|
static void getlinktype(struct net_device *dev);
|
||||||
static void getlinkstatus(struct net_device *dev);
|
static void getlinkstatus(struct net_device *dev);
|
||||||
static void netdev_timer(unsigned long data);
|
static void netdev_timer(struct timer_list *t);
|
||||||
static void reset_timer(unsigned long data);
|
static void reset_timer(struct timer_list *t);
|
||||||
static void fealnx_tx_timeout(struct net_device *dev);
|
static void fealnx_tx_timeout(struct net_device *dev);
|
||||||
static void init_ring(struct net_device *dev);
|
static void init_ring(struct net_device *dev);
|
||||||
static netdev_tx_t start_tx(struct sk_buff *skb, struct net_device *dev);
|
static netdev_tx_t start_tx(struct sk_buff *skb, struct net_device *dev);
|
||||||
|
@ -909,13 +909,13 @@ static int netdev_open(struct net_device *dev)
|
||||||
printk(KERN_DEBUG "%s: Done netdev_open().\n", dev->name);
|
printk(KERN_DEBUG "%s: Done netdev_open().\n", dev->name);
|
||||||
|
|
||||||
/* Set the timer to check for link beat. */
|
/* Set the timer to check for link beat. */
|
||||||
setup_timer(&np->timer, netdev_timer, (unsigned long)dev);
|
timer_setup(&np->timer, netdev_timer, 0);
|
||||||
np->timer.expires = RUN_AT(3 * HZ);
|
np->timer.expires = RUN_AT(3 * HZ);
|
||||||
|
|
||||||
/* timer handler */
|
/* timer handler */
|
||||||
add_timer(&np->timer);
|
add_timer(&np->timer);
|
||||||
|
|
||||||
setup_timer(&np->reset_timer, reset_timer, (unsigned long)dev);
|
timer_setup(&np->reset_timer, reset_timer, 0);
|
||||||
np->reset_timer_armed = 0;
|
np->reset_timer_armed = 0;
|
||||||
return rc;
|
return rc;
|
||||||
}
|
}
|
||||||
|
@ -1078,10 +1078,10 @@ static void allocate_rx_buffers(struct net_device *dev)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
static void netdev_timer(unsigned long data)
|
static void netdev_timer(struct timer_list *t)
|
||||||
{
|
{
|
||||||
struct net_device *dev = (struct net_device *) data;
|
struct netdev_private *np = from_timer(np, t, timer);
|
||||||
struct netdev_private *np = netdev_priv(dev);
|
struct net_device *dev = np->mii.dev;
|
||||||
void __iomem *ioaddr = np->mem;
|
void __iomem *ioaddr = np->mem;
|
||||||
int old_crvalue = np->crvalue;
|
int old_crvalue = np->crvalue;
|
||||||
unsigned int old_linkok = np->linkok;
|
unsigned int old_linkok = np->linkok;
|
||||||
|
@ -1167,10 +1167,10 @@ static void enable_rxtx(struct net_device *dev)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
static void reset_timer(unsigned long data)
|
static void reset_timer(struct timer_list *t)
|
||||||
{
|
{
|
||||||
struct net_device *dev = (struct net_device *) data;
|
struct netdev_private *np = from_timer(np, t, reset_timer);
|
||||||
struct netdev_private *np = netdev_priv(dev);
|
struct net_device *dev = np->mii.dev;
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
|
|
||||||
printk(KERN_WARNING "%s: resetting tx and rx machinery\n", dev->name);
|
printk(KERN_WARNING "%s: resetting tx and rx machinery\n", dev->name);
|
||||||
|
|
Loading…
Reference in New Issue