[coreboot-gerrit] Change in coreboot[master]: soc/amd: Use common functions to pass UMA information to ramstage

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36292 )

Change subject: soc/amd: Use common functions to pass UMA information to ramstage
......................................................................

Patch Set 2:

(1 comment)

Still -1 as biosram is required for cbmem_top(), UMA base/size change is probably fine.

https://review.coreboot.org/c/coreboot/+/36292/2/src/soc/amd/common/block/pi... 
File src/soc/amd/common/block/pi/agesawrapper.c:

https://review.coreboot.org/c/coreboot/+/36292/2/src/soc/amd/common/block/pi... 
PS2, Line 209: 		 PostParams->MemConfig.UmaSize * 64 * KiB);
Ah... I did not review preceding work and did not realise this does not depend of cbmem_initialize() having being called.