From dfc8a560d7fec715a652f428dc3cab3e0a92c680 Mon Sep 17 00:00:00 2001 From: Lee Leahy Date: Wed, 15 Mar 2017 16:47:33 -0700 Subject: cpu/x86: Fix misc. remaining issues detected by checkpatch Fix the following error and warnings detected by checkpatch.pl: ERROR: trailing statements should be on next line WARNING: storage class should be at the beginning of the declaration WARNING: type 'long unsigned int' should be specified in [[un]signed] [short|int|long|long long] order TEST=Build and run on Galileo Gen2 Change-Id: I9cfe42cf1836cfd40ffcf67237c818543f508feb Signed-off-by: Lee Leahy Reviewed-on: https://review.coreboot.org/18845 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/cpu/x86/lapic/lapic_cpu_init.c | 7 ++++--- src/cpu/x86/smm/smm_module_handler.c | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c index 262c6d4fb5..427e537ce6 100644 --- a/src/cpu/x86/lapic/lapic_cpu_init.c +++ b/src/cpu/x86/lapic/lapic_cpu_init.c @@ -95,7 +95,7 @@ static void copy_secondary_start_to_lowest_1M(void) (unsigned char *)_secondary_start, code_size); printk(BIOS_DEBUG, "start_eip=0x%08lx, code_size=0x%08lx\n", - (long unsigned int)AP_SIPI_VECTOR, code_size); + (unsigned long int)AP_SIPI_VECTOR, code_size); } static void recover_lowest_1M(void) @@ -408,7 +408,7 @@ void stop_this_cpu(void) #endif /* C entry point of secondary cpus */ -void asmlinkage secondary_cpu_init(unsigned int index) +asmlinkage void secondary_cpu_init(unsigned int index) { atomic_inc(&active_cpus); @@ -490,7 +490,8 @@ static void smm_other_cpus(struct bus *cpu_bus, device_t bsp_cpu) cpu->path.apic.apic_id); /* FIXME: endless loop */ - while (atomic_read(&active_cpus) != pre_count); + while (atomic_read(&active_cpus) != pre_count) + ; } } diff --git a/src/cpu/x86/smm/smm_module_handler.c b/src/cpu/x86/smm/smm_module_handler.c index 98ed1848a1..4bcd853b9a 100644 --- a/src/cpu/x86/smm/smm_module_handler.c +++ b/src/cpu/x86/smm/smm_module_handler.c @@ -116,7 +116,7 @@ void *smm_get_save_state(int cpu) return base; } -void asmlinkage smm_handler_start(void *arg) +asmlinkage void smm_handler_start(void *arg) { const struct smm_module_params *p; const struct smm_runtime *runtime; -- cgit v1.2.3