[U-Boot] [PATCH] powerpc: do not fixup NULL ptrs

Joakim Tjernlund
2010-10-24T20:48:14Z

"Schwarz,Andre" <andre.schwarz@matrix-vision.de> wrote on 2010/10/24 22:33:44:
>
> Wolfgang,
>
> Wolfgang Denk <wd@denx.de> hat am 24. Oktober 2010 um 22:18 geschrieben:
>
> > Dear "Schwarz,Andre",
> >
> > In message <1702240919.111.1287950899754.JavaMail.open-xchange@proteus> you wrote:
> > >
> > > Since I assume Wolfgang won't accept a board specific "add 4 nops after
> > > _start"-patch I'd like to solve this without trial and error.
> >
> > To the best of my knowledge no board made use f this "warmboot" entry
> > point. OK, I wasn't aware that Jocke obviously did something lioke
> > this, so there might be other boards as well - but it's unlikely.
> >
> > > Of course digging into this is beyond my knowledge and any help would be
> > > appreciated.
> > > I simply can't think of being the only one having this issue...
> >
> > Check how your board implements the "reset" command - I cannot see any
> > specific code for that in your sources, so it looks mostly harmless to
> > me.
>
> this is what I don't understand :
>
> The broken boards are *dead*, i.e. no single character on the serial line.
> To me this means even the initial code running from flash isn't executed properly.
> IMHO this doesn't look like something like a problem after relocation ... it's something more fundamental.

Yes, like something is wrong with the reset code.

>
> The boards in question are using low boot, i.e. reset vector @ 0x100.

Me too.

> There's normal power-on-reset sequencing and no such thing like a reboot or "warm" reset.
>
> Without that patch the boards are running fine.
>
> I can see no obvious difference between working and broken boards.
>
> I'd be happy to hear that the error is caused by something hidden within board specific code.

You still haven't reported weather the 4 nop's helped or not, yet you
seek my help. I am just going to ignore you until you do test it.

Jocke