Re: [PATCH 01/20] Convert CONFIG_SPL_STACK to Kconfig

[Tom Rini]

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
        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.

-- 
Tom

signature.asc
Description: PGP signature