diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-04-15 20:07:53 +0300 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-05-18 06:48:57 +0200 |
| commit | 61be3603f4b9f353e605d7b7c8d0d9f3b90f5636 (patch) | |
| tree | 0de776853b482a83faff1ca5bbfdc2df243f7214 /src/northbridge/amd/agesa/agesawrapper.c | |
| parent | 17bb225be7dd031b9803f33dec88e9d53e3a582f (diff) | |
AGESA: Fix UMA calculations
Vendorcode decides already in AMD_INIT_POST the exact location
of UMA memory. To meet alignment requirements, it will extend
uma_memory_size. We cannot calculate base from size and TOP_MEM1,
but need to calculate size from base and TOP_MEM1 instead.
Also allows selection of UmaMode==UMA_SPECIFIED to manually set
amount of memory reserved for framebuffer.
Change-Id: I2514c70a331c7fbf0056f22bf64f19c9374754c0
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/19328
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Diffstat (limited to 'src/northbridge/amd/agesa/agesawrapper.c')
| -rw-r--r-- | src/northbridge/amd/agesa/agesawrapper.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/northbridge/amd/agesa/agesawrapper.c b/src/northbridge/amd/agesa/agesawrapper.c index efacaff9e9..079625671a 100644 --- a/src/northbridge/amd/agesa/agesawrapper.c +++ b/src/northbridge/amd/agesa/agesawrapper.c @@ -13,6 +13,7 @@ * GNU General Public License for more details. */ +#include <cbmem.h> #include <stdint.h> #include <string.h> @@ -110,6 +111,8 @@ AGESA_STATUS agesawrapper_amdinitpost(void) status = AmdInitPost(PostParams); AGESA_EVENTLOG(status, &PostParams->StdHeader); + backup_top_of_ram(PostParams->MemConfig.Sub4GCacheTop); + AmdReleaseStruct(&AmdParamStruct); return status; |