diff options
author | Haavard Skinnemoen <haavard.skinnemoen@atmel.com> | 2008-07-16 16:52:36 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-07-20 20:12:34 -0400 |
commit | 6bb0e3a59a089e23eecc0af3b6f6012b2a9affba (patch) | |
tree | 06790c1e5a3505b9d790506710d59d88c780f819 /drivers/serial | |
parent | 15648f154a8faea97cbe931e189cf0a57fd066f4 (diff) |
Subject: [PATCH 1/2] serial: Add flush_buffer() operation to uart_ops
Serial drivers using DMA (like the atmel_serial driver) tend to get very
confused when the xmit buffer is flushed and nobody told them. They
also tend to spew a lot of garbage since the DMA engine keeps running
after the buffer is flushed and possibly refilled with unrelated data.
This patch adds a new flush_buffer operation to the uart_ops struct,
along with a call to it from uart_flush_buffer() right after the xmit
buffer has been cleared. The driver can implement this in order to
syncronize its internal DMA state with the xmit buffer when the buffer
is flushed.
Signed-off-by: Haavard Skinnemoen <haavard.skinnemoen@atmel.com>
Acked-by: Alan Cox <alan@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/serial')
-rw-r--r-- | drivers/serial/serial_core.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/serial/serial_core.c b/drivers/serial/serial_core.c index 42d2e108b679..9884bc9eecb1 100644 --- a/drivers/serial/serial_core.c +++ b/drivers/serial/serial_core.c | |||
@@ -573,6 +573,8 @@ static void uart_flush_buffer(struct tty_struct *tty) | |||
573 | 573 | ||
574 | spin_lock_irqsave(&port->lock, flags); | 574 | spin_lock_irqsave(&port->lock, flags); |
575 | uart_circ_clear(&state->info->xmit); | 575 | uart_circ_clear(&state->info->xmit); |
576 | if (port->ops->flush_buffer) | ||
577 | port->ops->flush_buffer(port); | ||
576 | spin_unlock_irqrestore(&port->lock, flags); | 578 | spin_unlock_irqrestore(&port->lock, flags); |
577 | tty_wakeup(tty); | 579 | tty_wakeup(tty); |
578 | } | 580 | } |