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 <Leroy.P.Leahy@intel.com> Reviewed-on: https://review.coreboot.org/18845 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
This commit is contained in:
parent
c5917079eb
commit
dfc8a560d7
|
@ -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)
|
||||
;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -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;
|
||||
|
|
Loading…
Reference in New Issue