summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2014-01-05 08:49:29 +0200
committerKyösti Mälkki <kyosti.malkki@gmail.com>2014-01-06 17:33:51 +0100
commit5b353002a96f1cad9be96b14781f391c6d3faeff (patch)
tree2ee49ecb26a7af54e8a941a8212b59bc961b2b7e
parentcb28f3f8ed13cdd9ae17441dacf6409eaf390105 (diff)
downloadcoreboot-5b353002a96f1cad9be96b14781f391c6d3faeff.tar.xz
CBMEM: Drop cbmem_base_check()
This function was for logging only, but we have both base and size already logged elsewhere. Change-Id: Ie6ac71fc859b8fd42fcf851c316a5f888f828dc2 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: http://review.coreboot.org/4620 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Vladimir Serbinenko <phcoder@gmail.com> Reviewed-by: Aaron Durbin <adurbin@google.com>
-rw-r--r--src/arch/armv7/tables.c4
-rw-r--r--src/arch/x86/boot/tables.c4
-rw-r--r--src/include/cbmem.h1
-rw-r--r--src/lib/cbmem.c11
4 files changed, 0 insertions, 20 deletions
diff --git a/src/arch/armv7/tables.c b/src/arch/armv7/tables.c
index 3f5338c821..7e7cbe409c 100644
--- a/src/arch/armv7/tables.c
+++ b/src/arch/armv7/tables.c
@@ -50,10 +50,6 @@ struct lb_memory *write_tables(void)
{
unsigned long table_pointer, new_table_pointer;
-#if !CONFIG_DYNAMIC_CBMEM
- cbmem_base_check();
-#endif
-
post_code(0x9d);
table_pointer = (unsigned long)cbmem_add(CBMEM_ID_CBTABLE,
diff --git a/src/arch/x86/boot/tables.c b/src/arch/x86/boot/tables.c
index eea9bf1c12..40bf087cc3 100644
--- a/src/arch/x86/boot/tables.c
+++ b/src/arch/x86/boot/tables.c
@@ -50,10 +50,6 @@ struct lb_memory *write_tables(void)
*/
unsigned long high_table_pointer;
-#if !CONFIG_DYNAMIC_CBMEM
- cbmem_base_check();
-#endif
-
rom_table_start = 0xf0000;
rom_table_end = 0xf0000;
diff --git a/src/include/cbmem.h b/src/include/cbmem.h
index f9d268aa8b..746f40cbcf 100644
--- a/src/include/cbmem.h
+++ b/src/include/cbmem.h
@@ -134,7 +134,6 @@ u64 cbmem_entry_size(const struct cbmem_entry *entry);
void set_top_of_ram(uint64_t ramtop);
void backup_top_of_ram(uint64_t ramtop);
void cbmem_late_set_table(uint64_t base, uint64_t size);
-int cbmem_base_check(void);
#endif
int cbmem_reinit(void);
diff --git a/src/lib/cbmem.c b/src/lib/cbmem.c
index e6d39132d4..6b0cbc8948 100644
--- a/src/lib/cbmem.c
+++ b/src/lib/cbmem.c
@@ -256,17 +256,6 @@ BOOT_STATE_INIT_ENTRIES(cbmem_bscb) = {
init_cbmem_post_device, NULL),
};
-int cbmem_base_check(void)
-{
- if (!cbmem_base) {
- printk(BIOS_ERR, "ERROR: CBMEM Base is not set.\n");
- // Are there any boards without?
- // Stepan thinks we should die() here!
- }
- printk(BIOS_DEBUG, "CBMEM Base is %llx.\n", cbmem_base);
- return !!cbmem_base;
-}
-
void cbmem_add_lb_mem(struct lb_memory *mem)
{
lb_add_memory_range(mem, LB_MEM_TABLE, cbmem_base, cbmem_size);