[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/systemagent/memmap.c: Align cached region

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45930 )

Change subject: soc/intel/common/block/systemagent/memmap.c: Align cached region
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45930/4/src/soc/intel/common/block/... 
File src/soc/intel/common/block/systemagent/memmap.c:

https://review.coreboot.org/c/coreboot/+/45930/4/src/soc/intel/common/block/... 
PS4, Line 59: FSP does not seem to bother w.r.t. alignment when asked to place cbmem_top()
> I think FSP-M places some data below the real TOLUD and reports the top below that, which explains the weird alignment.

Basically what the picture above lists between TSEG and TOLUM. The list
should be accurate, we acted on that information for a while and it worked
out mostly. Until we decided to just use FSP's HOP entry.