On Wed, Sep 10, 2014 at 09:30:00PM +0200, Sebastian Andrzej Siewior wrote: > serial8250_do_startup() adds UART_IER_RDI and UART_IER_RLSI to ier. > serial8250_stop_rx() should remove both. > This is what the serial-omap driver has been doing and is now moved to > the 8250-core since it does no look to be *that* omap specific. > > Cc: heikki.krogerus@xxxxxxxxxxxxxxx Looks good to me. FWIW... Reviewed-by: Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx> > Reviewed-by: Tony Lindgren <tony@xxxxxxxxxxx> > Tested-by: Tony Lindgren <tony@xxxxxxxxxxx> > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> > --- > drivers/tty/serial/8250/8250_core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/tty/serial/8250/8250_core.c b/drivers/tty/serial/8250/8250_core.c > index ac88e66df65d..139f3d2b8aa9 100644 > --- a/drivers/tty/serial/8250/8250_core.c > +++ b/drivers/tty/serial/8250/8250_core.c > @@ -1390,7 +1390,7 @@ static void serial8250_stop_rx(struct uart_port *port) > > serial8250_rpm_get(up); > > - up->ier &= ~UART_IER_RLSI; > + up->ier &= ~(UART_IER_RLSI | UART_IER_RDI); > up->port.read_status_mask &= ~UART_LSR_DR; > serial_port_out(port, UART_IER, up->ier); > > -- > 2.1.0 Thanks, -- heikki -- 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
