Re: [Xen-devel] [PATCH 7/8] arm, device tree: parse the DTB for RAM location and size

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2012-02-13 at 15:46 +0000, David Vrabel wrote:
> On 13/02/12 14:51, Ian Campbell wrote:
> > On Mon, 2012-02-13 at 13:18 +0000, David Vrabel wrote:
> >> From: David Vrabel <david.vrabel@xxxxxxxxxx>
> >>
> >> 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 <david.vrabel@xxxxxxxxxx>
> >> Acked-by: Tim Deegan <tim@xxxxxxx>
> >> ---
> > [...]
> >>  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/.

I'd be happy to do that if there is agreement among the other
maintainers.

Of course as soon as some joker ports x86 to use DT we are back where we
started ;-)

> Or, if you prefer, I can make sure to split the patches in common and
> arch bits.
> 
> David



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel