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

[Arthur Heymans (Code Review)]

Arthur Heymans 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 4:

(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've seen it happen with the Kaby and Coffee Lake FSPs. I don't have any other platforms to test on. […]

I think FSP-M places some data below the real TOLUD and reports the top below that, which explains the weird alignment.