Re: [PATCH] configs: versal: Fix initial stack pointer

[Michal Simek <michal.simek@amd.com>]

On 4/7/25 15:45, Padmarao Begari wrote:
> The mini u-boot is getting exception because of an initial
> stack pointer address is used at near the top of memory,
> and while executing u-boot is assigned pre-malloc and
> global_data memory after initial stack pointer and updated
> the stack pointer. Serial driver is used pre-malloc area
> for serial operations before relocation. But pre-malloc area
> is cleared while doing BSS at relocation time. The u-boot is
> called board_init() function and doing printf, relocation serial
> driver is not initialized yet, so it is using before relocation
> serial operations but it is cleared by BSS and got the exception.
> To fix, change an initial stack pointer address from near the
> top of memory to near the relocation memory.
> 
> Fixes: 685874939a5e ("configs: versal: update initial stack pointer")
> Signed-off-by: Padmarao Begari <padmarao.begari@amd.com>
> ---
>   configs/xilinx_versal_mini_ospi_defconfig | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configs/xilinx_versal_mini_ospi_defconfig b/configs/xilinx_versal_mini_ospi_defconfig
> index e1cdc186fb4..af9ce499169 100644
> --- a/configs/xilinx_versal_mini_ospi_defconfig
> +++ b/configs/xilinx_versal_mini_ospi_defconfig
> @@ -7,7 +7,7 @@ CONFIG_SYS_MALLOC_LEN=0x2000
>   CONFIG_SYS_MALLOC_F_LEN=0x500
>   CONFIG_NR_DRAM_BANKS=1
>   CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
> -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xFFFFFE00
> +CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xFFFE1000
>   CONFIG_SF_DEFAULT_SPEED=30000000
>   CONFIG_ENV_SIZE=0x80
>   # CONFIG_DM_GPIO is not set

Applied.
M