diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-16 17:49:42 -0700 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-17 02:36:36 +0100 |
| commit | d94cff6ab26d482554309041a9317cc3bf5e4b02 (patch) | |
| tree | a09902883736f6eca89e96e53642ce944ff0401b /src/soc/intel/braswell/acpi.c | |
| parent | 1072e7dcc30895f850143450504b78a4135978b6 (diff) | |
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/soc/intel/braswell/acpi.c')
| -rw-r--r-- | src/soc/intel/braswell/acpi.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/soc/intel/braswell/acpi.c b/src/soc/intel/braswell/acpi.c index 163d7a9eea..0acb90ce7b 100644 --- a/src/soc/intel/braswell/acpi.c +++ b/src/soc/intel/braswell/acpi.c @@ -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); |