x86: Use if instead of #ifdef in write_tables()

Use if() to remove the extra build path in this code.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
This commit is contained in:
Simon Glass 2020-11-04 09:57:24 -07:00 committed by Bin Meng
parent cac9c6a38f
commit f36e4c7d80

View file

@ -68,10 +68,8 @@ int write_tables(void)
{ {
u32 rom_table_start = ROM_TABLE_ADDR; u32 rom_table_start = ROM_TABLE_ADDR;
u32 rom_table_end; u32 rom_table_end;
#ifdef CONFIG_SEABIOS
u32 high_table, table_size; u32 high_table, table_size;
struct memory_area cfg_tables[ARRAY_SIZE(table_list) + 1]; struct memory_area cfg_tables[ARRAY_SIZE(table_list) + 1];
#endif
int i; int i;
debug("Writing tables to %x:\n", rom_table_start); debug("Writing tables to %x:\n", rom_table_start);
@ -81,30 +79,32 @@ int write_tables(void)
rom_table_end = table->write(rom_table_start); rom_table_end = table->write(rom_table_start);
rom_table_end = ALIGN(rom_table_end, ROM_TABLE_ALIGN); rom_table_end = ALIGN(rom_table_end, ROM_TABLE_ALIGN);
#ifdef CONFIG_SEABIOS if (IS_ENABLED(CONFIG_SEABIOS)) {
table_size = rom_table_end - rom_table_start; table_size = rom_table_end - rom_table_start;
high_table = (u32)high_table_malloc(table_size); high_table = (u32)(ulong)high_table_malloc(table_size);
if (high_table) { if (high_table) {
table->write(high_table); table->write(high_table);
cfg_tables[i].start = high_table; cfg_tables[i].start = high_table;
cfg_tables[i].size = table_size; cfg_tables[i].size = table_size;
} else { } else {
printf("%d: no memory for configuration tables\n", i); printf("%d: no memory for configuration tables\n",
i);
return -ENOSPC; return -ENOSPC;
} }
#endif }
debug("- wrote '%s' to %x, end %x\n", table->name, debug("- wrote '%s' to %x, end %x\n", table->name,
rom_table_start, rom_table_end); rom_table_start, rom_table_end);
rom_table_start = rom_table_end; rom_table_start = rom_table_end;
} }
#ifdef CONFIG_SEABIOS if (IS_ENABLED(CONFIG_SEABIOS)) {
/* make sure the last item is zero */ /* make sure the last item is zero */
cfg_tables[i].size = 0; cfg_tables[i].size = 0;
write_coreboot_table(CB_TABLE_ADDR, cfg_tables); write_coreboot_table(CB_TABLE_ADDR, cfg_tables);
#endif }
debug("- done writing tables\n"); debug("- done writing tables\n");
return 0; return 0;