public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ns16550: allow UART address to be set dynamically
Date: Fri, 14 Dec 2012 14:52:31 -0700	[thread overview]
Message-ID: <50CB9F9F.5010402@wwwdotorg.org> (raw)
In-Reply-To: <50CB8ED1.7020503@ti.com>

On 12/14/2012 01:40 PM, Tom Rini wrote:
> On 12/13/12 16:51, Simon Glass wrote:
> 
> [snip]
>>>> And from there we can move on and say "On ${SoC} we get a 
>>>> device tree (that we can't quite parse as we don't have
>>>> enough resources) AND $some-data (OMDATA or an abbreviated
>>>> device tree or $whatever), lets translate that into something
>>>> we can make use of very early rather than a hard-coded
>>>> initial console location"
>>> 
>>> It seems like you're saying that once we have dynamic serial 
>>> port assignment working based on DT, you'll be fine using
>>> ODMDATA to initialize the early console, but not before then?
>>> If so, I'm having a hard time understanding why enabling the
>>> DT-based support blocks using ODMDATA, since the code would be
>>> pretty orthogonal.
> 
>> Yes well dynamic console selection sounds find to me, ODMDATA or
>>  otherwise. To me it is a Tegra feature that should be supported
>> as such. Perhaps we can allow the FDT console alias to specify 
>> "odmdata" to mean that, and/or (as you suggest I think) set the 
>> console to USE_ODMDATA, which then selects
>> CONFIG_SYS_NS16550_COMx accordingly.
> 
> There's two parts to it.  One part is that sure, Tegra and only
> Tegra has ODMDATA.  But on am33xx if we poke the i2c eeprom (on
> platforms that do the eeprom) we can then know ...  And I bet other
> SoCs have other tricks for this or that.  So it's not just tegra
> that can tell us the initial console is $here or $there if we just
> ...something.

That's certainly true.

I personally view the method of retrieving this kind of information as
part of an SoC's boot architecture, or as part of a board's design. As
you have mentioned above, different SoCs/boards already have
mechanisms to represent/determine this information. These mechanisms
are already in-place and defined by the SoC or board designers.

> The other part is, take a look at the Allwinner thread from a week
> or so ago.  We really need to define how we want early board
> specific data to come in because if we start saying we'll accept
> per-SoC solutions we'll be drowning in them in short time.  We want
> to say here's our preferred way to pass this information in.

I don't understand why you think U-Boot is in a position to mandate
that the existing solutions that are already in place are incorrect,
and must be replaced with some alternative.

  parent reply	other threads:[~2012-12-14 21:52 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-12 23:23 [U-Boot] [PATCH] ns16550: allow UART address to be set dynamically Stephen Warren
2012-12-12 23:38 ` Simon Glass
2012-12-12 23:52   ` Stephen Warren
2012-12-13  0:38     ` Simon Glass
2012-12-13 10:29     ` Wolfgang Denk
2012-12-13 18:17       ` Stephen Warren
2012-12-13 20:36         ` Wolfgang Denk
2012-12-13 20:45           ` Stephen Warren
2012-12-13 20:53             ` Tom Rini
2012-12-13 21:07               ` Stephen Warren
2012-12-13 21:51                 ` Simon Glass
2012-12-14 20:40                   ` Tom Rini
2012-12-14 21:14                     ` Simon Glass
2012-12-14 22:03                       ` Stephen Warren
2012-12-14 22:22                         ` Simon Glass
2012-12-14 22:45                           ` Stephen Warren
2012-12-17 21:09                             ` Tom Rini
2012-12-17 22:24                               ` Stephen Warren
2012-12-17 22:37                                 ` Wolfgang Denk
2012-12-17 22:58                                   ` Stephen Warren
2012-12-18  6:39                                     ` Wolfgang Denk
2012-12-18 16:37                                       ` Stephen Warren
2012-12-18 19:15                                         ` Simon Glass
2012-12-17 21:09                           ` Tom Rini
2012-12-14 22:35                         ` Wolfgang Denk
2012-12-14 21:52                     ` Stephen Warren [this message]
2012-12-14 22:31                       ` Wolfgang Denk
2012-12-14 22:26                     ` Wolfgang Denk
2012-12-14 23:16                       ` Graeme Russ
2012-12-15  0:32                         ` Wolfgang Denk
2012-12-15  1:32                           ` Graeme Russ
2012-12-15  7:30                             ` Wolfgang Denk
2012-12-15  9:53                               ` Graeme Russ
2012-12-17 21:04                       ` Tom Rini
2012-12-13 23:11             ` Wolfgang Denk
2012-12-13 23:26               ` Stephen Warren
2012-12-13 10:27 ` Wolfgang Denk
2012-12-13 13:11   ` Tom Rini
2012-12-13 14:22     ` Wolfgang Denk

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=50CB9F9F.5010402@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox