summaryrefslogtreecommitdiff
path: root/src/arch/x86/acpigen.c
diff options
context:
space:
mode:
authorLee Leahy <leroy.p.leahy@intel.com>2017-03-16 17:49:42 -0700
committerLee Leahy <leroy.p.leahy@intel.com>2017-03-17 02:36:36 +0100
commitd94cff6ab26d482554309041a9317cc3bf5e4b02 (patch)
treea09902883736f6eca89e96e53642ce944ff0401b /src/arch/x86/acpigen.c
parent1072e7dcc30895f850143450504b78a4135978b6 (diff)
downloadcoreboot-d94cff6ab26d482554309041a9317cc3bf5e4b02.tar.xz
soc/intel/braswell: Fix most of the issues detected by checkpatch
Fix the following errors and warnings detected by checkpatch.pl: ERROR: that open brace { should be on the previous line ERROR: return is not a function, parentheses are not required WARNING: braces {} are not necessary for any arm of this statement WARNING: line over 80 characters WARNING: braces {} are not necessary for single statement blocks WARNING: Avoid unnecessary line continuations WARNING: break is not useful after a goto or return WARNING: else is not generally useful after a break or return False positives are generated by checkpatch for the following test: ERROR: Macros with complex values should be enclosed in parentheses TEST=Build for cyan Change-Id: I19048895145b138a63100b29f829ff446ff71b58 Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com> Reviewed-on: https://review.coreboot.org/18871 Reviewed-by: Duncan Laurie <dlaurie@chromium.org> Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/arch/x86/acpigen.c')
0 files changed, 0 insertions, 0 deletions