[PATCH 1/2] stm32mp: Fix board_get_usable_ram_top()

[Patrick DELAUNAY]

Hi,

On 1/9/25 11:21, Patrice Chotard wrote:
> mmu_set_region_dcache_behaviour() parameters must be aligned
> which is not always the case.
> For example for STM32MP2, we stayed stuck inside
> mmu_set_region_dcache_behaviour() in an infinite loop because
> set_one_region() always return 0 due to start parameter which is
> not aligned.
>
> 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 b06105768b3..34b958d7afd 100644
> --- a/arch/arm/mach-stm32mp/dram_init.c
> +++ b/arch/arm/mach-stm32mp/dram_init.c
> @@ -81,7 +81,7 @@ phys_addr_t board_get_usable_ram_top(phys_size_t total_size)
>   	/* add 8M for U-Boot reserved memory: display, fdt, gd,... */
>   	size = ALIGN(SZ_8M + CONFIG_SYS_MALLOC_LEN + total_size, MMU_SECTION_SIZE);
>   
> -	reg = gd->ram_top - size;
> +	reg = ALIGN(gd->ram_top - size, MMU_SECTION_SIZE);
>   
>   	/* Reserved memory for OP-TEE at END of DDR for STM32MP1 SoC */
>   	if (IS_ENABLED(CONFIG_STM32MP13X) || IS_ENABLED(CONFIG_STM32MP15X)) {



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

Thanks
Patrick