diff options
author | Kees Cook <keescook@chromium.org> | 2017-10-24 06:00:18 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-11-04 07:01:54 -0400 |
commit | a8497b31fee650846fcc5a1e41f4ef22318a9925 (patch) | |
tree | a7b92db99b501b547a22de5e90e6bf0d5e9c807a /drivers/tty/cyclades.c | |
parent | 4790b6dc7ad6c0f1eb71b94c4babe10a9c41c697 (diff) |
tty: cyclades: 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. Moves timer structures
from global to attached to struct cyclades_port.
Cc: Jiri Slaby <jslaby@suse.com>
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/cyclades.c')
-rw-r--r-- | drivers/tty/cyclades.c | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/drivers/tty/cyclades.c b/drivers/tty/cyclades.c index d272bc4e7fb5..f30f7a90995c 100644 --- a/drivers/tty/cyclades.c +++ b/drivers/tty/cyclades.c | |||
@@ -286,8 +286,7 @@ static long cyz_polling_cycle = CZ_DEF_POLL; | |||
286 | static DEFINE_TIMER(cyz_timerlist, cyz_poll, 0, 0); | 286 | static DEFINE_TIMER(cyz_timerlist, cyz_poll, 0, 0); |
287 | 287 | ||
288 | #else /* CONFIG_CYZ_INTR */ | 288 | #else /* CONFIG_CYZ_INTR */ |
289 | static void cyz_rx_restart(unsigned long); | 289 | static void cyz_rx_restart(struct timer_list *); |
290 | static struct timer_list cyz_rx_full_timer[NR_PORTS]; | ||
291 | #endif /* CONFIG_CYZ_INTR */ | 290 | #endif /* CONFIG_CYZ_INTR */ |
292 | 291 | ||
293 | static void cyy_writeb(struct cyclades_port *port, u32 reg, u8 val) | 292 | static void cyy_writeb(struct cyclades_port *port, u32 reg, u8 val) |
@@ -992,10 +991,8 @@ static void cyz_handle_rx(struct cyclades_port *info) | |||
992 | else | 991 | else |
993 | char_count = rx_put - rx_get + rx_bufsize; | 992 | char_count = rx_put - rx_get + rx_bufsize; |
994 | if (char_count >= readl(&buf_ctrl->rx_threshold) && | 993 | if (char_count >= readl(&buf_ctrl->rx_threshold) && |
995 | !timer_pending(&cyz_rx_full_timer[ | 994 | !timer_pending(&info->rx_full_timer)) |
996 | info->line])) | 995 | mod_timer(&info->rx_full_timer, jiffies + 1); |
997 | mod_timer(&cyz_rx_full_timer[info->line], | ||
998 | jiffies + 1); | ||
999 | #endif | 996 | #endif |
1000 | info->idle_stats.recv_idle = jiffies; | 997 | info->idle_stats.recv_idle = jiffies; |
1001 | tty_schedule_flip(&info->port); | 998 | tty_schedule_flip(&info->port); |
@@ -1197,9 +1194,9 @@ static irqreturn_t cyz_interrupt(int irq, void *dev_id) | |||
1197 | return IRQ_HANDLED; | 1194 | return IRQ_HANDLED; |
1198 | } /* cyz_interrupt */ | 1195 | } /* cyz_interrupt */ |
1199 | 1196 | ||
1200 | static void cyz_rx_restart(unsigned long arg) | 1197 | static void cyz_rx_restart(struct timer_list *t) |
1201 | { | 1198 | { |
1202 | struct cyclades_port *info = (struct cyclades_port *)arg; | 1199 | struct cyclades_port *info = from_timer(info, t, rx_full_timer); |
1203 | struct cyclades_card *card = info->card; | 1200 | struct cyclades_card *card = info->card; |
1204 | int retval; | 1201 | int retval; |
1205 | __u32 channel = info->line - card->first_line; | 1202 | __u32 channel = info->line - card->first_line; |
@@ -3097,8 +3094,7 @@ static int cy_init_card(struct cyclades_card *cinfo) | |||
3097 | else | 3094 | else |
3098 | info->xmit_fifo_size = 4 * CYZ_FIFO_SIZE; | 3095 | info->xmit_fifo_size = 4 * CYZ_FIFO_SIZE; |
3099 | #ifdef CONFIG_CYZ_INTR | 3096 | #ifdef CONFIG_CYZ_INTR |
3100 | setup_timer(&cyz_rx_full_timer[port], | 3097 | timer_setup(&info->rx_full_timer, cyz_rx_restart, 0); |
3101 | cyz_rx_restart, (unsigned long)info); | ||
3102 | #endif | 3098 | #endif |
3103 | } else { | 3099 | } else { |
3104 | unsigned short chip_number; | 3100 | unsigned short chip_number; |