All of lore.kernel.org
 help / color / mirror / Atom feed
From: Manfred Huber <man.huber@arcor.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/1 v2] omap3_beagle: Enabling UART3 first allows the Transmitter to be empty
Date: Fri, 29 Mar 2013 09:33:47 +0100	[thread overview]
Message-ID: <515551EB.8020405@arcor.de> (raw)
In-Reply-To: <5154033D.6090003@gmail.com>

Am 28.03.2013 09:45, schrieb Andreas Bie?mann:
> Dear Manfred Huber,
>
> On 03/28/2013 07:06 AM, Manfred Huber wrote:
>> On 2013-03-27 14:37, Andreas Bie?mann wrote:
>
> <snip>
>
>>> On 03/25/2013 11:02 PM, Manfred Huber wrote:
>
> <snip>
>
>>>> +        serial_out(UART_LCR_DLAB, &com_port->lcr);
>>>> +        serial_out(baud_divisor & 0xff, &com_port->dll);
>>>> +        serial_out((baud_divisor >> 8) & 0xff, &com_port->dlm);
>>>> +        serial_out(UART_LCRVAL, &com_port->lcr);
>>>> +        serial_out(0, &com_port->mdr1);
>>>
<snip>
> I wonder which use-case requires UART flushing in u-boot context before
> initializing the UART for u-boot correctly. Can someone explain this to
> me? Shouldn't we always start here from the very beginning and setup
> UART as configured?
Beagleboard has several ways to boot (NAND, SD/MMC, UART, ...). For the 
boot mode with UART, Beagleboard configures the UART and ends with a non 
empty transmitter. In a booting sequence where UART is before NAND, 
SD/MMC or wherever SPL starts from, we have not a clean UART.
>
<snip>

>>
>> It's not critical. So I guess it's not needed for this release.
>
> Well, if there are boards in the field that will not boot with the next
> release I think it is critical.
> We do have some omap3 (omap35xx and am37xx) based boards here. I can
> recall a situation where some few boards did not boot from sd-card while
> serial debug cable was attached (AFAIR this was not the case when
> booting from NAND). The root cause was never investigated, so maybe we
> suffered exactly this bug.
Can you test this boars with my patch?
>
> Best regards
>
> Andreas Bie?mann
>

  parent reply	other threads:[~2013-03-29  8:33 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-27 13:09 [U-Boot] Beagleboard: SPL hangs on serial init man.huber at arcor.de
2013-03-16 13:13 ` Manfred Huber
2013-03-19 14:49   ` Tom Rini
2013-03-19 23:52     ` Manfred Huber
2013-03-20  0:05     ` Javier Martinez Canillas
2013-03-20  1:27       ` Tom Rini
2013-03-20 23:09         ` Manfred Huber
2013-03-21 21:08           ` Javier Martinez Canillas
2013-03-23 10:11             ` Manfred Huber
2013-03-21 19:03       ` [U-Boot] [PATCH] omap3_beagle: Enable CONFIG_SYS_NS16550_BROKEN_TEMT Manfred Huber
2013-03-21 21:28         ` Javier Martinez Canillas
2013-03-21 22:21         ` Tom Rini
2013-03-21 22:28           ` Scott Wood
2013-03-25 22:02       ` [U-Boot] [PATCH 1/1 v2] omap3_beagle: Enabling UART3 first allows the Transmitter to be empty Manfred Huber
2013-03-27  4:50         ` Manfred Huber
2013-03-27  9:29           ` Javier Martinez Canillas
2013-03-27 13:57             ` Tom Rini
2013-03-28  5:55             ` Manfred Huber
2013-03-29  8:19             ` Manfred Huber
2013-03-28 15:21           ` Tom Rini
2013-03-27 13:37         ` Andreas Bießmann
2013-03-27 17:22           ` Javier Martinez Canillas
2013-03-28  6:06           ` Manfred Huber
2013-03-28  8:45             ` Andreas Bießmann
2013-03-28  9:11               ` Javier Martinez Canillas
2013-03-28  9:50                 ` Andreas Bießmann
2013-03-28 15:21                   ` Tom Rini
2013-03-29  8:33               ` Manfred Huber [this message]
2013-03-29  9:20       ` [U-Boot] [PATCH 1/1 v3] " Manfred Huber
2013-03-29  9:43         ` Albert ARIBAUD
2013-03-29 12:34           ` Tom Rini
2013-03-29 12:42       ` [U-Boot] [PATCH 1/1 v4] omap3_beagle: Flush UART3 xmit on enable if TEMT is broken Manfred Huber
2013-03-29 12:52       ` [U-Boot] [PATCH 1/1 v5] " Manfred Huber
2013-04-02  7:46         ` Javier Martinez Canillas
2013-04-02  8:59         ` Andreas Bießmann
2013-04-08 16:56         ` [U-Boot] [U-Boot, 1/1, " Tom Rini
2013-04-10 22:12       ` [U-Boot] [PATCH v1 1/1] omap3: Display MHz instead of mHz on the console Manfred Huber

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=515551EB.8020405@arcor.de \
    --to=man.huber@arcor.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.