[coreboot-gerrit] New patch to review for coreboot: arch/power8/tables: remove confusion over write_tables()

[Aaron Durbin]

Aaron Durbin ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14420

-gerrit

commit a2da8a0c72ac309b38570cb11aec8d79fc2213be
Author: Aaron Durbin 
Date:   Tue Apr 19 13:41:16 2016 -0500

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

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

Change-Id: I63924be47d3507d2d7ed006a553414f4ac60d2f9
    Signed-off-by: Aaron Durbin 
---
 src/arch/power8/tables.c | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/src/arch/power8/tables.c b/src/arch/power8/tables.c
index d2b2f34..b5bc9ab 100644
--- a/src/arch/power8/tables.c
+++ b/src/arch/power8/tables.c
@@ -25,8 +25,7 @@

#define MAX_COREBOOT_TABLE_SIZE (8 * 1024)

-// WTF. this does not agree with the prototype!
-static struct lb_memory *wtf_write_tables(void)
+void write_tables(void)
 {
    unsigned long table_pointer, new_table_pointer;

@@ -36,7 +35,7 @@ static struct lb_memory *wtf_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,
@@ -55,16 +54,8 @@ static struct lb_memory *wtf_write_tables(void)

/* Print CBMEM sections */
    cbmem_list();
-
-//	return get_lb_mem();
-	return NULL;
 }

void lb_arch_add_records(struct lb_header *header)
 {
 }
-
-void write_tables(void)
-{
-	wtf_write_tables();
-}