public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marian Balakowicz <m8@semihalf.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] RFC: New U-boot image format
Date: Wed, 19 Dec 2007 15:07:09 +0100	[thread overview]
Message-ID: <4769258D.6090709@semihalf.com> (raw)
In-Reply-To: <4767E487.8020103@freescale.com>

Timur Tabi wrote:
> Marian Balakowicz wrote:
>>
>> New format for U-boot images has been on the list few times already.
>> There were different ideas and features discussed but no final
>> conclusion has been made.
> 
> I have a request for a new feature.  I think we need image-format
> plug-ins. That is, we need a way for a plug-in to register itself with
> the main format processing code.  When the processing code sees a blob
> that it doesn't understand, it calls the plug-in to handle it.

The hooks idea sounds reasonable but, the problem
is that there isn't really any such thing like
"main fromat processing code".

The 'bootm' command code does a lot of image
related manipulation but it's not generic or main
format processing code. It is rather a specialized
command that handles one thing, starting
executables, like kernels or standalone
applications. And it's a command, it will not do
anything unless directly called.

So, having hooks in bootm, you would need to call
bootm and then your 'qe fw' command which is sort
of wired.

There are also other commands that deal with the
images, these are other boot commands like
scsiboot, usbboot, etc, and also imls, imxtract.
And all of them process images directly, finding
necessary information on their own.

Your command seem to be of the same type, it needs
information from the uImage (soon from the New
uImage) and it knows best what it needs.
IMHO, reasonable is to make it get this
information from New uImage directly.

Your current syntax is:
> qe fw <addr>

Extended syntax could be:
qe fw 200000:fw_image at 1

Where 200000 is the address of New uImage and
"fw_image at 1" is a example node name of your blob
data. The New uImage API will allow, in simple way
(one, two calls),  to get other details for a
given image, so you would be able to get the blob
data start address.

More on the New uImage format soon, stay tuned.

> This would be a handy way to handle stuff like the QE firmware binary
> blob format (see the thread titled "[PATCH] 85xx: add ability to upload
> QE firmware").  The format of the QE firmware blob has already been
> decided, so all I would need from the multi-image format is:
> 
> 1) A way to package my blob.
> 2) A way to pass the address of the blob to the QE code
> 
> For option #2, setting an environment variable would be the easiest.  To
> do that, the QE code could register a call-back that says, "If you see a
> QE firmware blob, call this function and pass the address of the blob".

Cheers,
Marian

  parent reply	other threads:[~2007-12-19 14:07 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-11 16:18 [U-Boot-Users] RFC: New U-boot image format Marian Balakowicz
2007-12-11 16:32 ` [U-Boot-Users] RFC: New U-boot image format - open issues Marian Balakowicz
2007-12-11 19:23 ` [U-Boot-Users] RFC: New U-boot image format Jerry Van Baren
2007-12-11 22:23   ` Wolfgang Denk
2007-12-12 12:38     ` Jerry Van Baren
2007-12-13 22:59       ` Marian Balakowicz
2007-12-13 22:41   ` Marian Balakowicz
2007-12-14  6:17     ` Wolfgang Denk
2007-12-14 21:44     ` T Ziomek
2007-12-19 14:26     ` Marian Balakowicz
2007-12-20  0:50       ` David Gibson
2007-12-20 16:41         ` Scott Wood
2007-12-20 22:25           ` Marian Balakowicz
2007-12-20 22:39             ` Scott Wood
2007-12-21  0:17               ` David Gibson
2007-12-18 15:17 ` Timur Tabi
2007-12-18 15:33   ` Jerry Van Baren
2007-12-18 15:36     ` Timur Tabi
2007-12-18 16:12   ` Wolfgang Denk
2007-12-18 16:17     ` Timur Tabi
2007-12-18 19:42       ` Wolfgang Denk
2007-12-18 19:47         ` Timur Tabi
2007-12-18 20:11           ` Wolfgang Denk
2007-12-18 20:18             ` Timur Tabi
2007-12-19 14:07   ` Marian Balakowicz [this message]
2007-12-20 19:42     ` Wolfgang Denk
2007-12-21 15:04       ` Marian Balakowicz
2007-12-21 15:17         ` Wolfgang Denk
2007-12-21 15:39           ` Marian Balakowicz

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=4769258D.6090709@semihalf.com \
    --to=m8@semihalf.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