commit 26eeb0f8ad554b1fa08d58080da8ce2d22081c1c
author Lee Leahy <leroy.p.leahy@intel.com> Wed Mar 15 18:08:50 2017 -0700
committer Lee Leahy <leroy.p.leahy@intel.com> Thu Mar 16 04:14:09 2017 +0100
tree 2b960b57e2175c94a852ca52e1c0db4abd1a69fb
parent 73a28942031675fce20d6649d2c2ce66fe62f416

cpu/intel: Fix brace issues detected by checkpatch.pl

Fix the following error and warning detected by checkpatch.pl:

ERROR: that open brace { should be on the previous line
WARNING: braces {} are not necessary for single statement blocks

TEST=Build and run on Galileo Gen2

Change-Id: Icdd6bd9ae578589b4d42002d200fa8f83920265e
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18849
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>

src/cpu/intel/hyperthreading/intel_sibling.c

diff --git a/src/cpu/intel/hyperthreading/intel_sibling.c b/src/cpu/intel/hyperthreading/intel_sibling.c
index 640d9b3..57aa00c 100644
--- a/src/cpu/intel/hyperthreading/intel_sibling.c
+++ b/src/cpu/intel/hyperthreading/intel_sibling.c
@@ -63,14 +63,13 @@
 	}
 	result = cpuid(1);
 	/* Is hyperthreading supported */
-	if (!(result.edx & (1 << 28))) {
+	if (!(result.edx & (1 << 28)))
 		return;
-	}
+
 	/* See how many sibling cpus we have */
 	siblings = (result.ebx >> 16) & 0xff;
-	if (siblings < 1) {
+	if (siblings < 1)
 		siblings = 1;
-	}
 
 	printk(BIOS_DEBUG, "CPU: %u %d siblings\n",
 		cpu->path.apic.apic_id,
@@ -78,9 +77,8 @@
 
 	/* See if I am a sibling cpu */
 	if (cpu->path.apic.apic_id & (siblings - 1)) {
-		if (disable_siblings) {
+		if (disable_siblings)
 			cpu->enabled = 0;
-		}
 		return;
 	}
 
@@ -98,9 +96,8 @@
 		 */
 		new = alloc_find_dev(cpu->bus, &cpu_path);
 
-		if (!new) {
+		if (!new)
 			continue;
-		}
 
 		printk(BIOS_DEBUG, "CPU: %u has sibling %u\n",
 			cpu->path.apic.apic_id,