[coreboot-gerrit] [S] Change in coreboot[main]: drivers/intel/fsp2_0: Add FSP_DOES_NOT_NEED_TEMP_RAM

[Shuo Liu (Code Review)]

Attention is currently required from: Andrey Petrov, Arthur Heymans, Chen, Gang C, David Hendricks, Johnny Lin, Lean Sheng Tan, Patrick Rudolph, Ronak Kanabar.

Shuo Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80328?usp=email )

Change subject: drivers/intel/fsp2_0: Add FSP_DOES_NOT_NEED_TEMP_RAM
......................................................................

Patch Set 3:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/80328/comment/79cffe79_6a49a958 :
PS3, Line 8:
> > Need to justify your change. It does neither describe a problem nor why this change is necessary. […]

Yeah, it is for the 2S issue. https://review.coreboot.org/c/coreboot/+/61164 cannot pass boot in our internal system and hence we need to have a further solution on this.

The real reason behind is that SPR FSP will do static allocation for temp ram and bootloader should align the mapping accordingly, otherwise data will be corrupted mutually. CR61164 is not reaching to fully alignment and hence still cause issue in some machines.