diff options
author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-16 17:08:03 -0700 |
---|---|---|
committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-17 02:35:27 +0100 |
commit | f4c4ab9826b521666d5ba887e69a636428ac9eb2 (patch) | |
tree | 2dfe4a3e54deb52916fef93bdf6b2cb6c9090efb /src/soc/intel/skylake/bootblock | |
parent | b439a929392ba54dee43455f6e164b884cb8c308 (diff) | |
download | coreboot-f4c4ab9826b521666d5ba887e69a636428ac9eb2.tar.xz |
soc/intel/skylake: Fix remaining issues detected by checkpatch
Fix the following errors and warnings detected by checkpatch.pl:
ERROR: code indent should use tabs where possible
ERROR: Macros with complex values should be enclosed in parentheses
ERROR: "foo * bar" should be "foo *bar"
ERROR: space required before the open parenthesis '('
ERROR: spaces required around that '=' (ctx:VxW)
WARNING: space prohibited between function name and open parenthesis '('
WARNING: storage class should be at the beginning of the declaration
WARNING: char * array declaration might be better as static const
WARNING: please, no space before tabs
WARNING: braces {} are not necessary for single statement blocks
WARNING: else is not generally useful after a break or return
WARNING: static const char * array should probably be static const char * const
TEST=Build for glados
Change-Id: Ic14ca3abd193cfe257504a55ab6b74782b26bf6d
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18868
Tested-by: build bot (Jenkins)
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Diffstat (limited to 'src/soc/intel/skylake/bootblock')
-rw-r--r-- | src/soc/intel/skylake/bootblock/bootblock.c | 2 | ||||
-rw-r--r-- | src/soc/intel/skylake/bootblock/pch.c | 2 | ||||
-rw-r--r-- | src/soc/intel/skylake/bootblock/report_platform.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/src/soc/intel/skylake/bootblock/bootblock.c b/src/soc/intel/skylake/bootblock/bootblock.c index 93a031f0f5..c11d3d2152 100644 --- a/src/soc/intel/skylake/bootblock/bootblock.c +++ b/src/soc/intel/skylake/bootblock/bootblock.c @@ -16,7 +16,7 @@ #include <bootblock_common.h> #include <soc/bootblock.h> -void asmlinkage bootblock_c_entry(uint64_t base_timestamp) +asmlinkage void bootblock_c_entry(uint64_t base_timestamp) { /* Call lib/bootblock.c main */ bootblock_main_with_timestamp(base_timestamp); diff --git a/src/soc/intel/skylake/bootblock/pch.c b/src/soc/intel/skylake/bootblock/pch.c index 07beae8f9e..62054f14ac 100644 --- a/src/soc/intel/skylake/bootblock/pch.c +++ b/src/soc/intel/skylake/bootblock/pch.c @@ -123,7 +123,7 @@ static void pch_interrupt_init(void) dev = dev_find_slot(0, PCI_DEVFN(PCH_DEV_SLOT_LPC, 0)); if (!dev || !dev->chip_info) - return; + return; config = dev->chip_info; pch_interrupt_routing[0] = config->pirqa_routing; diff --git a/src/soc/intel/skylake/bootblock/report_platform.c b/src/soc/intel/skylake/bootblock/report_platform.c index 12c283fda3..63cd6367ab 100644 --- a/src/soc/intel/skylake/bootblock/report_platform.c +++ b/src/soc/intel/skylake/bootblock/report_platform.c @@ -92,7 +92,7 @@ static void report_cpu_info(void) char cpu_string[50], *cpu_name = cpu_string; /* 48 bytes are reported */ int vt, txt, aes; msr_t microcode_ver; - const char *mode[] = {"NOT ", ""}; + static const char * const mode[] = {"NOT ", ""}; const char *cpu_type = "Unknown"; index = 0x80000000; |