Re: [PATCH RFC 4/7] serial: imx: set_termios(): do not enable autoRTS if RTS is unset
From: Sascha Hauer <s.hauer@pengutronix.de>
Date: 2019-06-20 09:37:51
Also in:
linux-serial
Hi Sergey, On Fri, Jun 14, 2019 at 03:11:31PM +0300, Sergey Organov wrote:
quoted hunk ↗ jump to hunk
set_termios() shouldn't set UCR2_CTSC bit if UCR2_CTS (=TIOCM_RTS) is cleared. Added corresponding check in imx_uart_rts_auto() to fix this. Signed-off-by: Sergey Organov <redacted> --- drivers/tty/serial/imx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c index 17e2322..8ee910f 100644 --- a/drivers/tty/serial/imx.c +++ b/drivers/tty/serial/imx.c@@ -405,7 +405,8 @@ static void imx_uart_rts_inactive(struct imx_port *sport, u32 *ucr2) /* called with port.lock taken and irqs caller dependent */ static void imx_uart_rts_auto(struct imx_port *sport, u32 *ucr2) { - *ucr2 |= UCR2_CTSC; + if (*ucr2 & UCR2_CTS) + *ucr2 |= UCR2_CTSC; }
*ucr2 is set like this in imx_uart_set_termios(): ucr2 = UCR2_SRST | UCR2_IRTS; if ((termios->c_cflag & CSIZE) == CS8) ucr2 |= UCR2_WS; ... imx_uart_rts_auto(sport, &ucr2); So the UCR2_CTS bit is never set, hence UCR2_CTSC will never be set. You meant to pass in the actual register value of the UCR2 register. This is shifted around a bit in the following patches, as an end result we have: old_ucr2 = imx_uart_readl(sport, UCR2); ucr2 = old_ucr2 & (UCR2_TXEN | UCR2_RXEN | UCR2_ATEN | UCR2_CTSC); ... if (ucr2 & UCR2_CTS) ucr2 |= UCR2_CTSC; Again the test can never be true, it should probably be if (old_ucr2 & UCR2_CTS). With this issue and the one Lothar has found fixed this series works for me. With these issues fixed I'd be happy to test this series and apply it in favour of my patch. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel