Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36275 )
Change subject: arch/arm64: Pass cbmem_top to ramstage via calling argument ......................................................................
Patch Set 20:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36275/20/src/arch/arm64/Kconfig File src/arch/arm64/Kconfig:
https://review.coreboot.org/c/coreboot/+/36275/20/src/arch/arm64/Kconfig@20 PS20, Line 20: select RAMSTAGE_CBMEM_TOP_ARG if !SOC_NVIDIA_TEGRA210
We need a way to pass arguments to this one. Otherwise it'll be a constant sore thumb. That or we drop this chipset from coreboot...
I'm not familiar with that platform but it would seem to require the argument to be in memory. How about picking a nice location within the linker script which is to be used by both romstage and ramstage (bottom of ram maybe?) and have platform_prog_run() in romstage put it there?