[PATCH v3] mxs: uart: allow setting RTS from software

Shawn Guo shawn.guo at linaro.org
Mon Jan 7 20:06:35 EST 2013


On Mon, Jan 07, 2013 at 10:43:38AM +0100, Marc Kleine-Budde wrote:
> On 12/14/2012 03:27 AM, Huang Shijie wrote:
> > 于 2012年12月13日 21:27, Steffen Trumtrar 写道:
> >> With the patch "serial: mxs-auart: fix the wrong RTS hardware flow control" the
> >> mainline mxs-uart driver now sets RTSEN only when hardware flow control is
> >> enabled via software. It is not possible any longer to set RTS manually via
> >> software. However, the manual modification is a valid operation.
> >> Regain the possibility to set RTS via software and only set RTSEN when hardware
> >> flow control is explicitly enabled via settermios cflag CRTSCTS.
> >>
> >> Signed-off-by: Steffen Trumtrar <s.trumtrar at pengutronix.de>
> >> ---
> >>  drivers/tty/serial/mxs-auart.c |    4 +++-
> >>  1 file changed, 3 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/tty/serial/mxs-auart.c b/drivers/tty/serial/mxs-auart.c
> >> index 6db23b0..9f63f88 100644
> >> --- a/drivers/tty/serial/mxs-auart.c
> >> +++ b/drivers/tty/serial/mxs-auart.c
> >> @@ -412,10 +412,12 @@ static void mxs_auart_set_mctrl(struct uart_port *u, unsigned mctrl)
> >>  
> >>  	u32 ctrl = readl(u->membase + AUART_CTRL2);
> >>  
> >> -	ctrl &= ~AUART_CTRL2_RTSEN;
> >> +	ctrl &= ~(AUART_CTRL2_RTSEN | AUART_CTRL2_RTS);
> >>  	if (mctrl & TIOCM_RTS) {
> >>  		if (tty_port_cts_enabled(&u->state->port))
> >>  			ctrl |= AUART_CTRL2_RTSEN;
> >> +		else
> >> +			ctrl |= AUART_CTRL2_RTS;
> >>  	}
> >>  
> >>  	s->ctrl = mctrl;
> > Reviewed-by: Huang Shijie <b32955 at freescale.com>
> 
> This patch is not not in today's next. What's the current status? Who
> will take it?
> 
I see stable at vger.kernel.org is on copy.  But it's not the correct way
submit a fix for stable tree.  Check out Documentation/stable_kernel_rules.txt
for the correct one.

Remember to copy Greg Kroah-Hartman <gregkh at linuxfoundation.org> who
is collecting serial patches believe.

Shawn




More information about the linux-arm-kernel mailing list