diff options
| author | Paul Menzel <pmenzel@molgen.mpg.de> | 2020-06-08 17:01:41 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2020-06-10 18:35:54 +0000 |
| commit | 24b642e72636346f3bfd196b9bcb84f7895d9aff (patch) | |
| tree | 081ef808af948fff5810ad54ba7a3245d1a15909 | |
| parent | 0f1394abd687410ecccdb0d8a7c0f39dafd5594f (diff) | |
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>
| -rw-r--r-- | src/soc/intel/cannonlake/acpi.c | 3 |
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 */ |