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