Marty E. Plummer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32373 )
Change subject: rockchip: rk3399: increase memory for fit payload. ......................................................................
Patch Set 2:
Increasing the ramstage to 2M (1M is not enough) is required to even build with the uImage embedded, otherwise you get
coreboot/util/crossgcc/xgcc/bin/aarch64-elf-ld.bfd: Ramstage exceeded its allotted size! (256K) make: *** [src/arch/arm64/Makefile.inc:154: build/cbfs/fallback/ramstage.debug] Error 1
After making this change, flashing and cold resetting, you get this:
CBFS: 'Master Header Locator' located CBFS at [20100:800000) CBFS: Locating 'fallback/payload' CBFS: Found @ offset 1a000 size 6f6a58 Payload not loaded.
Increasing the POSTRAM_CBFS_CACHE to 2M has the same results. 3M requires moving the RAMSTAGE, and has the same results. 4M has the same results. 5M has the same results. 6M has the same results. 7M is the minimal required size (assuming 1M increments) to get the payload booting in the first place, though a further change to the uImage required me to bump it to 8M.
So, what are your suggestions?