summaryrefslogtreecommitdiff
path: root/src/soc
diff options
context:
space:
mode:
authorPaul Menzel <pmenzel@molgen.mpg.de>2020-06-08 17:01:41 +0200
committerPatrick Georgi <pgeorgi@google.com>2020-06-10 18:35:54 +0000
commit24b642e72636346f3bfd196b9bcb84f7895d9aff (patch)
tree081ef808af948fff5810ad54ba7a3245d1a15909 /src/soc
parent0f1394abd687410ecccdb0d8a7c0f39dafd5594f (diff)
downloadcoreboot-24b642e72636346f3bfd196b9bcb84f7895d9aff.tar.xz
soc/intel/cannonlake: Put braces around *else* branch
From `Documentation/coding_style.md`: > This does not apply if only one branch of a conditional statement is a > single statement; in the latter case use braces in both branches: Change-Id: I5672949e587a9c0e4efa01521a659e4c224085d0 Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/42200 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
Diffstat (limited to 'src/soc')
-rw-r--r--src/soc/intel/cannonlake/acpi.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/soc/intel/cannonlake/acpi.c b/src/soc/intel/cannonlake/acpi.c
index 12758e1422..c041eb003e 100644
--- a/src/soc/intel/cannonlake/acpi.c
+++ b/src/soc/intel/cannonlake/acpi.c
@@ -203,8 +203,9 @@ void acpi_create_gnvs(struct global_nvs_t *gnvs)
if (CONFIG(EC_GOOGLE_CHROMEEC)) {
gnvs->chromeos.vbt2 = google_ec_running_ro() ?
ACTIVE_ECFW_RO : ACTIVE_ECFW_RW;
- } else
+ } else {
gnvs->chromeos.vbt2 = ACTIVE_ECFW_RO;
+ }
}
/* Enable DPTF based on mainboard configuration */