diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-04-19 19:57:01 +0300 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-05-27 13:54:47 +0200 |
commit | 70d92b9465b1edf646b25b89f1442f7107b5f1f6 (patch) | |
tree | 8d0a39990358f3fd92b00f0e790b7667ca90fd1c /src/southbridge/amd/cimx/sb700 | |
parent | ef8bb9136e9371753e50cb15b334c9d0f5c70930 (diff) |
CBMEM: Clarify CBMEM_TOP_BACKUP function usage
The deprecated LATE_CBMEM_INIT function is renamed:
set_top_of_ram -> set_late_cbmem_top
Obscure term top_of_ram is replaced:
backup_top_of_ram -> backup_top_of_low_cacheable
get_top_of_ram -> restore_top_of_low_cacheable
New function that always resolves to CBMEM top boundary, with
or without SMM, is named restore_cbmem_top().
Change-Id: I61d20f94840ad61e9fd55976e5aa8c27040b8fb7
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/19377
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>
Diffstat (limited to 'src/southbridge/amd/cimx/sb700')
-rw-r--r-- | src/southbridge/amd/cimx/sb700/ramtop.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/southbridge/amd/cimx/sb700/ramtop.c b/src/southbridge/amd/cimx/sb700/ramtop.c index f59a9a346b..cbc4596f57 100644 --- a/src/southbridge/amd/cimx/sb700/ramtop.c +++ b/src/southbridge/amd/cimx/sb700/ramtop.c @@ -18,7 +18,7 @@ #include <cbmem.h> #include <southbridge/amd/cimx/cimx_util.h> -void backup_top_of_ram(uint64_t ramtop) +void backup_top_of_low_cacheable(uintptr_t ramtop) { u32 dword = ramtop; int nvram_pos = 0xfc, i; @@ -29,7 +29,7 @@ void backup_top_of_ram(uint64_t ramtop) } } -unsigned long get_top_of_ram(void) +uintptr_t restore_top_of_low_cacheable(void) { u32 xdata = 0; int xnvram_pos = 0xfc, xi; @@ -39,5 +39,5 @@ unsigned long get_top_of_ram(void) xdata |= inb(BIOSRAM_DATA) << (xi *8); xnvram_pos++; } - return (unsigned long) xdata; + return xdata; } |