Paul Menzel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/42200 )
Change subject: soc/intel/cannonlake: Put braces around *else* branch ......................................................................
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 --- M src/soc/intel/cannonlake/acpi.c 1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/42200/1
diff --git a/src/soc/intel/cannonlake/acpi.c b/src/soc/intel/cannonlake/acpi.c index 886cba2..1c25f1a 100644 --- a/src/soc/intel/cannonlake/acpi.c +++ b/src/soc/intel/cannonlake/acpi.c @@ -203,8 +203,9 @@ 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 */