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

Benoît Thébaudeau
2014-07-04T21:42:03Z

Hi Magnus,

On Fri, Jul 4, 2014 at 12:42 PM, Magnus Lilja <lilja.magnus@gmail.com> wrote:
> Hi all,
>
> (I hope gmail formats this mail according to the mailing list requirements,
> my apologies if it doesn't)

I think that HTML is forbidden. You can switch Gmail to plain text
mode with the "More options" down arrow in the lower right corner of
the composing frame.

> On 4 July 2014 10:50, Albert ARIBAUD <albert.u.boot@aribaud.net> wrote:
>>
>> Hi Beno?t,
>>
>> On Thu, 3 Jul 2014 22:58:56 +0200, Beno?t Th?baudeau
>> <benoit.thebaudeau.dev@gmail.com> wrote:
>>
>> > Hi Albert,
>> > It makes sense to me. That should work, but it'd be better to avoid
>> > adding a custom linker script. A simple fix in vectors.S would be
>> > preferable if possible. Also, the __image_copy_start stuff should be
>> > taken care of with such a change.
>>
>> I do not intend to have this in a custom linker script; I want this to
>> be in the common ARM SPL linker script (I hope it is what Helmut's
>> TT-01 changes use) -- I also want to get rid of all custom linker
>> scripts in the ARM par of U-Boot, but I could not get this done for
>> 2014.07.
>>
>
> I have access to i.MX31 board (PDK) and can definitely help testing any
> patch to this problem.

Thanks, that will help. The first things to test will be:
- confirm the issue on mx31pdk with mainline,
- confirm that reverting commit 41623c91 on top of mainline is
sufficient to fix the issue,
- test from mainline whether removing the two "_reset" lines in
vectors.S and adding "b reset" or "bl reset" right after "_start:"
(still in vectors.S) fixes the issue or improves things in any way.

> However, I won't be able to do any verification until
> beginning of August.

No problem. Maybe Helmut will make some progress in the meantime.

Regards,
Beno?t