From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Subject: Re: [PATCH 7/8] arm, device tree: parse the DTB for RAM location and size Date: Mon, 13 Feb 2012 15:46:27 +0000 Message-ID: <4F393053.80903@citrix.com> References: <1329139131-27554-1-git-send-email-david.vrabel@citrix.com> <1329139131-27554-8-git-send-email-david.vrabel@citrix.com> <1329144704.31256.114.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1329144704.31256.114.camel@zakaz.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Campbell Cc: "xen-devel@lists.xensource.com" , "Keir (Xen.org)" List-Id: xen-devel@lists.xenproject.org On 13/02/12 14:51, Ian Campbell wrote: > On Mon, 2012-02-13 at 13:18 +0000, David Vrabel wrote: >> From: David Vrabel >> >> Prior to setting up the page tables, parse the DTB for the location >> and size of RAM. >> >> Use this information to get the physical address to relocate Xen to in >> setup_pagetables(). >> >> Signed-off-by: David Vrabel >> Acked-by: Tim Deegan >> --- > [...] >> xen/common/Makefile | 3 + >> xen/common/device_tree.c | 179 +++++++++++++++++++++++++++++++++++++++++ > [...] >> xen/include/xen/device_tree.h | 36 ++++++++ > > I'd have preferred these non-ARM bits in a separate patch if possible. > I'm going to commit them anyway though on the basis that they are in > reality ARM specific unless Keir (or someone else) objects quite soon. Dividing maintainer-ship by directory seems a little coarse grained. It may be more practical for you (Ian) to be the maintainer/committer for the device tree related stuff even though they live under xen/common/. Or, if you prefer, I can make sure to split the patches in common and arch bits. David