diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2015-03-19 06:58:15 +0200 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2015-03-19 06:17:07 +0100 |
commit | 4da487eae8858dc132d60b7ffb0015e9c96b0df6 (patch) | |
tree | 55b7e67bdc7e63256748e196fec5aab030040404 /src/southbridge/amd/sb700 | |
parent | e9424c593ad4bc3756a4c44e10ea1ec2fab1c3f5 (diff) |
CBMEM: Add LATE_CBMEM_INIT guards
Our target is to get rid of backup_top_of_ram() and get_top_of_ram()
entirely so only declare these with LATE_CBMEM_INIT=y.
Change-Id: I54f549fe774996f4d803f9ec527e0fac46f6576f
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: http://review.coreboot.org/8749
Tested-by: build bot (Jenkins)
Reviewed-by: Timothy Pearson <tpearson@raptorengineeringinc.com>
Diffstat (limited to 'src/southbridge/amd/sb700')
-rw-r--r-- | src/southbridge/amd/sb700/early_setup.c | 2 | ||||
-rw-r--r-- | src/southbridge/amd/sb700/lpc.c | 2 |
2 files changed, 4 insertions, 0 deletions
diff --git a/src/southbridge/amd/sb700/early_setup.c b/src/southbridge/amd/sb700/early_setup.c index 94f33b75de..50909f8b60 100644 --- a/src/southbridge/amd/sb700/early_setup.c +++ b/src/southbridge/amd/sb700/early_setup.c @@ -728,6 +728,7 @@ int acpi_get_sleep_type(void) return ((tmp & (7 << 10)) >> 10); } +#if IS_ENABLED(CONFIG_LATE_CBMEM_INIT) unsigned long get_top_of_ram(void) { uint32_t xdata = 0; @@ -742,5 +743,6 @@ unsigned long get_top_of_ram(void) } return (unsigned long) xdata; } +#endif #endif diff --git a/src/southbridge/amd/sb700/lpc.c b/src/southbridge/amd/sb700/lpc.c index 658e95498e..7bb21d2945 100644 --- a/src/southbridge/amd/sb700/lpc.c +++ b/src/southbridge/amd/sb700/lpc.c @@ -88,6 +88,7 @@ int acpi_get_sleep_type(void) return ((tmp & (7 << 10)) >> 10); } +#if IS_ENABLED(CONFIG_LATE_CBMEM_INIT) void backup_top_of_ram(uint64_t ramtop) { u32 dword = (u32) ramtop; @@ -98,6 +99,7 @@ void backup_top_of_ram(uint64_t ramtop) nvram_pos++; } } +#endif static void sb700_lpc_read_resources(device_t dev) { |