Hi Uli, On Fri, Dec 9, 2016 at 1:36 PM, Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> wrote: > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> > --- > drivers/tty/serial/sh-sci.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c > index 844288f..ce3cf03 100644 > --- a/drivers/tty/serial/sh-sci.c > +++ b/drivers/tty/serial/sh-sci.c > @@ -2729,6 +2734,7 @@ static int sci_init_single(struct platform_device *dev, > sci_port->overrun_reg = SCLSR; > sci_port->overrun_mask = SCLSR_ORER; > sci_port->sampling_rate_mask = SCI_SR_RANGE(8, 32); > + sci_port->rx_trigger = 64; > break; > case PORT_SCIFA: > port->fifosize = 64; > @@ -2747,12 +2753,14 @@ static int sci_init_single(struct platform_device *dev, > sci_port->overrun_mask = SCLSR_ORER; > sci_port->sampling_rate_mask = SCI_SR(32); > } > + sci_port->rx_trigger = 8; PORT_SCIF is also used with SCIx_SH7705_SCIF_REGTYPE, so this should be moved inside the "else" branch above. Then, what to do with the "if" branch? >From the sh7705/sh7720/sh7721 datasheets, it looks a lot like SCIFA, and thus would need "sci_port->rx_trigger = 32", and the same handling as SCIFA in scif_set_rtrg(). See also "[RFC] serial: sh-sci: Correct FIFO stages on sh7705/sh7720/sh7721" (https://patchwork.kernel.org/patch/6810191/). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
