diff options
author | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2013-01-08 09:20:05 -0500 |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2013-02-14 09:55:01 -0500 |
commit | 57985d7e1e48f16548aa6904264e21bca15af0fc (patch) | |
tree | fd6d29f4c5ceacbb6315125f0c121d2da830f0d1 /drivers/s390/char/tty3270.c | |
parent | 736c9fd2902d919b075cf9cf371d1733c5ff635d (diff) |
s390/3270: fix initialization order in tty3270_alloc_view
Corrects the order of tasklet_init vs. the allocation of the
read request which has been broken by git commit 9d2ae233
"TTY: tty3270, move initialization to allocation".
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers/s390/char/tty3270.c')
-rw-r--r-- | drivers/s390/char/tty3270.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/drivers/s390/char/tty3270.c b/drivers/s390/char/tty3270.c index 964018402b36..5e4b6fc49ae3 100644 --- a/drivers/s390/char/tty3270.c +++ b/drivers/s390/char/tty3270.c | |||
@@ -683,12 +683,6 @@ tty3270_alloc_view(void) | |||
683 | INIT_LIST_HEAD(&tp->update); | 683 | INIT_LIST_HEAD(&tp->update); |
684 | INIT_LIST_HEAD(&tp->rcl_lines); | 684 | INIT_LIST_HEAD(&tp->rcl_lines); |
685 | tp->rcl_max = 20; | 685 | tp->rcl_max = 20; |
686 | tty_port_init(&tp->port); | ||
687 | setup_timer(&tp->timer, (void (*)(unsigned long)) tty3270_update, | ||
688 | (unsigned long) tp); | ||
689 | tasklet_init(&tp->readlet, | ||
690 | (void (*)(unsigned long)) tty3270_read_tasklet, | ||
691 | (unsigned long) tp->read); | ||
692 | 686 | ||
693 | for (pages = 0; pages < TTY3270_STRING_PAGES; pages++) { | 687 | for (pages = 0; pages < TTY3270_STRING_PAGES; pages++) { |
694 | tp->freemem_pages[pages] = (void *) | 688 | tp->freemem_pages[pages] = (void *) |
@@ -710,6 +704,14 @@ tty3270_alloc_view(void) | |||
710 | tp->kbd = kbd_alloc(); | 704 | tp->kbd = kbd_alloc(); |
711 | if (!tp->kbd) | 705 | if (!tp->kbd) |
712 | goto out_reset; | 706 | goto out_reset; |
707 | |||
708 | tty_port_init(&tp->port); | ||
709 | setup_timer(&tp->timer, (void (*)(unsigned long)) tty3270_update, | ||
710 | (unsigned long) tp); | ||
711 | tasklet_init(&tp->readlet, | ||
712 | (void (*)(unsigned long)) tty3270_read_tasklet, | ||
713 | (unsigned long) tp->read); | ||
714 | |||
713 | return tp; | 715 | return tp; |
714 | 716 | ||
715 | out_reset: | 717 | out_reset: |