[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl, icl}: Allocate 64KB as FSP heap

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35237 )

Change subject: soc/intel/{cnl, icl}: Allocate 64KB as FSP heap
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/35237/4/src/soc/intel/cannonlake/Kc... 
File src/soc/intel/cannonlake/Kconfig:

https://review.coreboot.org/c/coreboot/+/35237/4/src/soc/intel/cannonlake/Kc... 
PS4, Line 133: The amount of anticipated heap usage in CAR by FSP.
Can we add details here about where the size requirement is coming from i.e. doc# if any? Or at least the integration guide name?

https://review.coreboot.org/c/coreboot/+/35237/4/src/soc/intel/icelake/Kconf... 
File src/soc/intel/icelake/Kconfig:

https://review.coreboot.org/c/coreboot/+/35237/4/src/soc/intel/icelake/Kconf... 
PS4, Line 84: The amount of anticipated heap usage in CAR by 
Same here?