From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <Ian.Campbell@citrix.com>
Subject: Re: [PATCH 10/10] xen: arm: update multiboot device
	tree bindings.
Date: Wed, 18 Jun 2014 16:17:01 +0100
Message-ID: <1403104621.6568.81.camel@kazak.uk.xensource.com>
References: <1402919079.14907.22.camel@kazak.uk.xensource.com>
	<1402919103-29642-10-git-send-email-ian.campbell@citrix.com>
	<alpine.DEB.2.02.1406181545410.13771@kaball.uk.xensource.com>
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: <alpine.DEB.2.02.1406181545410.13771@kaball.uk.xensource.com>
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: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Naresh Bhat <naresh.bhat@linaro.org>, julien.grall@linaro.org, tim@xen.org, xen-devel@lists.xen.org, Roy Franz <roy.franz@linaro.org>, Fu Wei <fu.wei@linaro.org>
List-Id: xen-devel@lists.xenproject.org

On Wed, 2014-06-18 at 15:56 +0100, Stefano Stabellini wrote:
> On Mon, 16 Jun 2014, Ian Campbell wrote:
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> > ---
> >  docs/misc/arm/device-tree/booting.txt |   24 ++++++++++++++++++------
> >  1 file changed, 18 insertions(+), 6 deletions(-)
> > 
> > diff --git a/docs/misc/arm/device-tree/booting.txt b/docs/misc/arm/device-tree/booting.txt
> > index bfb8d01..92af119 100644
> > --- a/docs/misc/arm/device-tree/booting.txt
> > +++ b/docs/misc/arm/device-tree/booting.txt
> > @@ -8,15 +8,27 @@ Each node contains the following properties:
> >  
> >  - compatible
> >  
> > -	Must be:
> > +	Must always include at least:
> >  
> > -		"xen,<type>", "xen,multiboot-module"
> > +		"multiboot,module"
> 
> As it stands this conflicts with the statement below that we are
> compatible with Xen 4.4: the old multiboot compatible strings didn't
> include "multiboot,module".
> You might want to reword it.

I'm not sure what you mean or I would. 

I don't think the doc says that we are compatible with Xen 4.4, it says
that in order to be compatible with Xen 4.4 the user should... Is that
what I need to clarify perhaps?

> > -	where <type> must be one of:
> > +	Optionally a more specific compatible string may be used in
> > +	addition to the above. One of:
> 
> We should probably recommend the usage of a more specific string in
> addition to "multiboot,module".

"Normally a more specific compatible string should be used in addition
to the above" perhaps?

At some point I'd like to see us doing more probing of magic numbers
(e.g. x86 does this for the XSM policy).

Ian.