On Sat, 2015-12-05 at 12:40PM -0500, Peter Hurley wrote:

On 11/21/2015 09:59 PM, Soren Brinkmann wrote:

quoted

Move RX-related IRQ handling into a helper function.

Fixes a problem where every char received after a parity or frame error
in the current isr will also be tagged as a parity or frame error.

Reviewed-by: Peter Hurley <redacted>

Thanks. I'll add your text to the commit message.

NB: the sysrq problem in cdns_uart_isr() with needing to drop the
locks is because cdns_uart_console_write() tries to take the port->lock.
The 8250 driver handles this problem by not trying to take the
port->lock if port->sysrq is non-zero. See serial8250_console_write().

I'll look into this. I'll see if I can include that in the next
iteration of this series or do it later.

	Thanks,
	S?ren