public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jerry Van Baren <gerald.vanbaren@smiths-aerospace.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 14:12:58 -0400	[thread overview]
Message-ID: <46153C2A.3050206@smiths-aerospace.com> (raw)
In-Reply-To: <OFC7BE117C.A87186A7-ON882572B4.0061BD27-882572B4.00631626@selinc.com>

Bruce_Leonard at selinc.com wrote:
> Hi all
> 
> u-boot-users-bounces at lists.sourceforge.net wrote on 04/05/2007 04:00:15 
> AM:
> 
>> 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.
>>>
> <snip snip>
> 
> My two cents worth (for what it's worth :-/): from the standpoint of 
> someone using uboot for the first time and having to learn from the ground 
> up without the benefit of having used this stuff for many years, I agree 
> with Wolfgang.  It took me two weeks just to figure out what a device tree 
> is and that I even needed one.  It took me two more days to figure out how 
> to create the blob and how to use it.  I still don't know the details of 
> DTS files, what needs to be in them or what the different fields mean. 
> Adding another step/level of obscurity with REQUIRING the use of fdt 
> commands and/or scripts is just another barrier to new users.  And I have 
> to tell you, this thing is a bear to learn.  For folks who have been 
> digging around in the guts of it, I'm sure it's trivial.  But I at least 
> am pretty overwhelmed by it.
> 
> I think it would be great to have the option of using the ftd commands if 
> it suited your purpose, but still be able to use things as they currently 
> are.  That would be the most flexible, give the expert users something 
> they want, and not add yet another thing that new people HAVE to learn 
> just to get an OS to boot.
> 
> Again just my 2 cents.  Flames welcome.
> 
> Bruce

Yes, but now you are worth 5 figures more: expect a $00,000 raise as 
part of your next performance appraisal.

I plan to implement Andy Fleming's suggestion which will resolve the issue:
 > > 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.

gvb
Pedantic Script: device trees and blobs are kernel things, they ain't 
our fault. :-P Yet. ;-)

  reply	other threads:[~2007-04-05 18:12 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
2007-04-05 18:02                       ` Bruce_Leonard at selinc.com
2007-04-05 18:12                         ` Jerry Van Baren [this message]
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=46153C2A.3050206@smiths-aerospace.com \
    --to=gerald.vanbaren@smiths-aerospace.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