erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

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

Browse source 
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: I19682d7da511d778cb2c9c41b9d2cb60e560e555
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Commit-Id: 26eeb0f8ad
Original-Change-Id: Icdd6bd9ae578589b4d42002d200fa8f83920265e
Original-Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Original-Reviewed-on: https://review.coreboot.org/18849
Original-Tested-by: build bot (Jenkins)
Original-Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://chromium-review.googlesource.com/456246
This commit is contained in:
Lee Leahy 2017-03-15 18:08:50 -07:00 • committed by chrome-bot
commit feb3940707
8 changed files with 15 additions and 28 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/cpu/intel/hyperthreading/intel_sibling.c
View file

@ -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,