diff options
author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-10 13:42:34 -0800 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-03-13 17:08:34 +0100 |
commit | 68ab0b5d1f7993735c6b113a624259bbf443ae98 (patch) | |
tree | 3adb1a21cc194700e6d911a521d0cd56a54e7280 | |
parent | 42e605be195b50606c8113f7f6ade3dbcdab462e (diff) | |
download | coreboot-68ab0b5d1f7993735c6b113a624259bbf443ae98.tar.xz |
soc/intel/common: Fix spacing issues
Fix the following errors detected by checkpatch.pl:
ERROR: spaces required around that '=' (ctx:VxV)
ERROR: spaces required around that ':' (ctx:VxW)
ERROR: space required after that ',' (ctx:VxV)
ERROR: space required before the open parenthesis '('
ERROR: switch and case should be at the same indent
ERROR: "foo ** bar" should be "foo **bar"
TEST=Build and run on Galileo Gen2
Change-Id: I52ba2a3c1e0fffad7145eecd878aba8dc450ac0b
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18755
Tested-by: build bot (Jenkins)
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
-rw-r--r-- | src/soc/intel/common/acpi/acpi.c | 4 | ||||
-rw-r--r-- | src/soc/intel/common/block/cse/cse.c | 2 | ||||
-rw-r--r-- | src/soc/intel/common/hda_verb.c | 4 | ||||
-rw-r--r-- | src/soc/intel/common/lpss_i2c.h | 12 | ||||
-rw-r--r-- | src/soc/intel/common/smihandler.c | 64 |
5 files changed, 43 insertions, 43 deletions
diff --git a/src/soc/intel/common/acpi/acpi.c b/src/soc/intel/common/acpi/acpi.c index 63877ee668..218430fc56 100644 --- a/src/soc/intel/common/acpi/acpi.c +++ b/src/soc/intel/common/acpi/acpi.c @@ -273,8 +273,8 @@ void generate_cpu_entries(device_t device) printk(BIOS_DEBUG, "Found %d CPU(s) with %d core(s) each.\n", numcpus, cores_per_package); - for (cpu_id=0; cpu_id < numcpus; cpu_id++) { - for (core_id=0; core_id < cores_per_package; core_id++) { + for (cpu_id = 0; cpu_id < numcpus; cpu_id++) { + for (core_id = 0; core_id < cores_per_package; core_id++) { if (core_id > 0) { pcontrol_blk = 0; plen = 0; diff --git a/src/soc/intel/common/block/cse/cse.c b/src/soc/intel/common/block/cse/cse.c index 77cedb1407..3808bd677e 100644 --- a/src/soc/intel/common/block/cse/cse.c +++ b/src/soc/intel/common/block/cse/cse.c @@ -323,7 +323,7 @@ heci_send(const void *msg, size_t len, uint8_t host_addr, uint8_t client_addr) hdr |= client_addr << MEI_HDR_CSE_ADDR_START; hdr |= host_addr << MEI_HDR_HOST_ADDR_START; hdr |= (MIN(max_length, remaining) == remaining) ? - MEI_HDR_IS_COMPLETE: 0; + MEI_HDR_IS_COMPLETE : 0; sent = send_one_message(hdr, p); p += sent; remaining -= sent; diff --git a/src/soc/intel/common/hda_verb.c b/src/soc/intel/common/hda_verb.c index 6e27e1188e..58732e5eff 100644 --- a/src/soc/intel/common/hda_verb.c +++ b/src/soc/intel/common/hda_verb.c @@ -172,9 +172,9 @@ static int hda_wait_for_valid(u8 *base) */ static u32 hda_find_verb(u32 verb_table_bytes, const u32 *verb_table_data, - u32 viddid, const u32 ** verb) + u32 viddid, const u32 **verb) { - int idx=0; + int idx = 0; while (idx < (verb_table_bytes / sizeof(u32))) { u32 verb_size = 4 * verb_table_data[idx+2]; // in u32 diff --git a/src/soc/intel/common/lpss_i2c.h b/src/soc/intel/common/lpss_i2c.h index 956697356d..19cf6ce299 100644 --- a/src/soc/intel/common/lpss_i2c.h +++ b/src/soc/intel/common/lpss_i2c.h @@ -65,12 +65,12 @@ struct lpss_i2c_bus_config { struct lpss_i2c_speed_config speed_config[LPSS_I2C_SPEED_CONFIG_COUNT]; }; -#define LPSS_I2C_SPEED_CONFIG(speedval,lcnt,hcnt,hold) \ - { \ - .speed = I2C_SPEED_ ## speedval, \ - .scl_lcnt = (lcnt), \ - .scl_hcnt = (hcnt), \ - .sda_hold = (hold), \ +#define LPSS_I2C_SPEED_CONFIG(speedval, lcnt, hcnt, hold) \ + { \ + .speed = I2C_SPEED_ ## speedval, \ + .scl_lcnt = (lcnt), \ + .scl_hcnt = (hcnt), \ + .sda_hold = (hold), \ } /* diff --git a/src/soc/intel/common/smihandler.c b/src/soc/intel/common/smihandler.c index 95e7c7639a..da91c4516c 100644 --- a/src/soc/intel/common/smihandler.c +++ b/src/soc/intel/common/smihandler.c @@ -388,21 +388,21 @@ static uint64_t em64t100_smm_save_state_get_reg(void *state, enum smm_reg reg) uintptr_t value = 0; em64t100_smm_state_save_area_t *smm_state = state; - switch(reg) { - case RAX: - value = smm_state->rax; - break; - case RBX: - value = smm_state->rbx; - break; - case RCX: - value = smm_state->rcx; - break; - case RDX: - value = smm_state->rdx; - break; - default: - break; + switch (reg) { + case RAX: + value = smm_state->rax; + break; + case RBX: + value = smm_state->rbx; + break; + case RCX: + value = smm_state->rcx; + break; + case RDX: + value = smm_state->rdx; + break; + default: + break; } return value; } @@ -410,7 +410,7 @@ static uint64_t em64t100_smm_save_state_get_reg(void *state, enum smm_reg reg) static void em64t100_smm_save_state_set_reg(void *state, enum smm_reg reg, uint64_t val) { em64t100_smm_state_save_area_t *smm_state = state; - switch(reg) { + switch (reg) { case RAX: smm_state->rax = val; break; @@ -439,21 +439,21 @@ static uint64_t em64t101_smm_save_state_get_reg(void *state, enum smm_reg reg) uintptr_t value = 0; em64t101_smm_state_save_area_t *smm_state = state; - switch(reg) { - case RAX: - value = smm_state->rax; - break; - case RBX: - value = smm_state->rbx; - break; - case RCX: - value = smm_state->rcx; - break; - case RDX: - value = smm_state->rdx; - break; - default: - break; + switch (reg) { + case RAX: + value = smm_state->rax; + break; + case RBX: + value = smm_state->rbx; + break; + case RCX: + value = smm_state->rcx; + break; + case RDX: + value = smm_state->rdx; + break; + default: + break; } return value; } @@ -461,7 +461,7 @@ static uint64_t em64t101_smm_save_state_get_reg(void *state, enum smm_reg reg) static void em64t101_smm_save_state_set_reg(void *state, enum smm_reg reg, uint64_t val) { em64t101_smm_state_save_area_t *smm_state = state; - switch(reg) { + switch (reg) { case RAX: smm_state->rax = val; break; |