coreboot/src/arch
Lee Leahy 0b5678f21f arch/x86: Fix most of remaining issues detected by checkpatch
Fix the following errors and warnings detected by checkpatch.pl:

ERROR: do not use assignment in if condition
ERROR: trailing statements should be on next line
ERROR: Macros with complex values should be enclosed in parentheses
ERROR: switch and case should be at the same indent
WARNING: char * array declaration might be better as static const
WARNING: else is not generally useful after a break or return
WARNING: storage class should be at the beginning of the declaration
WARNING: void function return statements are not generally useful
WARNING: break is not useful after a goto or return
WARNING: Single statement macros should not use a do {} while (0) loop
WARNING: sizeof *t should be sizeof(*t)
WARNING: Comparisons should place the constant on the right side of the test

TEST=Build and run on Galileo Gen2

Change-Id: I39d49790c5eaeedec5051e1fab0b1279275f6e7f
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18865
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
2017-03-20 16:36:24 +01:00
..
arm buildsystem: Drop explicit (k)config.h includes 2016-12-08 19:46:53 +01:00
arm64 arm-trusted-firmware: Disable a couple of warnings for GCC 6.2 2017-02-24 03:27:41 +01:00
mips build system: remove CBFSTOOL_PRE1_OPTS 2016-05-03 11:40:49 +02:00
power8 region: Add writeat and eraseat support 2016-06-24 20:48:12 +02:00
riscv riscv: Suppress invalid coverity errors 2017-02-20 04:40:01 +01:00
x86 arch/x86: Fix most of remaining issues detected by checkpatch 2017-03-20 16:36:24 +01:00