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/mpspec.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/mpspec.c')
| -rw-r--r-- | src/arch/x86/mpspec.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/arch/x86/mpspec.c b/src/arch/x86/mpspec.c index 5f251e31c9..da2c21ab72 100644 --- a/src/arch/x86/mpspec.c +++ b/src/arch/x86/mpspec.c @@ -63,9 +63,8 @@ static unsigned char smp_compute_checksum(void *v, int len) int i; bytes = v; checksum = 0; - for (i = 0; i < len; i++) { + for (i = 0; i < len; i++) checksum -= bytes[i]; - } return checksum; } @@ -443,7 +442,9 @@ void mptable_add_isa_interrupts(struct mp_config_table *mc, unsigned long bus_is smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_EDGE|MP_IRQ_POLARITY_HIGH, bus_isa, 0xf, apicid, 0xf); } -void mptable_write_buses(struct mp_config_table *mc, int *max_pci_bus, int *isa_bus) { +void mptable_write_buses(struct mp_config_table *mc, int *max_pci_bus, + int *isa_bus) +{ int dummy, i, highest; char buses[256]; struct device *dev; |