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