[PATCH] test/py: test_efi_fit: Update #size-cells to 1

[Heinrich Schuchardt]

On 6/27/20 2:12 PM, Bin Meng wrote:
> From: Bin Meng <bin.meng at windriver.com>
>
> test_efi_fit tests fail on RISC-V currently. This is due to the
> RISC-V arch_fixup_fdt() checks the #size-cells of the root node
> in order to correctly fix up the reserved memory node.
>
> Update #size-cells to 1 so tests can pass.
>
> Signed-off-by: Bin Meng <bin.meng at windriver.com>
> ---
>
>  test/py/tests/test_efi_fit.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/py/tests/test_efi_fit.py b/test/py/tests/test_efi_fit.py
> index beaf4a3..3616e99 100644
> --- a/test/py/tests/test_efi_fit.py
> +++ b/test/py/tests/test_efi_fit.py
> @@ -106,7 +106,7 @@ FDT_DATA = '''
>
>  / {
>      #address-cells = <1>;
> -    #size-cells = <0>;
> +    #size-cells = <1>;

As you describe above the test failed for RISC-V because the #size-cells
property is incorrect for the /reserved-memory node.

Changing the #size-cells property here does not solve the underlying
problem that the code inserting a /reserved-memory node has a bug. That
code should insert properties #address-cells and #size-cells as needed
into the /reserved-memory node itself.

Have you been able to figure out where the /reserved-memory node is created?

In the resulting device tree check that #size-cells remains <0> for the
/reset at 0 node.

Best regards

Heinrich

>
>      model = "%(sys-arch)s %(fdt_type)s EFI FIT Boot Test";
>      compatible = "%(sys-arch)s";
>