Re: [Xen-devel] [PATCH 10/10] xen: arm: update multiboot device tree bindings.

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Mon, 16 Jun 2014, Ian Campbell wrote:
> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> ---
>  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.


> -     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".


> -     - "linux-zimage" -- the dom0 kernel
> -     - "linux-initrd" -- the dom0 ramdisk
> -     - "xsm-policy"   -- XSM policy blob
> +     - "multiboot,kernel"    -- the dom0 kernel
> +     - "multiboot,ramdisk"   -- the dom0 ramdisk
> +     - "xen,xsm-policy"      -- XSM policy blob
> +
> +     For compatibility with Xen 4.4 the following are also
> +     supported:
> +
> +             "xen,multiboot-module" equivalent to "multiboot,module"
> +             "xen,linux-zimage" equivalent to "multiboot,kernel"
> +             "xen,linux-initrd" equivalent to "multiboot,ramdisk"
> +
> +     Xen 4.4 does not know the "multiboot,*" compatibility
> +     strings. Also when using Xen 4.4 the more specific
> +     "xen,linux-*" names are non-optional.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel