From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Thu, 27 Dec 2012 10:41:06 +0100
Subject: [U-Boot] [PATCH] arm/bootstage: Move the bootstage call after
	gd init
In-Reply-To: <CAPnjgZ2vjRLP9mqnmpkB1uXaiA3i0t4GrEm10ZfdEC0hfyVbxg@mail.gmail.com>
References: <50D6EA9C.30809@gmail.com>
	<CAPnjgZ2vjRLP9mqnmpkB1uXaiA3i0t4GrEm10ZfdEC0hfyVbxg@mail.gmail.com>
Message-ID: <20121227104106.392a328d@lilith>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Simon,

On Wed, 26 Dec 2012 12:39:32 -0800, Simon Glass <sjg@chromium.org>
wrote:
> Hi Vikram,
> 
> On Sun, Dec 23, 2012 at 3:27 AM, Vikram Narayanan <vikram186@gmail.com> 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 <vikram186@gmail.com>
> > Cc: Simon Glass <sjg@chromium.org>
> > Cc: Wolfgang Denk <wd@denx.de>
> > Cc: Albert Aribaud <albert.u.boot@aribaud.net>
> 
> Acked-by: Simon Glass <sjg@chromium.org>
> 
> 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.

Amicalement,
-- 
Albert.