GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

arch/riscv/tables: remove confusion over write_tables() 

Apparently the memo was missed about the write_tables()
signature. Fix the confusion.

Change-Id: I8ef367345dd54584c57e9d5cd8cc3d81ce109fef
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/14421
Tested-by: build bot (Jenkins)
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
Aaron Durbin 2016-04-19 13:42:27 -05:00
parent f7dca0a275
commit 0eb1f1cba9
1 changed files with 2 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/arch/riscv/tables.c
View File

@ -25,8 +25,7 @@
#define MAX_COREBOOT_TABLE_SIZE (8 * 1024) #define MAX_COREBOOT_TABLE_SIZE (8 * 1024)
// WTF. this does not agre with the prototype! void write_tables(void)
static struct lb_memory *wtf_write_tables(void)
{ {
unsigned long table_pointer, new_table_pointer; unsigned long table_pointer, new_table_pointer;
@ -36,7 +35,7 @@ static struct lb_memory *wtf_write_tables(void)
MAX_COREBOOT_TABLE_SIZE); MAX_COREBOOT_TABLE_SIZE);
if (!table_pointer) { if (!table_pointer) {
printk(BIOS_ERR, "Could not add CBMEM for coreboot table.\n"); printk(BIOS_ERR, "Could not add CBMEM for coreboot table.\n");
return NULL; return;
} }
new_table_pointer = write_coreboot_table(0UL, 0UL, new_table_pointer = write_coreboot_table(0UL, 0UL,
@ -54,13 +53,6 @@ static struct lb_memory *wtf_write_tables(void)
/* Print CBMEM sections */ /* Print CBMEM sections */
cbmem_list(); cbmem_list();
// return get_lb_mem();
return NULL;
}
void write_tables(void)
{
wtf_write_tables();
} }
void lb_arch_add_records(struct lb_header *header) void lb_arch_add_records(struct lb_header *header)