public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] serial_pl011: Set RTS during initialization
@ 2012-12-03  3:09 Rob Herring
  2012-12-03  4:56 ` Marek Vasut
  2012-12-07 15:51 ` [U-Boot] " Tom Rini
  0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring @ 2012-12-03  3:09 UTC (permalink / raw)
  To: u-boot

From: Joshua Housh <joshua.housh@calxeda.com>

If the pl011 is connected to another device which has hardware
flow-control on, characters are never received by the pl011.
Asserting RTS when flow-control is off will have no effect.

This is in line with how Linux behaves.

Signed-off-by: Joshua Housh <joshua.housh@calxeda.com>
---
 drivers/serial/serial_pl01x.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/serial/serial_pl01x.c b/drivers/serial/serial_pl01x.c
index b331be7..dfdba9f 100644
--- a/drivers/serial/serial_pl01x.c
+++ b/drivers/serial/serial_pl01x.c
@@ -163,8 +163,8 @@ static int pl01x_serial_init(void)
 	}
 #endif
 	/* Finally, enable the UART */
-	writel(UART_PL011_CR_UARTEN | UART_PL011_CR_TXE | UART_PL011_CR_RXE,
-	       &regs->pl011_cr);
+	writel(UART_PL011_CR_UARTEN | UART_PL011_CR_TXE | UART_PL011_CR_RXE |
+	       UART_PL011_CR_RTS, &regs->pl011_cr);
 
 	return 0;
 }
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] serial_pl011: Set RTS during initialization
  2012-12-03  3:09 [U-Boot] [PATCH] serial_pl011: Set RTS during initialization Rob Herring
@ 2012-12-03  4:56 ` Marek Vasut
  2012-12-07 15:51 ` [U-Boot] " Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Marek Vasut @ 2012-12-03  4:56 UTC (permalink / raw)
  To: u-boot

Hi Rob, Joshua,

> From: Joshua Housh <joshua.housh@calxeda.com>
> 
> If the pl011 is connected to another device which has hardware
> flow-control on, characters are never received by the pl011.
> Asserting RTS when flow-control is off will have no effect.
> 
> This is in line with how Linux behaves.
[...]

Tested-by: Marek Vasut <marex@denx.de>

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] serial_pl011: Set RTS during initialization
  2012-12-03  3:09 [U-Boot] [PATCH] serial_pl011: Set RTS during initialization Rob Herring
  2012-12-03  4:56 ` Marek Vasut
@ 2012-12-07 15:51 ` Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2012-12-07 15:51 UTC (permalink / raw)
  To: u-boot

On Sun, Dec 02, 2012 at 05:09:26PM -0000, Rob Herring wrote:

> From: Joshua Housh <joshua.housh@calxeda.com>
> 
> If the pl011 is connected to another device which has hardware
> flow-control on, characters are never received by the pl011.
> Asserting RTS when flow-control is off will have no effect.
> 
> This is in line with how Linux behaves.
> 
> Signed-off-by: Joshua Housh <joshua.housh@calxeda.com>
> Tested-by: Marek Vasut <marex@denx.de>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20121207/7dab7c25/attachment.pgp>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-12-07 15:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-03  3:09 [U-Boot] [PATCH] serial_pl011: Set RTS during initialization Rob Herring
2012-12-03  4:56 ` Marek Vasut
2012-12-07 15:51 ` [U-Boot] " Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox