Lean Sheng Tan has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84302?usp=email )
Change subject: soc/intel/xeon_sp/gnr: Use default DCACHE_BSP_STACK_SIZE ......................................................................
soc/intel/xeon_sp/gnr: Use default DCACHE_BSP_STACK_SIZE
For Xeon-SP, DCACHE_BSP_STACK_SIZE is by default 0x10000. For GNR, this default size is enough. Use the default size so that more CAR spaces could be saved for other purpose.
Change-Id: I68a79df150c4954ef8d703987d7c0bb446ba4cda Signed-off-by: Gang Chen gang.c.chen@intel.com Signed-off-by: Shuo Liu shuo.liu@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/84302 Reviewed-by: Lean Sheng Tan sheng.tan@9elements.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/intel/xeon_sp/gnr/Kconfig 1 file changed, 0 insertions(+), 9 deletions(-)
Approvals: Lean Sheng Tan: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/soc/intel/xeon_sp/gnr/Kconfig b/src/soc/intel/xeon_sp/gnr/Kconfig index 790f20e..fae9d8d 100644 --- a/src/soc/intel/xeon_sp/gnr/Kconfig +++ b/src/soc/intel/xeon_sp/gnr/Kconfig @@ -56,15 +56,6 @@ and/or romstage. FSP-T reserves the upper 0x100 for FspReservedBuffer.
-config DCACHE_BSP_STACK_SIZE - hex - default 0x60000 - help - The amount of anticipated stack usage in CAR by bootblock and - other stages. It needs to include FSP-M stack requirement and - CB romstage stack requirement. The integration documentation - says this needs to be 256KiB. - config FSP_M_RC_HEAP_SIZE hex default 0x142000