diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-15 18:08:50 -0700 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-16 04:14:09 +0100 |
| commit | 26eeb0f8ad554b1fa08d58080da8ce2d22081c1c (patch) | |
| tree | 2b960b57e2175c94a852ca52e1c0db4abd1a69fb /src/cpu/intel/hyperthreading | |
| parent | 73a28942031675fce20d6649d2c2ce66fe62f416 (diff) | |
| download | coreboot-26eeb0f8ad554b1fa08d58080da8ce2d22081c1c.tar.xz | |
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>
Diffstat (limited to 'src/cpu/intel/hyperthreading')
| -rw-r--r-- | src/cpu/intel/hyperthreading/intel_sibling.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/src/cpu/intel/hyperthreading/intel_sibling.c b/src/cpu/intel/hyperthreading/intel_sibling.c index 640d9b37f5..57aa00c007 100644 --- a/src/cpu/intel/hyperthreading/intel_sibling.c +++ b/src/cpu/intel/hyperthreading/intel_sibling.c @@ -63,14 +63,13 @@ void intel_sibling_init(struct device *cpu) } 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 @@ void intel_sibling_init(struct device *cpu) /* 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 @@ void intel_sibling_init(struct device *cpu) */ 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, |