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)
This commit is contained in:
@@ -525,12 +525,10 @@ void southcluster_inject_dsdt(device_t device)
|
||||
if (gnvs) {
|
||||
acpi_create_gnvs(gnvs);
|
||||
/* Fill in the Wifi Region id */
|
||||
if (IS_ENABLED(CONFIG_HAVE_REGULATORY_DOMAIN)) {
|
||||
if (IS_ENABLED(CONFIG_HAVE_REGULATORY_DOMAIN))
|
||||
gnvs->cid1 = wifi_regulatory_domain();
|
||||
} else {
|
||||
|
||||
else
|
||||
gnvs->cid1 = WRDD_DEFAULT_REGULATORY_DOMAIN;
|
||||
}
|
||||
acpi_save_gnvs((unsigned long)gnvs);
|
||||
/* And tell SMI about it */
|
||||
smm_setup_structures(gnvs, NULL, NULL);
|
||||
|
Reference in New Issue
Block a user