On Wed, 26 Mar 2008, Haavard Skinnemoen wrote:

> On Wed, 26 Mar 2008 15:16:44 +0100
> Wolfgang Denk <[EMAIL PROTECTED]> wrote:
> 
> > now that the new image  format  has  been  merged  into  mainline,  I
> > propose  that  we  make it mandatory for all newly added boards. That
> > means, that each new board that gets added shall at least contain
> > 
> >     #define CONFIG_FIT              1
> > 
> > in it's board config file; I recommend to add also
> > 
> >     #define CONFIG_FIT_VERBOSE      1 /* enable 
> > fit_format_{error,warning}() */
> > 
> > 
> > Can we agree on this?
> 
> Oh my...
> 
> Configuring for atstk1002 board...
>    text          data     bss     dec     hex filename
>  112936          8032  216860  337828   527a4 ./u-boot
> 
> with CONFIG_FIT and CONFIG_OF_LIBFDT enabled, compared to
> 
>    text          data     bss     dec     hex filename
>   87724          6840  215848  310412   4bc8c ./u-boot
> 
> on v1.3.2. That's 26KB...do we really want to make something _that_
> expensive mandatory? Or can I somehow disable the old code to reclaim
> some of it?

I'm getting 34KB increase iin the resulting binary just by enabling the 
required for FIT config macros in an arm11 build with the current git 
HEAD. Which is almost 30% increase... Sorry, I didn't follow the "new 
format" discussion, is it also going to be useful on ARM and on other 
platforms not using FDT?

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: [EMAIL PROTECTED]

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to