diff options
author | Kees Cook <keescook@chromium.org> | 2017-10-25 06:53:59 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-10-26 23:09:16 -0400 |
commit | 032cfd66afcc2dd2c7be89c71b020fcb15bcc37d (patch) | |
tree | 3d4fdf1cc5ef8e4f1e6cfe12b73d32291ad12021 | |
parent | 605ea2f9356f315c9c03d69a54dc910fde32fd71 (diff) |
drivers/net: wan/sdla: 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: Allen Pais <allen.lkml@gmail.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Tobias Klauser <tklauser@distanz.ch>
Cc: netdev@vger.kernel.org
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/wan/sdla.c | 12 | ||||
-rw-r--r-- | include/linux/if_frad.h | 1 |
2 files changed, 6 insertions, 7 deletions
diff --git a/drivers/net/wan/sdla.c b/drivers/net/wan/sdla.c index 0cc48902dbb9..57ed259c8208 100644 --- a/drivers/net/wan/sdla.c +++ b/drivers/net/wan/sdla.c | |||
@@ -927,13 +927,10 @@ static irqreturn_t sdla_isr(int dummy, void *dev_id) | |||
927 | return IRQ_HANDLED; | 927 | return IRQ_HANDLED; |
928 | } | 928 | } |
929 | 929 | ||
930 | static void sdla_poll(unsigned long device) | 930 | static void sdla_poll(struct timer_list *t) |
931 | { | 931 | { |
932 | struct net_device *dev; | 932 | struct frad_local *flp = from_timer(flp, t, timer); |
933 | struct frad_local *flp; | 933 | struct net_device *dev = flp->dev; |
934 | |||
935 | dev = (struct net_device *) device; | ||
936 | flp = netdev_priv(dev); | ||
937 | 934 | ||
938 | if (sdla_byte(dev, SDLA_502_RCV_BUF)) | 935 | if (sdla_byte(dev, SDLA_502_RCV_BUF)) |
939 | sdla_receive(dev); | 936 | sdla_receive(dev); |
@@ -1616,8 +1613,9 @@ static void setup_sdla(struct net_device *dev) | |||
1616 | flp->assoc = sdla_assoc; | 1613 | flp->assoc = sdla_assoc; |
1617 | flp->deassoc = sdla_deassoc; | 1614 | flp->deassoc = sdla_deassoc; |
1618 | flp->dlci_conf = sdla_dlci_conf; | 1615 | flp->dlci_conf = sdla_dlci_conf; |
1616 | flp->dev = dev; | ||
1619 | 1617 | ||
1620 | setup_timer(&flp->timer, sdla_poll, (unsigned long)dev); | 1618 | timer_setup(&flp->timer, sdla_poll, 0); |
1621 | flp->timer.expires = 1; | 1619 | flp->timer.expires = 1; |
1622 | } | 1620 | } |
1623 | 1621 | ||
diff --git a/include/linux/if_frad.h b/include/linux/if_frad.h index 46df7e565d6f..82a1b4e93570 100644 --- a/include/linux/if_frad.h +++ b/include/linux/if_frad.h | |||
@@ -83,6 +83,7 @@ struct frad_local | |||
83 | 83 | ||
84 | /* fields that are used by the Sangoma SDLA cards */ | 84 | /* fields that are used by the Sangoma SDLA cards */ |
85 | struct timer_list timer; | 85 | struct timer_list timer; |
86 | struct net_device *dev; | ||
86 | int type; /* adapter type */ | 87 | int type; /* adapter type */ |
87 | int state; /* state of the S502/8 control latch */ | 88 | int state; /* state of the S502/8 control latch */ |
88 | int buffer; /* current buffer for S508 firmware */ | 89 | int buffer; /* current buffer for S508 firmware */ |