diff options
author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-16 13:41:11 -0700 |
---|---|---|
committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-17 03:17:39 +0100 |
commit | 024b13d07c6dcef7acefc3b69c16c46be431fb82 (patch) | |
tree | b31fb158c61e7c5609a5f16c39624363aa2048d1 /src/arch/x86/acpi_s3.c | |
parent | 9c7c6f7213decfc0d0fee4bbc911a291ee93bcdb (diff) | |
download | coreboot-024b13d07c6dcef7acefc3b69c16c46be431fb82.tar.xz |
arch/x86: Fix space issues detected by checkpatch
Fix the following errors and warnings detected by checkpatch.pl:
ERROR: space required before the open parenthesis '('
ERROR: space prohibited after that open parenthesis '('
ERROR: space prohibited before that close parenthesis ')'
ERROR: space prohibited after that open square bracket '['
ERROR: space required after that ',' (ctx:VxV)
ERROR: space prohibited before that ',' (ctx:WxW)
ERROR: space required after that ';' (ctx:VxV)
ERROR: spaces required around that ':' (ctx:ExV)
ERROR: spaces required around that ':' (ctx:VxW)
ERROR: spaces required around that ':' (ctx:WxV)
ERROR: spaces required around that '=' (ctx:VxV)
ERROR: spaces required around that '+=' (ctx:VxV)
ERROR: spaces required around that '<=' (ctx:WxV)
ERROR: spaces required around that '||' (ctx:VxW)
ERROR: space prohibited before that '++' (ctx:WxO)
ERROR: need consistent spacing around '+' (ctx:WxV)
ERROR: spaces required around that '<' (ctx:WxV)
ERROR: spaces required around that '<' (ctx:VxV)
ERROR: need consistent spacing around '>>' (ctx:WxV)
ERROR: "(foo*)" should be "(foo *)"
ERROR: "foo* bar" should be "foo *bar"
ERROR: "foo * bar" should be "foo *bar"
ERROR: code indent should use tabs where possible
WARNING: space prohibited between function name and open parenthesis '('
WARNING: unnecessary whitespace before a quoted newline
WARNING: please, no spaces at the start of a line
WARNING: please, no space before tabs
WARNING: Unnecessary space before function pointer arguments
TEST=Build and run on Galileo Gen2
Change-Id: I2d7e1a329c6b2e8ca9633a97b595566544d7fd33
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18862
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/arch/x86/acpi_s3.c')
-rw-r--r-- | src/arch/x86/acpi_s3.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/arch/x86/acpi_s3.c b/src/arch/x86/acpi_s3.c index 0001885db2..ea04029b7b 100644 --- a/src/arch/x86/acpi_s3.c +++ b/src/arch/x86/acpi_s3.c @@ -133,7 +133,7 @@ void *acpi_backup_container(uintptr_t base, size_t size) return NULL; backup_mem->valid = 1; - return (void*)(uintptr_t)backup_mem->cbmem; + return (void *)(uintptr_t)backup_mem->cbmem; } void backup_ramstage_section(uintptr_t base, size_t size) @@ -157,8 +157,8 @@ void backup_ramstage_section(uintptr_t base, size_t size) return; /* Back up the OS-controlled memory where ramstage will be loaded. */ - memcpy((void*)(uintptr_t)backup_mem->cbmem, - (void*)(uintptr_t)backup_mem->lowmem, (size_t)backup_mem->size); + memcpy((void *)(uintptr_t)backup_mem->cbmem, + (void *)(uintptr_t)backup_mem->lowmem, (size_t)backup_mem->size); backup_mem->valid = 1; } @@ -174,8 +174,8 @@ void acpi_prepare_for_resume(void) return; /* Back up the OS-controlled memory where ramstage will be loaded. */ - memcpy((void*)(uintptr_t)backup_mem->cbmem, - (void*)(uintptr_t)backup_mem->lowmem, (size_t)backup_mem->size); + memcpy((void *)(uintptr_t)backup_mem->cbmem, + (void *)(uintptr_t)backup_mem->lowmem, (size_t)backup_mem->size); backup_mem->valid = 1; } @@ -199,7 +199,7 @@ void acpi_prepare_resume_backup(void) #define WAKEUP_BASE 0x600 void (*acpi_do_wakeup)(uintptr_t vector, u32 backup_source, u32 backup_target, - u32 backup_size) asmlinkage = (void *)WAKEUP_BASE; + u32 backup_size) asmlinkage = (void *)WAKEUP_BASE; extern unsigned char __wakeup; extern unsigned int __wakeup_size; |