[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