diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-16 11:24:09 -0700 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-17 03:12:39 +0100 |
| commit | 9c7c6f7213decfc0d0fee4bbc911a291ee93bcdb (patch) | |
| tree | d9e23ea1f89f75c6e178734bbe375860515bf3e2 /src/arch/x86/tables.c | |
| parent | d94cff6ab26d482554309041a9317cc3bf5e4b02 (diff) | |
arch/x86: Fix issues with braces detected by checkpatch
Fix the following errors and warnings detected by checkpatch.pl:
ERROR: open brace '{' following function declarations go on the next line
ERROR: that open brace { should be on the previous line
ERROR: else should follow close brace '}'
WARNING: braces {} are not necessary for any arm of this statement
WARNING: braces {} are not necessary for single statement blocks
TEST=Build and run on Galileo Gen2
Change-Id: I13d1967757e106c8300a15baed25d920c52a1a95
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18861
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/arch/x86/tables.c')
| -rw-r--r-- | src/arch/x86/tables.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/arch/x86/tables.c b/src/arch/x86/tables.c index edcb71705c..c63bd39ef7 100644 --- a/src/arch/x86/tables.c +++ b/src/arch/x86/tables.c @@ -47,9 +47,9 @@ static unsigned long write_pirq_table(unsigned long rom_table_end) new_high_table_pointer = write_pirq_routing_table(high_table_pointer); // FIXME make pirq table code intelligent enough to know how // much space it's going to need. - if (new_high_table_pointer > (high_table_pointer + MAX_PIRQ_TABLE_SIZE)) { + if (new_high_table_pointer > (high_table_pointer + + MAX_PIRQ_TABLE_SIZE)) printk(BIOS_ERR, "ERROR: Increase PIRQ size.\n"); - } printk(BIOS_DEBUG, "PIRQ table: %ld bytes.\n", new_high_table_pointer - high_table_pointer); } @@ -74,9 +74,9 @@ static unsigned long write_mptable(unsigned long rom_table_end) new_high_table_pointer = write_smp_table(high_table_pointer); // FIXME make mp table code intelligent enough to know how // much space it's going to need. - if (new_high_table_pointer > (high_table_pointer + MAX_MP_TABLE_SIZE)) { + if (new_high_table_pointer > (high_table_pointer + + MAX_MP_TABLE_SIZE)) printk(BIOS_ERR, "ERROR: Increase MP table size.\n"); - } printk(BIOS_DEBUG, "MP table: %ld bytes.\n", new_high_table_pointer - high_table_pointer); @@ -112,9 +112,9 @@ static unsigned long write_acpi_table(unsigned long rom_table_end) rom_table_end = ALIGN(rom_table_end, 16); new_high_table_pointer = write_acpi_tables(high_table_pointer); - if (new_high_table_pointer > ( high_table_pointer + MAX_ACPI_SIZE)) { + if (new_high_table_pointer > ( high_table_pointer + + MAX_ACPI_SIZE)) printk(BIOS_ERR, "ERROR: Increase ACPI size\n"); - } printk(BIOS_DEBUG, "ACPI tables: %ld bytes.\n", new_high_table_pointer - high_table_pointer); @@ -122,9 +122,9 @@ static unsigned long write_acpi_table(unsigned long rom_table_end) /* First we look for the high table RSDP */ while (acpi_start < new_high_table_pointer) { - if (memcmp(((acpi_rsdp_t *)acpi_start)->signature, RSDP_SIG, 8) == 0) { + if (memcmp(((acpi_rsdp_t *)acpi_start)->signature, + RSDP_SIG, 8) == 0) break; - } acpi_start++; } @@ -165,9 +165,9 @@ static unsigned long write_smbios_table(unsigned long rom_table_end) memcpy((void *)rom_table_end, (void *)high_table_pointer, sizeof(struct smbios_entry)); rom_table_end += sizeof(struct smbios_entry); - if (new_high_table_pointer > ( high_table_pointer + MAX_SMBIOS_SIZE)) { + if (new_high_table_pointer > ( high_table_pointer + + MAX_SMBIOS_SIZE)) printk(BIOS_ERR, "ERROR: Increase SMBIOS size\n"); - } printk(BIOS_DEBUG, "SMBIOS tables: %ld bytes.\n", new_high_table_pointer - high_table_pointer); } else { |