public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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