[PATCH v2 4/5] ARM: ep93xx: fix inverted RTS/DTR signals on uart1

H Hartley Sweeten hartleys at visionengravers.com
Mon Jun 13 13:38:15 EDT 2011


On Sunday, June 12, 2011 8:05 AM, Petr Štetiar wrote:
>
> It was discovered by Roberto Bergo, that RTS/DTR signals are inverted after
> the boot, because it was causing him problems with hardware controlled modem
> connected on ttyAM0. Todd Valentic came with this patch for the issue.
>
> Discussion: http://tech.groups.yahoo.com/group/ts-7000/message/20259
>
> Cc: Ryan Mallon <ryan at bluewatersys.com>
> Cc: Hartley Sweeten <hsweeten at visionengravers.com>
> Signed-off-by: Todd Valentic <todd.valentic at sri.com>
> Tested-by: Roberto Bergo <roberto.bergo at robson.it>
> Signed-off-by: Petr Štetiar <ynezz at true.cz>
> ---
>  arch/arm/mach-ep93xx/core.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-ep93xx/core.c b/arch/arm/mach-ep93xx/core.c
> index 8207954..cf1ce32 100644
> --- a/arch/arm/mach-ep93xx/core.c
> +++ b/arch/arm/mach-ep93xx/core.c
> @@ -251,9 +251,9 @@ static void ep93xx_uart_set_mctrl(struct amba_device *dev,
>  	unsigned int mcr;
>  
>  	mcr = 0;
> -	if (!(mctrl & TIOCM_RTS))
> +	if (mctrl & TIOCM_RTS)
>  		mcr |= 2;
> -	if (!(mctrl & TIOCM_DTR))
> +	if (mctrl & TIOCM_DTR)
>  		mcr |= 1;
>  
>  	__raw_writel(mcr, base + EP93XX_UART_MCR_OFFSET);

This looks correct.

Acked-by: H Hartley Sweeten <hsweeten at visionengravers.com>

Please add the patch to Russell's Patch Tracker.

Thanks,
Hartley



More information about the linux-arm-kernel mailing list