[U-Boot-Users] RFC: make new image support mandatory

Guennadi Liakhovetski lg at denx.de
Wed Mar 26 21:37:02 CET 2008


On Wed, 26 Mar 2008, Haavard Skinnemoen wrote:

> On Wed, 26 Mar 2008 15:16:44 +0100
> Wolfgang Denk <wd at denx.de> 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: office at denx.de




More information about the U-Boot mailing list