[U-Boot-Users] [PATCH] Make MPC83xx one step closer to full relocation.

Joakim Tjernlund
2008-04-08T09:31:24Z

On Tue, 2008-04-08 at 10:58 +0200, Stefan Roese wrote:
> Hi Joakim,
>
> On Saturday 29 March 2008, Joakim Tjernlund wrote:
> > > Joakim Tjernlund <Joakim.Tjernlund@transmode.se> wrote:
> > > > Remove a few absolute references to CFG_MONITOR_BASE for ppc/mpc83xx
> > > > and use GOT relative reference.
> > > > ---
> > > > cpu/mpc83xx/start.S | 11 +++++++----
> > > > lib_ppc/board.c | 3 ++-
> > > > 2 files changed, 9 insertions(+), 5 deletions(-)
>
> <snip>
>
> > > btw, shouldn't it fix up the other ppc start.S files?
> >
> > The other ppc's won't break and I only got a 83xx, hence I only did
> > it for 83xx, hoping it would serve as a guide for the rest.
>
> I'm afraid, but the "other ppc's" did break with this patch. At least 4xx
> does. With your patch applied relocation to SDRAM does not work anymore.
> Here's what I get:
>
> CFG_MONITOR_BASE=fffa0000
> (ulong)&_start + EXC_OFF_SYS_RESET=fffa2200
> EXC_OFF_SYS_RESET=100
>
> I haven't looked into it closer yet. Any idea on how to fix this?
>
> Thanks.
>
> Best regards,
> Stefan

Oops, didn't see that coming. Your _start symbol in ppc4xx/start.S isn't
pointing to your real start of execution. Seems like _start_440 is your
real start but I can't be sure. There are some strange code in there
that I don't understand.

Jocke