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

[Wolfgang Denk]

Dear Joakim Tjernlund,

In message <1254960231-11441-1-git-send-email-Joakim.Tjernlund at transmode.se> you wrote:
> NULL is an absolute value and should not be relocated.
> After this correction code like:
>  void weak_fun(void) __attribute__((weak));
>  printf("weak_fun:%p\n", weak_fun);
> will still print null after relocation.
> 
> Signed-off-by: Joakim Tjernlund <Joakim.Tjernlund at transmode.se>
> ---
> 
>  v2 - add ppc4xx
>  v3 - add 74xx_7xx

Could you next time _please_ add this "v3" information to the
Subject:, say as "[PATCH v3] relocation: ..." ?

Thanks!


>  cpu/74xx_7xx/start.S |    6 ++++--
>  cpu/mpc512x/start.S  |    6 ++++--
>  cpu/mpc5xx/start.S   |    6 ++++--
>  cpu/mpc5xxx/start.S  |    6 ++++--
>  cpu/mpc8220/start.S  |    6 ++++--
>  cpu/mpc824x/start.S  |    6 ++++--
>  cpu/mpc8260/start.S  |    6 ++++--
>  cpu/mpc83xx/start.S  |    6 ++++--
>  cpu/mpc85xx/start.S  |    6 ++++--
>  cpu/mpc86xx/start.S  |    6 ++++--
>  cpu/mpc8xx/start.S   |    6 ++++--
>  cpu/ppc4xx/start.S   |    6 ++++--
>  12 files changed, 48 insertions(+), 24 deletions(-)

I really wonder why we have that many copis of that code :-(


Applied to reloc branch.


Jocke, Peter: am I understanding correctly that we now have everything
in the "reloc" branch that we want to include with this upcoming
release, i. e. that we can do the "final" testing now before I merge
that branch into master?

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
"I go on working for the same reason a hen goes on laying eggs."
- H. L. Mencken