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: Thu, 13 Dec 2007 23:41:44 +0100	[thread overview]
Message-ID: <4761B528.3080601@semihalf.com> (raw)
In-Reply-To: <475EE3AE.6060102@ge.com>


Hi Jerry,

Thanks for your comments, see my replies below.

Jerry Van Baren wrote:
[...]
>> 3. 'New image' format must support the following features:
> 
> [snip]
> 
>> 	- 'container' image blob shall include 'component' images'
>> 	  data, which means direct data embedding  - as opposed to
>> 	  having only references
> 
> Q for Jon L: This would require an extension to the dtc to "include" a 
> raw file into the blob?  I'm presuming that we don't want to take a 
> binary (ELF) file, turn it into ASCII bytes, include it into a dts, and 
> then use dtc to compile it back into binary.
> 
> Am I missing something that is already available?  Do you see any 
> problems with extending dtc to support this?

AFAIK dtc currently has no support for data includes. I've seen such
feature on a dtc wish list though, so adding it should not be troublesome.


>>   (b) 'container' source file (.dts)
>>
>> 	- the following bindings and properties shall be defined for a device
>> 	  tree source file (.dts) that is corresponding to a 'container'
>> 	  image blob:
>>
>> 		- root node of the DTS shall represent a 'container' node
>> 		- 'container' node shall support:
>> 			- label property
>> 			- timestamp property
>> 		- 'container' node shall support multiple 'component' subnodes
>>
>> 		- 'component' subnode shall support:
>> 			- label property
>> 			- type property
>> 			- hash properties (crc32, md5, sha1, etc.)
> 
> Would hash be two entries (type and value), or would it be just the type 
> and use conventions for where the value is stored (i.e. last /n/ bytes 
> of the image)?  I would vote for (type and value) if this were a democracy.
>
> Are image hashes to validate what is stored in the blob (compressed) or 
> to validate what is in memory after decompressing?  (Ability to support 
> both options would be very good IMHO.)

Agree, entries are much more flexible, e.g. we can easily add third
entry which will distinguish compressed/uncompressed data hashes.

>>
>> 	- detailed image bindings description shall be provided as a separate
>> 	  document (e.g. wiki web page)
> 
> I vote for capturing it git as a text document equivalent to 
> booting_without_of.txt (fdt_images.txt?).

I mentioned wiki as it's easier to update and work with, but I am fine
with the git txt as well.

> (Do we need a [Dd]ocumentation subdirectory?)

Not sure, there is a doc directory which contains board README files.
And there is a (quite large) README file that documents a lot of U-boot
internals. I guess we may want to hear Wolfgang's opinion?

Cheers,
Marian

  parent reply	other threads:[~2007-12-13 22:41 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 [this message]
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
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=4761B528.3080601@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