[U-Boot] [RESEND PATCH] core: ofnode: Add ofnode_get_addr_size_index

[Keerthy]

On 14/08/19 3:17 PM, Eugeniu Rosca wrote:
> Hi Keerthy,
> 
> On Wed, Aug 14, 2019 at 01:53:50PM +0530, Keerthy wrote:
>> Eugeniu,
>>
>> Could you kindly help me with the steps to reproduce the issue?
> 
> Sure. My pleasure. See [0].
> 
>> I am sending out a fix patch to convert all the size instances to fdt_size_t
>> type.
>>
>> I have posted: https://patchwork.ozlabs.org/patch/1146830/
>>
>> I believe the above should fix the size mismatch.
> 
> The patch appears to fix the problem. Thanks!


Thanks for testing it. If all your tests have passed please post your
Tested-by on that patch.

> 
> [0] Steps to reproduce the issue reported in
>      https://patchwork.ozlabs.org/patch/1090094/#2212555

Thanks for the instructions. I will make sure i test this
before posting any patch related to drivers/core/of

Best Regards,
Keerthy

> 
> * Apply patch [1] to u-boot/master
> * u-boot (master) gcc --version
> gcc (Ubuntu 7.4.0-1ubuntu1~16.04~ppa1) 7.4.0
> * u-boot (master) make defconfig all -j12
> * u-boot (master) ./u-boot -d arch/sandbox/dts/test.dtb
> 
> [1] Dirty patch to enable ASAN in sandbox U-Boot (v2019.10-rc2):
> 
> diff --git a/Makefile b/Makefile
> index 3b0864ae8eaa..b8ef4f267ed5 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -391,7 +391,7 @@ KBUILD_CPPFLAGS := -D__KERNEL__ -D__UBOOT__
>   
>   KBUILD_CFLAGS   := -Wall -Wstrict-prototypes \
>   		   -Wno-format-security \
> -		   -fno-builtin -ffreestanding $(CSTD_FLAG)
> +		   -fno-builtin -ffreestanding $(CSTD_FLAG) -fsanitize=address
>   KBUILD_CFLAGS	+= -fshort-wchar -fno-strict-aliasing
>   KBUILD_AFLAGS   := -D__ASSEMBLY__
>   
> @@ -768,6 +768,7 @@ PLATFORM_LIBGCC = arch/$(ARCH)/lib/lib.a
>   else
>   PLATFORM_LIBGCC := -L $(shell dirname `$(CC) $(c_flags) -print-libgcc-file-name`) -lgcc
>   endif
> +PLATFORM_LIBS := -lasan $(PLATFORM_LIBS)
>   PLATFORM_LIBS += $(PLATFORM_LIBGCC)
>   
>   ifdef CONFIG_CC_COVERAGE
>