x86: Change to use start/end address pair in write_tables()
authorBin Meng <bmeng.cn@gmail.com>
Sun, 28 Feb 2016 06:57:58 +0000 (22:57 -0800)
committerBin Meng <bmeng.cn@gmail.com>
Thu, 17 Mar 2016 02:27:22 +0000 (10:27 +0800)
Add a new variable rom_table_start and pass it to ROM table write
routines. This reads better than previous single rom_table_end.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
arch/x86/lib/tables.c

index 14b15cf..fc6c0d8 100644 (file)
@@ -39,26 +39,32 @@ void table_fill_string(char *dest, const char *src, size_t n, char pad)
 
 void write_tables(void)
 {
-       u32 __maybe_unused rom_table_end = ROM_TABLE_ADDR;
+       u32 __maybe_unused rom_table_start = ROM_TABLE_ADDR;
+       u32 __maybe_unused rom_table_end;
 
 #ifdef CONFIG_GENERATE_PIRQ_TABLE
-       rom_table_end = write_pirq_routing_table(rom_table_end);
+       rom_table_end = write_pirq_routing_table(rom_table_start);
        rom_table_end = ALIGN(rom_table_end, 1024);
+       rom_table_start = rom_table_end;
 #endif
 #ifdef CONFIG_GENERATE_SFI_TABLE
-       rom_table_end = write_sfi_table(rom_table_end);
+       rom_table_end = write_sfi_table(rom_table_start);
        rom_table_end = ALIGN(rom_table_end, 1024);
+       rom_table_start = rom_table_end;
 #endif
 #ifdef CONFIG_GENERATE_MP_TABLE
-       rom_table_end = write_mp_table(rom_table_end);
+       rom_table_end = write_mp_table(rom_table_start);
        rom_table_end = ALIGN(rom_table_end, 1024);
+       rom_table_start = rom_table_end;
 #endif
 #ifdef CONFIG_GENERATE_ACPI_TABLE
-       rom_table_end = write_acpi_tables(rom_table_end);
+       rom_table_end = write_acpi_tables(rom_table_start);
        rom_table_end = ALIGN(rom_table_end, 1024);
+       rom_table_start = rom_table_end;
 #endif
 #ifdef CONFIG_GENERATE_SMBIOS_TABLE
-       rom_table_end = write_smbios_table(rom_table_end);
+       rom_table_end = write_smbios_table(rom_table_start);
        rom_table_end = ALIGN(rom_table_end, 1024);
+       rom_table_start = rom_table_end;
 #endif
 }