Re: [PATCH] arm64: make sure fdt has #address-cells and #size-cells properties

[Leif Lindholm]

On Thu, Oct 31, 2019 at 11:37:40AM +0100, Javier Martinez Canillas wrote:
> From: Mark Salter <address@hidden>
> 
> Recent upstream changes to kexec-tools relies on #address-cells
> and #size-cells properties in the FDT. If grub2 needs to create
> a chosen node, it is likely because firmware did not provide one.
> In that case, set #address-cells and #size-cells properties to
> make sure they exist.

I assumed we fixed that in 347210a5d5ce
("efi/fdt: Set address/size cells to 2 for empty tree")?

If we didn't, I still think the fix should be in
grub-core/loader/efi/fdt.c rather than here.

/
    Leif

> Signed-off-by: Mark Salter <address@hidden>
> Signed-off-by: Javier Martinez Canillas <address@hidden>
> ---
> 
>  grub-core/loader/arm64/linux.c | 16 +++++++++++++++-
>  1 file changed, 15 insertions(+), 1 deletion(-)
> 
> diff --git grub-core/loader/arm64/linux.c grub-core/loader/arm64/linux.c
> index ef3e9f9444c..24d73732d89 100644
> --- grub-core/loader/arm64/linux.c
> +++ grub-core/loader/arm64/linux.c
> @@ -78,7 +78,21 @@ finalize_params_linux (void)
>  
>    node = grub_fdt_find_subnode (fdt, 0, "chosen");
>    if (node < 0)
> -    node = grub_fdt_add_subnode (fdt, 0, "chosen");
> +    {
> +      /*
> +       * If we have to create a chosen node, Make sure we
> +       * have #address-cells and #size-cells properties.
> +       */
> +      retval = grub_fdt_set_prop32(fdt, 0, "#address-cells", 2);
> +      if (retval)
> +        goto failure;
> +
> +      retval = grub_fdt_set_prop32(fdt, 0, "#size-cells", 2);
> +      if (retval)
> +        goto failure;
> +
> +      node = grub_fdt_add_subnode (fdt, 0, "chosen");
> +    }
>  
>    if (node < 1)
>      goto failure;
> -- 
> 2.21.0
> 
> 
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/grub-devel