Marc Jones has uploaded this change for review. ( https://review.coreboot.org/23818
Change subject: soc/amd/common: Save the UMA settings from AGESA ......................................................................
soc/amd/common: Save the UMA settings from AGESA
Save the UMA base and size settings returned by AGESA in amdinitpost();
Change-Id: Id96cc65582118ad41d397b1a600cab1615676a55 Signed-off-by: Marc Jones marcj303@gmail.com --- M 3rdparty/blobs M src/soc/amd/common/block/pi/agesawrapper.c 2 files changed, 7 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/18/23818/1
diff --git a/3rdparty/blobs b/3rdparty/blobs index 19dea8d..a5efee5 160000 --- a/3rdparty/blobs +++ b/3rdparty/blobs @@ -1 +1 @@ -Subproject commit 19dea8d171544f01f12ee6b78af0cc356ab994aa +Subproject commit a5efee5fdea55f398decc9cabebce3744bbd8147 diff --git a/src/soc/amd/common/block/pi/agesawrapper.c b/src/soc/amd/common/block/pi/agesawrapper.c index dca7222..cfa3760 100644 --- a/src/soc/amd/common/block/pi/agesawrapper.c +++ b/src/soc/amd/common/block/pi/agesawrapper.c @@ -26,6 +26,7 @@ #include <amdblocks/s3_resume.h> #include <amdblocks/agesawrapper.h> #include <amdblocks/BiosCallOuts.h> +#include <soc/southbridge.h>
void __attribute__((weak)) SetMemParams(AMD_POST_PARAMS *PostParams) {} void __attribute__((weak)) OemPostParams(AMD_POST_PARAMS *PostParams) {} @@ -207,12 +208,16 @@ * higher than UmaBase. With UMA_NONE we see UmaBase==0. */ uintptr_t top; - if (PostParams->MemConfig.UmaBase) + if (PostParams->MemConfig.UmaBase && + (PostParams->MemConfig.UmaBase < ((4ull * GiB) >> 16))) top = PostParams->MemConfig.UmaBase << 16; else top = PostParams->MemConfig.Sub4GCacheTop; backup_top_of_low_cacheable(top);
+ save_uma_size(PostParams->MemConfig.UmaSize * 64 * KiB); + save_uma_base((u64)PostParams->MemConfig.UmaBase * 64 * KiB); + print_init_post_settings(PostParams);
if (status != AGESA_SUCCESS)