Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/28474 )
Change subject: amd/stoneyridge: Adjust memory map for reserved ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/28474/1/src/soc/amd/stoneyridge/ramtop.c File src/soc/amd/stoneyridge/ramtop.c:
https://review.coreboot.org/#/c/28474/1/src/soc/amd/stoneyridge/ramtop.c@41 PS1, Line 41: BERT_REGION_MAX_SIZE CONFIG_SMM_TSEG_SIZE likely way bigger than ever required, but I think our mapping has some assumptions built into it. Need to look more closely.