[U-Boot] [PATCH] common, image: introduce new uimage types

Wolfgang Denk wd at denx.de
Mon Jan 16 09:51:01 CET 2012


Dear Heiko Schocher,

In message <4F13C325.3010200 at denx.de> you wrote:
> 
> >> - ubootimage
> >>   U-Boot image
> >> - splimage
> >>   SPL image
> >> - dfenvimage
> >>   Default Environment image
> >> - ublheader
> >>   TI davinci UBL header image
> > 
> > for ... ?
> 
> Oh, right, forgotten to add this in the commit message.
> 
> They are needed for cam_enc_4xx update:
> 
> http://patchwork.ozlabs.org/patch/136165/
> 
> There, the UBL-header, SPL-image, U-Boot image and an default image
> are wrapped in one FIT image.

But this is conceptually broken.  The IH_TYPE_* definitions are
actually only relevant for legacy images.

FIX images use an XML based, extendable data format.  The whole idea
of this is that we do NOT have to change to code when anybody needs a
new (probably private) entry or type or whatever.

In your case, it should be possible to use an existing image type
(say, IH_TYPE_FIRMWARE) for your code, and use other distinguishing
marks to determine which parts you work on.  Probably it makes sense
to define your own entry for each of the parts.  Then only your own
code needs to understand this.

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Any time things appear to be going better, you have overlooked  some-
thing.


More information about the U-Boot mailing list