[U-Boot] [PATCH v2 REPOST 2/3] image: Implement IH_TYPE_KERNEL_REL

Stephen Warren swarren at nvidia.com
Thu Oct 20 21:52:56 CEST 2011


Wolfgang Denk wrote at Thursday, October 20, 2011 1:39 PM: 
> In message <CAPnjgZ2BqZhBEHt6ha7pFi7syLz9e9Fm8MNAw_O7M-krN37UnQ at mail.gmail.com> you wrote:
> >
> > My comment was that if code size could increase slightly the code
> > could have less #ifdefs. Wolfgang said that it could not, and
> > therefore my comments do not apply. Please ignore them Wolfgang and
> > Stephen.
> 
> DOn;t give up to oearly.  My opinion is that #ifdef's are bad, and
> increased code size is bad either.  If we can trade off one for the
> other, we should know exactly what we can win here and how much we
> have to pay for that there.  That's all I want to know.

Do you mean the total without /any/ ifdefs? If so, please see where I
answered this question earlier in the thread:

http://lists.denx.de/pipermail/u-boot/2011-October/104427.html

The first time I submitted this patch, there were no ifdefs, and you
stated that the additional 576 bytes text plus 136 bytes rodata was
unacceptable, so I rewrote the patch to add ifdefs everywhere.

-- 
nvpublic



More information about the U-Boot mailing list