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/pirq_routing.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/pirq_routing.c')
-rw-r--r-- | src/arch/x86/pirq_routing.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/arch/x86/pirq_routing.c b/src/arch/x86/pirq_routing.c index f758dbb572..e534a0656b 100644 --- a/src/arch/x86/pirq_routing.c +++ b/src/arch/x86/pirq_routing.c @@ -100,8 +100,7 @@ static u8 pirq_get_next_free_irq(u8* pirq, u16 bitmap) { int i, link; u8 irq = 0; - for (i = 2; i <= 15; i++) - { + for (i = 2; i <= 15; i++) { /* Can we assign this IRQ ? */ if (!((bitmap >> i) & 1)) continue; @@ -161,13 +160,11 @@ static void pirq_route_irqs(unsigned long addr) } /* yet not routed */ - if (!pirq[link - 1]) - { + if (!pirq[link - 1]) { irq = pirq_get_next_free_irq(pirq, bitmap); if (irq) pirq[link - 1] = irq; - } - else + } else irq = pirq[link - 1]; printk(BIOS_DEBUG, "IRQ: %d\n", irq); |