On Sat, 2015-12-05 at 12:40PM -0500, Peter Hurley wrote: > On 11/21/2015 09:59 PM, Soren Brinkmann wrote: > > 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 <peter@xxxxxxxxxxxxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html