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

[Paul Menzel (Code Review)]

Paul Menzel 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)

What to do? Add or remove braces?

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;
> Well, it uses multiple lines, so using braces makes the code clearer (even if only slightly). […]

Raul, could catch, I didn’t see that. I’ll update the commit message.

PS: Patrick once proposed on the mailing list to require braces even around single line statements.