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

Benoît Thébaudeau
2014-09-03T00:53:17Z

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>'?

Best regards,
Beno?t