public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jerry Van Baren <gvb.uboot@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Warning for mpc8360emds users: fdt-cmd from	u-boot-fdt.git
Date: Wed, 04 Apr 2007 23:08:50 -0400	[thread overview]
Message-ID: <46146842.8060509@gmail.com> (raw)
In-Reply-To: <20070404224643.7ABF2353A31@atlas.denx.de>

Wolfgang Denk wrote:
> In message <4613CF9F.4040501@smiths-aerospace.com> you wrote:
>> I view autogenerating fdt entries inside "bootm" as being evil. 
>> Obviously, if there are enough people on the Dark Side to overwhelm me, 
>> that can be changed.
> 
> "bootm" is intended to boot an  image  located  somewhere  in  system
> memory.  It  will  pass arguments like the booargs and the FDT to the
> kernel. It evaluates bootargs and substitured env vars for this. It
> might not be insane to expect that it also does reasonable things to
> the FDT.
> 
> I'm not in a position to determine what's reasonable there, though.
> 
> Best regards,
> 
> Wolfgang Denk

Well, I'm feeling my way around here too. :-/

If bootm edits/augments the FDT, the boot scripts/user has no chance to 
change the items it edits/augments (biggie: the chosen node), or even 
print it before linux is launched.  This defeats 90% of the purpose of 
the fdt command - allowing the user/script customize the blob before 
linux is launched.

My reasoning is that you can string together (or script) the proper 
"fdt" commands to do what previously was done by "bootm" in one step, so 
we are losing a small bit of convenience and gaining a whole lot of 
flexibility.  As a for instance, currently/previously you had to 
uncomment the blob dump call and recompile u-boot to get a dump of what 
is fed to linux - fugly.  With the New Improved[tm] behavior, you simply 
add "fdt print" to the super-bootm script and it Just Works[tm].  This 
is actually the itch that started me down this path. :-/

Much, perhaps most, possibly even all of the current stuff that is 
wedged into bootm is or should be passed through the blob.

Best regards,
gvb

  reply	other threads:[~2007-04-05  3:08 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-31 17:43 [U-Boot-Users] (Try 2) Please pull branch fdt-cmd from u-boot-fdt.git Jerry Van Baren
2007-03-31 18:20 ` Wolfgang Denk
2007-03-31 18:48   ` Jerry Van Baren
2007-04-03 23:50     ` Wolfgang Denk
2007-04-04 10:16       ` Jerry Van Baren
2007-04-04 12:22         ` [U-Boot-Users] Warning for mpc8360emds users: " Jerry Van Baren
2007-04-04 15:46           ` Timur Tabi
2007-04-04 16:17             ` Jerry Van Baren
2007-04-04 22:46               ` Wolfgang Denk
2007-04-05  3:08                 ` Jerry Van Baren [this message]
2007-04-05  8:06                   ` Wolfgang Denk
2007-04-05 11:00                     ` Jerry Van Baren
2007-04-05 18:02                       ` Bruce_Leonard at selinc.com
2007-04-05 18:12                         ` Jerry Van Baren
2007-04-05 18:40                           ` Bruce_Leonard at selinc.com
2007-04-06 21:57           ` Timur Tabi
2007-04-06 22:39             ` Jerry Van Baren
2007-04-07  0:15               ` Wolfgang Denk
2007-04-07  1:29                 ` Jerry Van Baren
2007-03-31 18:27 ` [U-Boot-Users] (Try 2) Please pull branch " Jerry Van Baren
2007-04-04  0:21   ` Wolfgang Denk
2007-04-03  9:39 ` Joakim Tjernlund
2007-04-03 10:34   ` Jerry Van Baren
2007-04-03 11:37     ` Joakim Tjernlund
2007-04-03 12:06       ` Jerry Van Baren
2007-04-03 12:59       ` [U-Boot-Users] dtb in env sector - was: (Try 2) Please pull Wolfgang Denk
2007-04-03 14:04         ` Joakim Tjernlund
2007-04-03 14:21           ` Jerry Van Baren
2007-04-03 14:36             ` Martin Krause
2007-04-03 15:14             ` Joakim Tjernlund
2007-04-03 15:17               ` Jerry Van Baren
2007-04-03 15:24             ` Wolfgang Denk
2007-04-03 18:53               ` Joakim Tjernlund
2007-04-03 12:53     ` Wolfgang Denk
2007-04-03 12:49   ` Wolfgang Denk
2007-04-03 13:58     ` Joakim Tjernlund
2007-04-03 15:18       ` 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=46146842.8060509@gmail.com \
    --to=gvb.uboot@gmail.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