[U-Boot] ARM: interrupt_init before relocation, write fails

[Rob Herring]

On Thu, Oct 24, 2013 at 4:37 PM, Andrew Ruder <andy at aeruder.net> wrote:
> On Wed, Oct 23, 2013 at 11:41:45AM -0600, Joe Kulikauskas wrote:
>> If I revert that patch, I don't see that problem.
>
> FWIW, I am working on a PXA270 target, and have had to revert this patch
> as well.  I hadn't gotten around to tracking down where and why I was
> crashing though so hadn't emailed in a bug report yet.  Now seeing as
> there's someone else now seeing it too I thought I would chime in with
> a "me too".
>
> Rob: CC'd you since you were the author and might have some insight.
> Full email in entirety below.

Other than doing stack setup in a completely different way by setting
the mode in the CPSR and setting up SP_irq and SP_abt directly, I
don't see an easy fix. So we should revert this change. It works for
me because I run from RAM before relocation.

Albert or Tom, can you please revert commit 0f5141e9c57e96de11642a.

Rob

>
> - Andy
>
> On Wed, Oct 23, 2013 at 11:41:45AM -0600, Joe Kulikauskas wrote:
>> v2013.10-rc1 (and after) has a patch (
>> http://lists.denx.de/pipermail/u-boot/2013-June/156298.html) which sets up
>> the abort stack before relocation. However, what I am seeing:
>> IRQ_STACK_START_IN
>> is in flash at that time, so this write fails.
>>
>> interrupt_init:
>> FF0A0FA8 e59f3010   LDR       R3,ff0a0fc0 (ff0a0050=IRQ_STACK_START_IN)
>>
>> Before this patch, abort stack setup was done after relocation, so target
>> location is in RAM and writeable.
>>
>> interrupt_init:
>> 9FFB4020 e59f3010   LDR       R3,9ffb4038 (9ffb3054=IRQ_STACK_START_IN)
>>
>> If I revert that patch, I don't see that problem.
>>
>> Joe Kulikauskas
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot