diff options
author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-07 15:17:04 -0800 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-03-12 15:47:06 +0100 |
commit | bfdb8937b27700546bc233287c657e2f634a3177 (patch) | |
tree | b15370717de3a6f0f578cff1d575f0b53383cd9b /src/include/cpu/amd | |
parent | 6625ecc3448b59b6adde3f10674cd578f06f786f (diff) | |
download | coreboot-bfdb8937b27700546bc233287c657e2f634a3177.tar.xz |
src/include: Remove braces for single statements
Fix the following warning detected by checkpatch.pl:
WARNING: braces {} are not necessary for single statement blocks
TEST=Build and run on Galileo Gen2
Change-Id: I00b59f6a27c3acb393deaa763596363b7e958efd
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18654
Tested-by: build bot (Jenkins)
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/include/cpu/amd')
-rw-r--r-- | src/include/cpu/amd/model_fxx_rev.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/include/cpu/amd/model_fxx_rev.h b/src/include/cpu/amd/model_fxx_rev.h index e34b4ed649..210a65227e 100644 --- a/src/include/cpu/amd/model_fxx_rev.h +++ b/src/include/cpu/amd/model_fxx_rev.h @@ -74,9 +74,8 @@ static inline int is_e0_later_in_bsp(int nodeid) return 1; // we don't need to do that for node 0 in core0/node0 - if (nodeid == 0) { + if (nodeid == 0) return !is_cpu_pre_e0(); - } // d0 will be treated as e0 with this methods, but the d0 nb_cfg_54 always 0 pci_devfn_t dev; @@ -89,9 +88,8 @@ static inline int is_e0_later_in_bsp(int nodeid) e0_later = !!(val & (1<<3)); // pre_e0 bit 3 always be 0 and can not be changed - if (e0_later) { + if (e0_later) pci_write_config32(dev, 0x80, val_old); // restore it - } return e0_later; } |