[PATCH v1 3/9] stm32mp: dram_init: Fix AARCH64 compilation warnings

Patrick DELAUNAY patrick.delaunay at foss.st.com
Thu Nov 9 10:13:08 CET 2023


Hi,


On 10/27/23 16:42, Patrice Chotard wrote:
> From: Patrick Delaunay <patrick.delaunay at foss.st.com>
>
> When building with AARCH64 defconfig, we got warnings for debug
> message
> - format '%x' expects argument of type 'unsigned int',
>     but argument 3 has type 'size_t' {aka 'long unsigned int'}).
> - format '%lx' expects argument of type 'long unsigned int',
>    but argument 2 has type 'phys_addr_t' {aka 'long long unsigned
>    int'}
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at foss.st.com>
> Signed-off-by: Patrice Chotard <patrice.chotard at foss.st.com>
> ---
>
>   arch/arm/mach-stm32mp/dram_init.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-stm32mp/dram_init.c b/arch/arm/mach-stm32mp/dram_init.c
> index a1e77a42e4f..cb35ed60ca1 100644
> --- a/arch/arm/mach-stm32mp/dram_init.c
> +++ b/arch/arm/mach-stm32mp/dram_init.c
> @@ -36,7 +36,7 @@ int dram_init(void)
>   		log_debug("Cannot get RAM size: %d\n", ret);
>   		return ret;
>   	}
> -	log_debug("RAM init base=%lx, size=%x\n", ram.base, ram.size);
> +	log_debug("RAM init base=%p, size=%zx\n", (void *)ram.base, ram.size);
>   
>   	gd->ram_size = ram.size;
>   


Reviewed-by: Patrick Delaunay <patrick.delaunay at foss.st.com>

Thanks
Patrick



More information about the U-Boot mailing list