[PATCH] stm32mp1: mmu_set_region_dcache_behaviour

[Patrice CHOTARD]

On 7/24/20 11:21 AM, Patrick Delaunay wrote:
> Since commit d877f8fd0f09 ("arm: provide a function for boards init
> code to modify MMU virtual-physical map") the parameter of
> mmu_set_region_dcache_behaviour need to be MMU_SECTION_SIZE
> aligned.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> ---
>
>  arch/arm/mach-stm32mp/cpu.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-stm32mp/cpu.c b/arch/arm/mach-stm32mp/cpu.c
> index 56092c8bf6..b7fcee2b36 100644
> --- a/arch/arm/mach-stm32mp/cpu.c
> +++ b/arch/arm/mach-stm32mp/cpu.c
> @@ -225,9 +225,10 @@ static void early_enable_caches(void)
>  	dcache_enable();
>  
>  	if (IS_ENABLED(CONFIG_SPL_BUILD))
> -		mmu_set_region_dcache_behaviour(STM32_SYSRAM_BASE,
> -						STM32_SYSRAM_SIZE,
> -						DCACHE_DEFAULT_OPTION);
> +		mmu_set_region_dcache_behaviour(
> +			ALIGN(STM32_SYSRAM_BASE, MMU_SECTION_SIZE),
> +			round_up(STM32_SYSRAM_SIZE, MMU_SECTION_SIZE),
> +			DCACHE_DEFAULT_OPTION);
>  	else
>  		mmu_set_region_dcache_behaviour(STM32_DDR_BASE, STM32_DDR_SIZE,
>  						DCACHE_DEFAULT_OPTION);

Reviewed-by: Patrice Chotard <patrice.chotard at st.com>

Thanks

Patrice