[PATCH v2] arm/docs: Clarify legacy DT bindings on UEFI

Luca Fancellu posted 1 patch 2 years, 5 months ago
Test gitlab-ci passed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20211013145202.53070-1-luca.fancellu@arm.com
docs/misc/arm/device-tree/booting.txt | 2 ++
1 file changed, 2 insertions(+)
[PATCH v2] arm/docs: Clarify legacy DT bindings on UEFI
Posted by Luca Fancellu 2 years, 5 months ago
Since the introduction of UEFI boot for Xen, the legacy
compatible strings were not supported and the stub code
was checking only the presence of “multiboot,module” to
require the Xen UEFI configuration file or not.
The documentation was not updated to specify that behavior.

Add a phrase to docs/misc/arm/device-tree/booting.txt
to clarify it.

Signed-off-by: Luca Fancellu <luca.fancellu@arm.com>
Acked-by: Julien Grall <jgrall@amazon.com>
---
Changes in v2:
- dropped redundant wording in docs/misc/arm/device-tree/booting.txt
- Add more details to the commit message
---
 docs/misc/arm/device-tree/booting.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/docs/misc/arm/device-tree/booting.txt b/docs/misc/arm/device-tree/booting.txt
index c6a775f4e8..71895663a4 100644
--- a/docs/misc/arm/device-tree/booting.txt
+++ b/docs/misc/arm/device-tree/booting.txt
@@ -51,6 +51,8 @@ Each node contains the following properties:
 	Xen 4.4 supported a different set of legacy compatible strings
 	which remain supported such that systems supporting both 4.4
 	and later can use a single DTB.
+	However when booting Xen using UEFI, the legacy compatible
+	strings are not supported.
 
 	- "xen,multiboot-module" equivalent to "multiboot,module"
 	- "xen,linux-zimage"     equivalent to "multiboot,kernel"
-- 
2.17.1


Re: [PATCH v2] arm/docs: Clarify legacy DT bindings on UEFI
Posted by Stefano Stabellini 2 years, 5 months ago
On Wed, 13 Oct 2021, Luca Fancellu wrote:
> Since the introduction of UEFI boot for Xen, the legacy
> compatible strings were not supported and the stub code
> was checking only the presence of “multiboot,module” to
> require the Xen UEFI configuration file or not.
> The documentation was not updated to specify that behavior.
> 
> Add a phrase to docs/misc/arm/device-tree/booting.txt
> to clarify it.
> 
> Signed-off-by: Luca Fancellu <luca.fancellu@arm.com>
> Acked-by: Julien Grall <jgrall@amazon.com>

Committed


> ---
> Changes in v2:
> - dropped redundant wording in docs/misc/arm/device-tree/booting.txt
> - Add more details to the commit message
> ---
>  docs/misc/arm/device-tree/booting.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/docs/misc/arm/device-tree/booting.txt b/docs/misc/arm/device-tree/booting.txt
> index c6a775f4e8..71895663a4 100644
> --- a/docs/misc/arm/device-tree/booting.txt
> +++ b/docs/misc/arm/device-tree/booting.txt
> @@ -51,6 +51,8 @@ Each node contains the following properties:
>  	Xen 4.4 supported a different set of legacy compatible strings
>  	which remain supported such that systems supporting both 4.4
>  	and later can use a single DTB.
> +	However when booting Xen using UEFI, the legacy compatible
> +	strings are not supported.
>  
>  	- "xen,multiboot-module" equivalent to "multiboot,module"
>  	- "xen,linux-zimage"     equivalent to "multiboot,kernel"
> -- 
> 2.17.1
>