diff mbox

[Xen-devel] xen: arm: remove innaccurate statement about multiboot module path

Message ID 1391591544-16072-1-git-send-email-ian.campbell@citrix.com
State Accepted
Commit af82a77f3abc600c735e12e14874d64c80fa986f
Headers show

Commit Message

Ian Campbell Feb. 5, 2014, 9:12 a.m. UTC
It is the compatible string which matters, not the absolute path, and in any
case /chosen/module@N is more often used than /chosen/modules/module@N.

Reported-by: Fu Wei <fu.wei@linaro.org>
Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
 docs/misc/arm/device-tree/booting.txt |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Julien Grall Feb. 5, 2014, 10:32 a.m. UTC | #1
On 05/02/14 09:12, Ian Campbell wrote:
> It is the compatible string which matters, not the absolute path, and in any
> case /chosen/module@N is more often used than /chosen/modules/module@N.

I'm using /chosen/modules/module@N on my script :)

Is it for Xen 4.4?

>
> Reported-by: Fu Wei <fu.wei@linaro.org>
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Acked-by: Julien Grall <julien.gral@linaro.org>
Ian Campbell Feb. 5, 2014, 10:36 a.m. UTC | #2
On Wed, 2014-02-05 at 10:32 +0000, Julien Grall wrote:
> 
> On 05/02/14 09:12, Ian Campbell wrote:
> > It is the compatible string which matters, not the absolute path, and in any
> > case /chosen/module@N is more often used than /chosen/modules/module@N.
> 
> I'm using /chosen/modules/module@N on my script :)

I knew I'd regret making that statement ;-)

> 
> Is it for Xen 4.4?

I don't see any reason to hold off on docs improvements.

> 
> >
> > Reported-by: Fu Wei <fu.wei@linaro.org>
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> Acked-by: Julien Grall <julien.gral@linaro.org>

Thanks,
Ian.
Ian Campbell Feb. 6, 2014, 12:39 p.m. UTC | #3
On Wed, 2014-02-05 at 10:36 +0000, Ian Campbell wrote:
> On Wed, 2014-02-05 at 10:32 +0000, Julien Grall wrote:
> > 
> > On 05/02/14 09:12, Ian Campbell wrote:
> > > It is the compatible string which matters, not the absolute path, and in any
> > > case /chosen/module@N is more often used than /chosen/modules/module@N.
> > 
> > I'm using /chosen/modules/module@N on my script :)
> 
> I knew I'd regret making that statement ;-)
> 
> > 
> > Is it for Xen 4.4?
> 
> I don't see any reason to hold off on docs improvements.
> 
> > 
> > >
> > > Reported-by: Fu Wei <fu.wei@linaro.org>
> > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> > Acked-by: Julien Grall <julien.gral@linaro.org>
> 

Applied.
diff mbox

Patch

diff --git a/docs/misc/arm/device-tree/booting.txt b/docs/misc/arm/device-tree/booting.txt
index 8da1e0b..07fde27 100644
--- a/docs/misc/arm/device-tree/booting.txt
+++ b/docs/misc/arm/device-tree/booting.txt
@@ -4,8 +4,7 @@  Dom0 kernel and ramdisk modules
 Xen is passed the dom0 kernel and initrd via a reference in the /chosen
 node of the device tree.
 
-Each node has the form /chosen/modules/module@<N> and contains the following
-properties:
+Each node contains the following properties:
 
 - compatible