[PATCH 1/2] stm32mp1: spl: Fix compilation warnings

Patrick DELAUNAY patrick.delaunay at foss.st.com
Fri Jun 14 09:59:03 CEST 2024


Hi Patrice,

On 6/11/24 11:52, Patrice Chotard wrote:
> Fix the following compilation warnings :
>
> ../arch/arm/mach-stm32mp/stm32mp1/spl.c: In function 'stm32_init_tzc_for_optee':
> ../arch/arm/mach-stm32mp/stm32mp1/spl.c:148:37: warning: 'optee_size' may be used uninitialized [-Wmaybe-uninitialized]
>    148 |         tee_shmem_base = optee_base + optee_size - CFG_SHMEM_SIZE;
>        |                          ~~~~~~~~~~~^~~~~~~~~~~~
> ../arch/arm/mach-stm32mp/stm32mp1/spl.c:137:30: note: 'optee_size' was declared here
>    137 |         uint32_t optee_base, optee_size, tee_shmem_base;
>        |                              ^~~~~~~~~~
> ../arch/arm/mach-stm32mp/stm32mp1/spl.c:148:37: warning: 'optee_base' may be used
> uninitialized [-Wmaybe-uninitialized]
>    148 |         tee_shmem_base = optee_base + optee_size - CFG_SHMEM_SIZE;
>        |                          ~~~~~~~~~~~^~~~~~~~~~~~
> ../arch/arm/mach-stm32mp/stm32mp1/spl.c:137:18: note: 'optee_base' was declared here
>    137 |         uint32_t optee_base, optee_size, tee_shmem_base;
>        |                  ^~~~~~~~~~
>
> Fix also the following checkpatch "check" :
>
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> 37: FILE: arch/arm/mach-stm32mp/stm32mp1/spl.c:137:
> +	uint32_t optee_base = 0, optee_size = 0, tee_shmem_base;
>
> Signed-off-by: Patrice Chotard <patrice.chotard at foss.st.com>
> ---
>
>   arch/arm/mach-stm32mp/stm32mp1/spl.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-stm32mp/stm32mp1/spl.c b/arch/arm/mach-stm32mp/stm32mp1/spl.c
> index 6c79259b2c8..10abbed87f0 100644
> --- a/arch/arm/mach-stm32mp/stm32mp1/spl.c
> +++ b/arch/arm/mach-stm32mp/stm32mp1/spl.c
> @@ -134,7 +134,7 @@ void stm32_init_tzc_for_optee(void)
>   {
>   	const uint32_t dram_size = stm32mp_get_dram_size();
>   	const uintptr_t dram_top = STM32_DDR_BASE + (dram_size - 1);
> -	uint32_t optee_base, optee_size, tee_shmem_base;
> +	u32 optee_base = 0, optee_size = 0, tee_shmem_base;
>   	const uintptr_t tzc = STM32_TZC_BASE;
>   	int ret;
>   

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

Thanks
Patrick




More information about the U-Boot mailing list