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

[Patrice CHOTARD]

On 1/30/25 11:51, Patrick DELAUNAY wrote:
> 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
> 
> 
Applied to u-boot-stm32/master

Thanks
Patrice