[coreboot-gerrit] Change in coreboot[master]: soc/intel/*/acpi: Put braces around *else* branch

[Marshall Dawson (Code Review)]

Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42257 )

Change subject: soc/intel/*/acpi: Put braces around *else* branch
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42257/2/src/soc/intel/jasperlake/ac... 
File src/soc/intel/jasperlake/acpi.c:

https://review.coreboot.org/c/coreboot/+/42257/2/src/soc/intel/jasperlake/ac... 
PS2, Line 293: gnvs->chromeos.vbt2 = google_ec_running_ro() ?
             : 				ACTIVE_ECFW_RO : ACTIVE_ECFW_RW;
> Raul, could catch, I didn’t see that. I’ll update the commit message. […]

Since coreboot is still no-braces for single statements, my preference is to remove them, and then indent l.294 an extra tabstop giving an extra visual cue that it's one statement broken over two lines.