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: Thu, 05 Apr 2007 07:00:15 -0400	[thread overview]
Message-ID: <4614D6BF.9000100@gmail.com> (raw)
In-Reply-To: <20070405080611.460FE353B36@atlas.denx.de>

Wolfgang Denk wrote:
> In message <46146842.8060509@gmail.com> you wrote:
>> 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.
> 
> I agree that it should be *possible* to do this, if wanted.Similar
> like we can set up our own contents of thebootargs variable.
> 
> On the other hand, bootm should do everything that is necessary to
> start a kernel without such interaction, if needed.
> 
> Remember for example that bootm gets called automatically and without
> user interaction after download commands when "autostart" is  set  to
> "yes".
> 
> Now assume we set "autostart" to "yes" nd  use  "dhcp"  to  load  and
> start a multifile image containing the FDT. This is required to work,
> to.
> 
>> 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 
> 
> Agreed, but as an option, and without breaking compatibility of
> exiting methods.
> 
>> Much, perhaps most, possibly even all of the current stuff that is 
>> wedged into bootm is or should be passed through the blob.
> 
> I cannot parse this. Please explain what you have in mind.

Sorry, not my best example of English composition. :-/

What I was trying to say was that the linux bootargs (command line - in 
the chosen node), env variables (optional), and bd_t stuff (optional) is 
passed through the fdt blob now.  I was advocating that, by moving the 
creation of that stuff into the "fdt" command, we could simplify bootm. 
  Quite likely unrealistic, but it sounded good at the time.  Since then 
I got a solid 6 hrs sleep. ;-)

> Best regards,
> Wolfgang Denk
> 

Andy Fleming sent a suggestion that is really growing on me:
> What if we made it so if there isn't a chosen node in the blob when
> bootm is called, it fills in a default one.  This prevents some odd
> failures, and allows people to continue using device trees in the
> current manner, while still enabling the extra flexibility. 

<foghorn> That boy, I say, that boy has a /point/. </foghorn>

This means that, if you use "fdt chosen" to create the chosen node, you 
will be able to look at it and edit it and bootm will do the Right 
Thing[tm].  If you don't bootm will still do the Right Thing[tm] and 
autogenerate the necessary nodes.

Thanks,
gvb
<http://en.wikipedia.org/wiki/Foghorn_Leghorn>

  reply	other threads:[~2007-04-05 11:00 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
2007-04-05  8:06                   ` Wolfgang Denk
2007-04-05 11:00                     ` Jerry Van Baren [this message]
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=4614D6BF.9000100@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