Commit 799be6ff authored by Jiri Slaby's avatar Jiri Slaby Committed by Greg Kroah-Hartman

TTY: amiserial/simserial, use close delays from tty_port

Note that previously simserial set the delay to 0. So we preserve
that. BUT, is it correct?
Signed-off-by: default avatarJiri Slaby <jslaby@suse.cz>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 87758791
...@@ -535,8 +535,8 @@ static void rs_close(struct tty_struct *tty, struct file * filp) ...@@ -535,8 +535,8 @@ static void rs_close(struct tty_struct *tty, struct file * filp)
tty_ldisc_flush(tty); tty_ldisc_flush(tty);
info->tport.tty = NULL; info->tport.tty = NULL;
if (info->tport.blocked_open) { if (info->tport.blocked_open) {
if (info->close_delay) if (info->tport.close_delay)
schedule_timeout_interruptible(info->close_delay); schedule_timeout_interruptible(info->tport.close_delay);
wake_up_interruptible(&info->tport.open_wait); wake_up_interruptible(&info->tport.open_wait);
} }
info->flags &= ~(ASYNC_NORMAL_ACTIVE|ASYNC_CLOSING); info->flags &= ~(ASYNC_NORMAL_ACTIVE|ASYNC_CLOSING);
...@@ -829,6 +829,7 @@ simrs_init (void) ...@@ -829,6 +829,7 @@ simrs_init (void)
*/ */
for (i = 0, state = rs_table; i < NR_PORTS; i++,state++) { for (i = 0, state = rs_table; i < NR_PORTS; i++,state++) {
tty_port_init(&state->tport); tty_port_init(&state->tport);
state->tport.close_delay = 0; /* XXX really 0? */
if (state->type == PORT_UNKNOWN) continue; if (state->type == PORT_UNKNOWN) continue;
......
...@@ -1022,8 +1022,8 @@ static int get_serial_info(struct serial_state *state, ...@@ -1022,8 +1022,8 @@ static int get_serial_info(struct serial_state *state,
tmp.flags = state->flags; tmp.flags = state->flags;
tmp.xmit_fifo_size = state->xmit_fifo_size; tmp.xmit_fifo_size = state->xmit_fifo_size;
tmp.baud_base = state->baud_base; tmp.baud_base = state->baud_base;
tmp.close_delay = state->close_delay; tmp.close_delay = state->tport.close_delay;
tmp.closing_wait = state->closing_wait; tmp.closing_wait = state->tport.closing_wait;
tmp.custom_divisor = state->custom_divisor; tmp.custom_divisor = state->custom_divisor;
tty_unlock(); tty_unlock();
if (copy_to_user(retinfo,&tmp,sizeof(*retinfo))) if (copy_to_user(retinfo,&tmp,sizeof(*retinfo)))
...@@ -1052,7 +1052,7 @@ static int set_serial_info(struct tty_struct *tty, struct serial_state *state, ...@@ -1052,7 +1052,7 @@ static int set_serial_info(struct tty_struct *tty, struct serial_state *state,
if (!serial_isroot()) { if (!serial_isroot()) {
if ((new_serial.baud_base != state->baud_base) || if ((new_serial.baud_base != state->baud_base) ||
(new_serial.close_delay != state->close_delay) || (new_serial.close_delay != state->tport.close_delay) ||
(new_serial.xmit_fifo_size != state->xmit_fifo_size) || (new_serial.xmit_fifo_size != state->xmit_fifo_size) ||
((new_serial.flags & ~ASYNC_USR_MASK) != ((new_serial.flags & ~ASYNC_USR_MASK) !=
(state->flags & ~ASYNC_USR_MASK))) (state->flags & ~ASYNC_USR_MASK)))
...@@ -1077,8 +1077,8 @@ static int set_serial_info(struct tty_struct *tty, struct serial_state *state, ...@@ -1077,8 +1077,8 @@ static int set_serial_info(struct tty_struct *tty, struct serial_state *state,
state->flags = ((state->flags & ~ASYNC_FLAGS) | state->flags = ((state->flags & ~ASYNC_FLAGS) |
(new_serial.flags & ASYNC_FLAGS)); (new_serial.flags & ASYNC_FLAGS));
state->custom_divisor = new_serial.custom_divisor; state->custom_divisor = new_serial.custom_divisor;
state->close_delay = new_serial.close_delay * HZ/100; state->tport.close_delay = new_serial.close_delay * HZ/100;
state->closing_wait = new_serial.closing_wait * HZ/100; state->tport.closing_wait = new_serial.closing_wait * HZ/100;
tty->low_latency = (state->flags & ASYNC_LOW_LATENCY) ? 1 : 0; tty->low_latency = (state->flags & ASYNC_LOW_LATENCY) ? 1 : 0;
check_and_exit: check_and_exit:
...@@ -1413,8 +1413,8 @@ static void rs_close(struct tty_struct *tty, struct file * filp) ...@@ -1413,8 +1413,8 @@ static void rs_close(struct tty_struct *tty, struct file * filp)
* the line discipline to only process XON/XOFF characters. * the line discipline to only process XON/XOFF characters.
*/ */
tty->closing = 1; tty->closing = 1;
if (state->closing_wait != ASYNC_CLOSING_WAIT_NONE) if (state->tport.closing_wait != ASYNC_CLOSING_WAIT_NONE)
tty_wait_until_sent(tty, state->closing_wait); tty_wait_until_sent(tty, state->tport.closing_wait);
/* /*
* At this point we stop accepting input. To do this, we * At this point we stop accepting input. To do this, we
* disable the receive line status interrupts, and tell the * disable the receive line status interrupts, and tell the
...@@ -1444,8 +1444,8 @@ static void rs_close(struct tty_struct *tty, struct file * filp) ...@@ -1444,8 +1444,8 @@ static void rs_close(struct tty_struct *tty, struct file * filp)
tty->closing = 0; tty->closing = 0;
state->tport.tty = NULL; state->tport.tty = NULL;
if (state->tport.blocked_open) { if (state->tport.blocked_open) {
if (state->close_delay) { if (state->tport.close_delay) {
msleep_interruptible(jiffies_to_msecs(state->close_delay)); msleep_interruptible(jiffies_to_msecs(state->tport.close_delay));
} }
wake_up_interruptible(&state->tport.open_wait); wake_up_interruptible(&state->tport.open_wait);
} }
...@@ -1863,8 +1863,6 @@ static int __init amiga_serial_probe(struct platform_device *pdev) ...@@ -1863,8 +1863,6 @@ static int __init amiga_serial_probe(struct platform_device *pdev)
state->port = (int)&custom.serdatr; /* Just to give it a value */ state->port = (int)&custom.serdatr; /* Just to give it a value */
state->line = 0; state->line = 0;
state->custom_divisor = 0; state->custom_divisor = 0;
state->close_delay = 5*HZ/10;
state->closing_wait = 30*HZ;
state->icount.cts = state->icount.dsr = state->icount.cts = state->icount.dsr =
state->icount.rng = state->icount.dcd = 0; state->icount.rng = state->icount.dcd = 0;
state->icount.rx = state->icount.tx = 0; state->icount.rx = state->icount.tx = 0;
......
...@@ -36,8 +36,6 @@ struct serial_state { ...@@ -36,8 +36,6 @@ struct serial_state {
int xmit_fifo_size; int xmit_fifo_size;
int custom_divisor; int custom_divisor;
int count; int count;
unsigned short close_delay;
unsigned short closing_wait; /* time to wait before closing */
struct async_icount icount; struct async_icount icount;
struct tty_port tport; struct tty_port tport;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment