[U-Boot] [PATCH v2 09/25] common/cmd_*.c: sparse fixes

Kim Phillips kim.phillips at freescale.com
Fri Nov 9 02:04:30 CET 2012


On Thu, 8 Nov 2012 11:48:54 +0100
Wolfgang Denk <wd at denx.de> wrote:

> In message <1351553687-12469-10-git-send-email-kim.phillips at freescale.com> you wrote:
> >
> ...
> > Signed-off-by: Kim Phillips <kim.phillips at freescale.com>
> 
> Has anybody tested the impact of this change on the memory
> footprint?

without this commit, a P4080DS build yields:

   text	   data	    bss	    dec	    hex	filename
 404934	  14340	 280400	 699674	  aad1a	./u-boot

with this commit:

   text	   data	    bss	    dec	    hex	filename
 398593	  19896	 280400	 698889	  aaa09	./u-boot

so basically it's a tiny improvement overall.  The U_BOOT_CMD
changes moved the help text from the .rodata section into the .data
section (text).

> I object against the implementation.
> 
> The whole idea of the U_BOOT_CMD() macro was to be able to write 
> command entries in a _convenient_ way.  This patch makes this worse.
> 
> Please fix this such that we do NOT have to do all this ugly manual
> encoding of the help texts.

it's only in the case of long help text that changes with
configurations, and I'm not sure on how it can be avoided.

> Note also that this patch implements quite a few pretty much
> different changes - these should be split into separate commits.

it's been applied already, but I can resubmit as a series by sparse
error type if you want to revert it (it does so cleanly).

Kim



More information about the U-Boot mailing list