Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35233 )
Change subject: intel/fsp2_0: Move temporary RAM to .bss with FSP_USES_CB_STACK ......................................................................
Patch Set 6:
(2 comments)
https://review.coreboot.org/c/coreboot/+/35233/3/src/drivers/intel/fsp2_0/Kc... File src/drivers/intel/fsp2_0/Kconfig:
https://review.coreboot.org/c/coreboot/+/35233/3/src/drivers/intel/fsp2_0/Kc... PS3, Line 157: 0x10000
Cleanest would be to explicitly have only SoC set the value. […]
Done
https://review.coreboot.org/c/coreboot/+/35233/4/src/drivers/intel/fsp2_0/Kc... File src/drivers/intel/fsp2_0/Kconfig:
https://review.coreboot.org/c/coreboot/+/35233/4/src/drivers/intel/fsp2_0/Kc... PS4, Line 158: depends on FSP_USES_CB_STACK
It would be good to have a help text explaining what this is used for. […]
Done