From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Honor /memory/reg node in DTB files
Date: Wed, 08 Dec 2010 23:34:05 +0100 [thread overview]
Message-ID: <20101208223405.6AC07CF5CA6@gemini.denx.de> (raw)
In-Reply-To: <0DDCBDA1-188F-433D-BDCC-5FDCF709A131@digitaldans.com>
Dear Dan,
In message <0DDCBDA1-188F-433D-BDCC-5FDCF709A131@digitaldans.com> you wrote:
>
> > If you want to make this switchable at runtime, then we should
> > probably use an environment setting.
>
> I experimented with this, but could never determine the
> best way to cover all behavior. Do we have a variable that
> indicates "don't update DT?" If so, it means we have to
> place all values in the DT, when it's really nice for U-Boot
> to do some of that for us. In fact, we want U-Boot to update
> many things it discovers, just not the few we wish to actually
> (sometimes) define for ourselves.
"You can please all the people some of the time and some of the people
all of the time but you can't please all the people all of the time."
> The other alternative (granted, I'm not as smart as I used
> to be :-)) was to have an environment variable that specified
> which node we didn't want updated by U-Boot. For now,
> that seems reasonable since there is only one, but is that
> the general approach we want in the future? It also presents
> the challenge of having to update both environment and
> the provided DT.
I guess we can argue that the normal situation is that U-Boot will
know how to update the DT such as needed to boot the OS. So what we
are dealing with is a small percentage of cases where we need special
behaviour, and where it may be acceptable if the solution is only
semi-perfect ;-)
My current thinking is to introduce something like
dt_skip=memory,mac-address
including eventually "dt_skip=ALL". This should cover most of the
current use cases.
If someone gets fancy he can add wildcard support.
And if we need even more flexibility, we can add some "dt_include"
with higher priority, so one could do for example
dt_skip=ALL
dt_include=memory
to have only the memory node updated.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If you can't beat it or corrupt it, you pretend it was your idea in
the first place. - Terry Pratchett, _Guards! Guards!_
next prev parent reply other threads:[~2010-12-08 22:34 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-07 0:56 [U-Boot] [PATCH] Honor /memory/reg node in DTB files Deepak Saxena
2010-12-07 6:52 ` Wolfgang Denk
2010-12-07 18:05 ` Deepak Saxena
2010-12-07 19:09 ` Wolfgang Denk
2010-12-08 18:30 ` Hollis Blanchard
2010-12-08 20:53 ` Wolfgang Denk
2010-12-08 21:08 ` Hollis Blanchard
2010-12-08 21:38 ` Wolfgang Denk
2010-12-08 22:08 ` Dan Malek
2010-12-08 22:34 ` Wolfgang Denk [this message]
2010-12-08 23:33 ` Peter Tyser
2010-12-08 23:59 ` Dan Malek
2010-12-09 10:00 ` Wolfgang Denk
2010-12-10 18:04 ` Deepak Saxena
2010-12-12 21:19 ` Wolfgang Denk
2010-12-08 18:59 ` Deepak Saxena
2010-12-08 21:00 ` Wolfgang Denk
2010-12-07 18:40 ` Hollis Blanchard
2010-12-07 19:21 ` Wolfgang Denk
2010-12-07 21:22 ` Scott Wood
2010-12-08 18:59 ` Deepak Saxena
2010-12-08 19:11 ` Scott Wood
2010-12-08 19:22 ` Dan Malek
2010-12-08 19:33 ` Scott Wood
2012-03-31 19:32 ` Marek Vasut
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=20101208223405.6AC07CF5CA6@gemini.denx.de \
--to=wd@denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox