[U-Boot] SPL broken on i.mx31 platforms

Tom Rini
2014-09-03T01:13:48Z

On Wed, Sep 03, 2014 at 02:53:17AM +0200, Beno?t Th?baudeau wrote:
> Hi Helmut, all,
>
> On Tue, Aug 19, 2014 at 10:55 PM, Beno?t Th?baudeau
> <benoit.thebaudeau.dev@gmail.com> wrote:
> > On Fri, Aug 15, 2014 at 7:45 PM, Magnus Lilja <lilja.magnus@gmail.com> wrote:
> >> On 13 August 2014 14:01, Helmut Raiger <helmut.raiger@hale.at> wrote:
> >>> On 08/05/2014 02:32 PM, Magnus Lilja wrote:
> >>>> I would expect Helmut to create a formal patch then I can test that
> >>>> and add a Tested-by.
> >>>>
> >>> The problem is it does not work with only the 'b reset' change on my
> >>> platform.
> >>> Should I provide a patch with the nops and the question marks around them?
> >>> It still could be a toolchain difference, mine is pretty old:
> >>>
> >>> $ arm-angstrom-linux-gnueabi-gcc --version
> >>> arm-angstrom-linux-gnueabi-gcc (GCC) 4.7.2
> >>> Copyright (C) 2012 Free Software Foundation, Inc.
> >>>
> >>> When I objdump the elf file I can see the very same code in cpu_init_crit()
> >>> as in start.S,
> >>> whatever that might mean (objdump is from the same toolchain).
> >>
> >> I use an even older gcc so I don't think that's the problem. I use:
> >> arm-none-linux-gnueabi-gcc (Sourcery CodeBench Lite 2011.09-70) 4.6.1
> >>
> >> Not sure how you should proceed with the path.
> >
> > IMHO, the 'b reset' and the 'nop nop nop' are two different issues, so
> > Helmut should create a formal patch for the 'b reset' issue right now,
> > which will fix mx31pdk (and maybe other boards) for the release. Then,
> > once the 'nop nop nop' issue has been resolved for TT-01 (cache issue
> > or something else), another formal patch should be created for this
> > issue, unless it is purely out of tree.
>
> v2014.10 is getting closer with the release of -rc2. It would be much
> better to get mx31pdk fixed for this release. Helmut, can you send a
> patch for the 'b reset' issue? If not, do you agree that someone else
> (maybe the board maintainer: Magnus?) sends it with a 'Reported-by:
> Helmut Raiger <helmut.raiger@hale.at>'?

Yes, please, I'd like to see this fixed for the release proper.

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140902/eb98954b/attachment.pgp>