[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/broadwell: Use C_ENVIRONMENT_BOOTBLOCK

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30383 )

Change subject: soc/intel/broadwell: Use C_ENVIRONMENT_BOOTBLOCK
......................................................................

Patch Set 39:

(1 comment)

https://review.coreboot.org/#/c/30383/37/src/soc/intel/broadwell/Kconfig 
File src/soc/intel/broadwell/Kconfig:

https://review.coreboot.org/#/c/30383/37/src/soc/intel/broadwell/Kconfig@117 
PS37, Line 117: 	default 0x2000
> It can only be statically decided if there is no dynamic stack […]

No idea actually if the mrc.bin uses our stack. It has a separate heap in DCACHE_RAM_MRC_VAR_SIZE.

It might be a better idea to use cpu/intel/car/romstage.c as it places stack guards.