[PATCH 01/20] Convert CONFIG_SPL_STACK to Kconfig

[Kever Yang]

Hi Tom,

On 2024/1/24 01:58, Tom Rini wrote:
> On Tue, Jan 23, 2024 at 11:53:18AM +0800, Kever Yang wrote:
>
>> Hi Tom,
>>
>> rk3036 only has small size internal sram, so it can only use very
>> simple SPL which only including dram init, and no SPL_FRAMEWORK
>> support.
>> In this convert, the CONFIG_SPL_STACK is lost for this SoC, any
>> suggestions on this kind of issue?
> Does this work?
>
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index e7b84fc1fa68..9deee0fdbea8 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -372,7 +372,8 @@ config SPL_SHARES_INIT_SP_ADDR
>   
>   config SPL_STACK
>   	hex "Initial stack pointer location"
> -	depends on (ARM || ARCH_JZ47XX || MICROBLAZE || RISCV) && SPL_FRAMEWORK
> +	depends on (ARM || ARCH_JZ47XX || MICROBLAZE || RISCV) && \
> +		SPL_FRAMEWORK || ROCKCHIP_RK3036
This works with adding CONFIG_SPL_STACK in rk3036 board defconfig.
>   	depends on !SPL_SHARES_INIT_SP_ADDR
>   	default 0x946bb8 if ARCH_MX7
>   	default 0x93ffb8 if ARCH_MX6 && MX6_OCRAM_256KB
>
> You will have to update the ROCKCHIP_RK3036 platforms otherwise the
> build will hang now.

What do you mean to update the platform?

The rk3036 case should be one of the available option for those 
"small"/low end SoCs has very limit

size of internal dram, only need minimal SPL+ U-Boot,  and no 
requirement for ATF/OPTEEwhich will have to use TPL+SPL+U-Boot.


The CONFIG_SPL_STACK is used in assemble init code, it doesn't need to 
depends on SPL_FRAMEWORK, isn't it?

Thanks,

- Kever

>