[U-Boot] [PATCH v4 2/5] spl: don't mark __u_boot_cmd* as undefined

Ilya Yanok ilya.yanok at cogentembedded.com
Tue Aug 7 10:12:09 CEST 2012


Hi Tom,

On Tue, Aug 7, 2012 at 1:11 AM, Ilya Yanok <ilya.yanok at cogentembedded.com>wrote:

>
> Yes.  What I meant was that not all of the stuff that is guarded today
>> is garbage collected so the resulting image is larger than it must be.
>>
>
> Yep. And that's actually goes beyond the subject of this patch: as long as
> we rely on linker GC and not putting guards by hands we will get unused
> string literals compiled in.
>
> I still think that this patch does the correct things (UNDEF_SYM is there
> to protect commands from GC and we don't really want commands in SPL so
> there is nothing to protect). But I will try to add guards to make net-spl
> compile even without this patch.
>

I've just posted v5 for patch 4/5 that doesn't depend on this patch. It
really has some improvement wrt size (~50KB net-only SPL image) so probably
it could be useful not only for AM33xx...

Regards, Ilya.


More information about the U-Boot mailing list