diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-15 17:40:50 -0700 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-16 04:13:24 +0100 |
| commit | 9d62e7e75e43d6737df9d0ab5603446d7f5e408d (patch) | |
| tree | 318963e23b89ca65cc05502774d0af1e831af578 /src/cpu/intel/model_2065x | |
| parent | 7b5f12b9b2695359b6ccb4d62bdb868166c7f8d1 (diff) | |
| download | coreboot-9d62e7e75e43d6737df9d0ab5603446d7f5e408d.tar.gz coreboot-9d62e7e75e43d6737df9d0ab5603446d7f5e408d.tar.bz2 coreboot-9d62e7e75e43d6737df9d0ab5603446d7f5e408d.zip | |
cpu/intel: Fix the spacing issues
Fix the following errors and warnings detected by checkpatch.pl:
ERROR: spaces required around that '=' (ctx:VxV)
ERROR: space required after that ',' (ctx:VxV)
ERROR: space prohibited after that open parenthesis '('
ERROR: space prohibited before that close parenthesis ')'
ERROR: need consistent spacing around '-' (ctx:WxV)
ERROR: spaces required around that '>' (ctx:VxV)
ERROR: need consistent spacing around '>>' (ctx:WxV)
ERROR: need consistent spacing around '<<' (ctx:VxW)
ERROR: spaces required around that '||' (ctx:VxV)
ERROR: "foo * bar" should be "foo *bar"
ERROR: "(foo*)" should be "(foo *)"
WARNING: space prohibited between function name and open parenthesis '('
WARNING: storage class should be at the beginning of the declaration
TEST=Build and run on Galileo Gen2
Change-Id: I6602fbc8602171ab6c2f3b6c204558ad2c811179
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18847
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/cpu/intel/model_2065x')
| -rw-r--r-- | src/cpu/intel/model_2065x/acpi.c | 4 | ||||
| -rw-r--r-- | src/cpu/intel/model_2065x/model_2065x_init.c | 5 |
2 files changed, 5 insertions, 4 deletions
diff --git a/src/cpu/intel/model_2065x/acpi.c b/src/cpu/intel/model_2065x/acpi.c index 262258938390..e64405acf4d0 100644 --- a/src/cpu/intel/model_2065x/acpi.c +++ b/src/cpu/intel/model_2065x/acpi.c @@ -306,8 +306,8 @@ void generate_cpu_entries(device_t device) numcpus, cores_per_package); for (cpuID = 1; cpuID <= numcpus; cpuID++) { - for (coreID=1; coreID<=cores_per_package; coreID++) { - if (coreID>1) { + for (coreID = 1; coreID <= cores_per_package; coreID++) { + if (coreID > 1) { pcontrol_blk = 0; plen = 0; } diff --git a/src/cpu/intel/model_2065x/model_2065x_init.c b/src/cpu/intel/model_2065x/model_2065x_init.c index 8f1311ec9cd2..68e86a66633d 100644 --- a/src/cpu/intel/model_2065x/model_2065x_init.c +++ b/src/cpu/intel/model_2065x/model_2065x_init.c @@ -42,7 +42,7 @@ */ static acpi_cstate_t cstate_map[] = { { /* 0: C0 */ - },{ /* 1: C1 */ + }, { /* 1: C1 */ .latency = 1, .power = 1000, .resource = { @@ -321,7 +321,8 @@ static void model_2065x_init(struct device *cpu) /* Print processor name */ fill_processor_name(processor_name); printk(BIOS_INFO, "CPU: %s.\n", processor_name); - printk(BIOS_INFO, "CPU:lapic=%ld, boot_cpu=%d\n", lapicid (), boot_cpu ()); + printk(BIOS_INFO, "CPU:lapic=%ld, boot_cpu=%d\n", lapicid(), + boot_cpu()); /* Setup MTRRs based on physical address size */ x86_setup_mtrrs_with_detect(); |