Re: [XEN/ARM PATCH v6 1/1] Add OdroidXU board (Exynos 5410)

[Ian Campbell <Ian.Campbell@citrix.com>]

On Thu, 2014-09-11 at 11:36 -0700, Julien Grall wrote:
> Hi Ian,
> 
> On 11/09/14 05:58, Ian Campbell wrote:
> > I think what you suggest sounds plausible. For me the following fixes
> > boot on my arndale (I mistakenly thought that it worked earlier but I
> > was testing the wrong thing). If the presence of samsung,secure-firmware
> > is the way to distinguish the two cases then lets go that way.
> >
> > Since arndale is used in automated tests I'm inclined towards applying
> > this patch until a solution is found to detect the Odroid case.
> > -----8<--------------
> >
> >  From 24f74cbb981689b37d6bf83978c628a9fbabe246 Mon Sep 17 00:00:00 2001
> > From: Ian Campbell <ian.campbell@citrix.com>
> > Date: Thu, 11 Sep 2014 13:55:08 +0100
> > Subject: [PATCH] xen: arm: fix boot on arndale.
> >
> > The differences between Arndale and the Odoid-XU are more interesting
> > than first though, which results in 0bf8ddecb4df "xen/arm: Add
> > support for the Odroid-XU board." breaking boot on arndale.
> >
> > Revert back to arndale compatible behaviour while we sort this out.
> > Specifically we must (counterintuitively) use the regular (!ns)
> > sysram and the correct offset is 0x0 and 0x1c.
> >
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> Reviewed-by: Julien Grall <julien.grall@linaro.org>
> 
> Also I will give a look to the fix up patch ("Unbreak Arndale XEN Boot") 
> today.

Thanks. In the meantime I have applied the fix to make arndale work
again.

Ian.