From: Jerry Van Baren <gerald.vanbaren@ge.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] A simple howto boot sequoia (ARCH=powerpc) using flattened device tree (FDT)
Date: Fri, 23 May 2008 08:57:51 -0400 [thread overview]
Message-ID: <4836BF4F.7060002@ge.com> (raw)
In-Reply-To: <200805231429.42728.sr@denx.de>
Stefan Roese wrote:
> On Friday 23 May 2008, Jerry Van Baren wrote:
[snip]
>> It would be relatively easy to do c) by creating Yet Another Config
>> Variable that bootm used for the "force" flag rather than hardcoding it.
>> I'm not wild about creating YACV. Adding Yet Another Parameter to
>> bootm is probably as bad or worse. Hmmmm.
>
> I'm wondering who really wants to *not* override the chosen bootargs property?
> I'm probably missing something here, but for me it just makes no sense to not
> override the bootargs property with the U-Boot bootargs env variable.
>
> Best regards,
> Stefan
Like most decisions, we kinda backed into it in an effort to mimic
previous behavior at the start, and then tweaking our decision and code
repeatedly over time.
The original fdt support (CONFIG_OF_FLAT_TREE) did not touch the /chosen
node *at all* if it existed. I carried this behavior forward with the
libfdt version under the banner of compatibility. Subsequently, Scott
Wood suggested and one of us implemented a fine-grained behavior where
we would create /chosen properties if they didn't exist but not override
existing /chosen properties.
It may be time to reevaluate our assumptions and decisions.
Observations:
* Removing the "force" option would simplify the code.
* I've been threatening to remove the deprecated CONFIG_OF_FLAT_TREE
for a while now. Maybe it is time, maybe I /make/ time, to do it
this window.
Best regards,
gvb
next prev parent reply other threads:[~2008-05-23 12:57 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-14 20:54 [U-Boot-Users] A simple howto boot sequoia (ARCH=powerpc) using flattened device tree (FDT) Niklaus Giger
2008-05-22 16:58 ` Larry Johnson
2008-05-23 8:04 ` Stefan Roese
2008-05-23 12:12 ` Jerry Van Baren
2008-05-23 12:29 ` Stefan Roese
2008-05-23 12:57 ` Jerry Van Baren [this message]
2008-05-23 13:36 ` Stefan Roese
2008-05-23 15:10 ` 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=4836BF4F.7060002@ge.com \
--to=gerald.vanbaren@ge.com \
--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