From: Grant Likely <grant.likely@secretlab.ca>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 3/3] powerpc: Add support for ram filesystems in FIT uImages
Date: Wed, 30 Dec 2009 17:01:35 -0700 [thread overview]
Message-ID: <fa686aa40912301601s6cd0ec4y85b88976159a36af@mail.gmail.com> (raw)
In-Reply-To: <1262216393.29396.41.camel@localhost.localdomain>
On Wed, Dec 30, 2009 at 4:39 PM, Peter Tyser <ptyser@xes-inc.com> wrote:
> Hi Grant,
> I put U-Boot ML on CC.
Thinking further, I do actually have another concern, at least with
regard to the way the current patch set implements things. Is it
expected or even "recommended" that fit images will *always* contain a
.dtb image? The current patch only handles the case of a .dtb
embedded inside the fit image.
Personally, I don't get any benefit out of the new image format, so I
haven't spent any time looking at it. However, I'm concerned about
the drift back towards a different image per target when the move over
the last 4 years has been towards multiplatform kernel images. I
certainly don't want to encourage embedding the device tree blob into
the kernel image, and I'm not very interested in merging code to do
that into the kernel tree. If someone really needs to do that for
their particular target, it is certainly easy enough for them to weld
in the .dtb after the fact before transferring the image to the
target, but I want that mode to be the exception, not the rule.
g.
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
next prev parent reply other threads:[~2009-12-31 0:01 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1261446643-21714-1-git-send-email-ptyser@xes-inc.com>
[not found] ` <1261446643-21714-4-git-send-email-ptyser@xes-inc.com>
[not found] ` <fa686aa40912301502x48785614ya4dd5c71815a7633@mail.gmail.com>
2009-12-30 23:39 ` [U-Boot] [PATCH v2 3/3] powerpc: Add support for ram filesystems in FIT uImages Peter Tyser
2009-12-31 0:01 ` Grant Likely [this message]
2009-12-31 1:10 ` Peter Tyser
2010-01-03 5:08 ` Grant Likely
2010-01-03 10:10 ` Wolfgang Denk
2010-01-04 1:07 ` Peter Tyser
2010-01-04 8:27 ` Grant Likely
2009-12-31 8:01 ` Peter Korsgaard
2010-01-01 14:12 ` Wolfgang Denk
2010-01-03 5:18 ` Grant Likely
2010-01-03 10:15 ` 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=fa686aa40912301601s6cd0ec4y85b88976159a36af@mail.gmail.com \
--to=grant.likely@secretlab.ca \
--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