On 12/27/2012 3:52 PM, Albert ARIBAUD wrote:
On Thu, 27 Dec 2012 10:41:06 +0100, Albert ARIBAUD
<[email protected]> wrote:
Hi Simon,

On Wed, 26 Dec 2012 12:39:32 -0800, Simon Glass <[email protected]>
wrote:
Hi Vikram,

On Sun, Dec 23, 2012 at 3:27 AM, Vikram Narayanan <[email protected]> wrote:
bootstage_mark_name calls timer_get_boot_us which inturn calls
the arch timer. The arch timer uses the gd pointer to save the
lastinc count. A call to bootstage_mark_name here results in a
data abort as gd is uninitialized.

Signed-off-by: Vikram Narayanan <[email protected]>
Cc: Simon Glass <[email protected]>
Cc: Wolfgang Denk <[email protected]>
Cc: Albert Aribaud <[email protected]>

Acked-by: Simon Glass <[email protected]>

This bug will affect boards which don't have their own
timer_get_boot_us() function.

Albert, I think we should try to get this in as a bug fix in the
upcoming release.

Thanks for the fix Vikram.

Thanks Vikram and Simon; I'll pull this in today.

... actually, this will depend on whether my crt0 change gets in for
2013.01, as with this change, gd is already initialized when
entering board_init_f().

Yes. You are right. If your patch goes in, my patch need not be applied.

Regards,
Vikram
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to