From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <Ian.Campbell@citrix.com>
Subject: Re: [XEN/ARM PATCH v6 1/1] Add OdroidXU board (Exynos
	5410)
Date: Fri, 12 Sep 2014 11:19:55 +0100
Message-ID: <1410517195.27902.0.camel@kazak.uk.xensource.com>
References: <1409871443-15641-1-git-send-email-suriyan.r@gmail.com>
	<1410171636.10156.148.camel@kazak.uk.xensource.com>
	<CANoR_OCtwM0=gdriVCa7V=O34Rzta5mOhvZ=BbJNdUBv9a6_XQ@mail.gmail.com>
	<5410CF05.9070403@linaro.org>
	<CANoR_OCXow=7aUXR+0ZWd+2HZ4UP2-g7P8vK6eZS_C1-MiWjzg@mail.gmail.com>
	<5410F4D5.9000503@linaro.org>
	<CANoR_ODq-UK1t8pVj8zHGasqNHnZFOtr4vNySEGb6Jqj1dGGPg@mail.gmail.com>
	<1410440292.25388.19.camel@kazak.uk.xensource.com>
	<5411EBB4.4000700@linaro.org>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <5411EBB4.4000700@linaro.org>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Julien Grall <julien.grall@linaro.org>
Cc: keir@xen.org, Suriyan Ramasami <suriyan.r@gmail.com>, Tim Deegan <tim@xen.org>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>, Jan Beulich <jbeulich@suse.com>, ian.jackson@citrix.com, Julien Grall <julien.grall@linaro.com>
List-Id: xen-devel@lists.xenproject.org


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.