[U-Boot] [PATCH] relocation: Do not relocate NULL pointers.

[Wolfgang Denk]

Dear Peter,

In message <1254577553.23101.14.camel at ptyser-laptop> you wrote:
>
> > So how should we proceed? My plan was to merge the "reloc" branch by
> > the end of next week. Is this still realistic?
> 
> That plan is realistic.  The executive summary is:
> The current "reloc" branch works, and is an improvement on U-Boot's
> previous "semi-reolcation", so we should merge it for this release.

OK, so let's do that.

> There are a few change which would further improve relocation that Jocke
> and I want to get merged.  Whether these improvements occur in this
> release or the next is not a big deal to me.
> 1. Fix relocation of NULL pointers.
> eg the following code would print the relocation fixup offset instead of
> the expected NULL.
>    void weak_fun(void) __attribute__((weak));
>    printf("weak_fun:%p\n", weak_fun);
> This was already an issue, so we're not breaking anything in the "reloc"
> branch
> 
> 2. Move relocation fixup code to C-code
> 
> 3. Possibly get true relocation working so that U-Boot could be located
> anywhere and still execute.

I tend to get all of this (as far as it's available and considered to
be ready) into this release, so we have it all in one big block.

> Jocke has 1 and 2 mostly figured out, I just need some time to play with
> the linker scripts a bit more to ensure nothing breaks.  (Sorry Jocke,
> its been a busy week).

You don't have to apologize. We've all been in similar situations
before. And we really appreciate your efforts (and Jocke's, of
course).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Q:  Why do mountain climbers rope themselves together?
A:  To prevent the sensible ones from going home.