diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2014-02-18 19:36:05 -0600 |
|---|---|---|
| committer | Aaron Durbin <adurbin@google.com> | 2014-03-03 23:25:53 +0100 |
| commit | 3f7ad7b216b4021c7cb93201a94b0fae46f5e19e (patch) | |
| tree | c48ac8ce036cc5be5029e6089e583f2d8746aa8c /src/arch/armv7 | |
| parent | b013c279a93627c7edeb10cb5b976483e61166d9 (diff) | |
| download | coreboot-3f7ad7b216b4021c7cb93201a94b0fae46f5e19e.tar.xz | |
coreboot: don't return struct lb_memory * from write_tables()
No one is interrogating the write_tables() return value. Therefore,
drop it.
Change-Id: I97e707f071942239c9a0fa0914af3679ee7a9c3c
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/5301
Tested-by: build bot (Jenkins)
Reviewed-by: Vladimir Serbinenko <phcoder@gmail.com>
Diffstat (limited to 'src/arch/armv7')
| -rw-r--r-- | src/arch/armv7/tables.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/arch/armv7/tables.c b/src/arch/armv7/tables.c index 7e7cbe409c..20c5a9f7a8 100644 --- a/src/arch/armv7/tables.c +++ b/src/arch/armv7/tables.c @@ -46,7 +46,7 @@ void cbmem_fail_resume(void) { } -struct lb_memory *write_tables(void) +void write_tables(void) { unsigned long table_pointer, new_table_pointer; @@ -56,7 +56,7 @@ struct lb_memory *write_tables(void) MAX_COREBOOT_TABLE_SIZE); if (!table_pointer) { printk(BIOS_ERR, "Could not add CBMEM for coreboot table.\n"); - return NULL; + return; } new_table_pointer = write_coreboot_table(0UL, 0UL, @@ -74,6 +74,4 @@ struct lb_memory *write_tables(void) /* Print CBMEM sections */ cbmem_list(); - - return get_lb_mem(); } |