build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47308 )
Change subject: mb/google/zork: Init fingerprint GPIOs for boot vs resume ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/47308/2/src/mainboard/google/zork/b... File src/mainboard/google/zork/bootblock.c:
https://review.coreboot.org/c/coreboot/+/47308/2/src/mainboard/google/zork/b... PS2, Line 12: gpios=variant_poweron_gpio_table(&num_gpios, acpi_get_sleep_type()); spaces required around that '=' (ctx:VxV)
https://review.coreboot.org/c/coreboot/+/47308/2/src/mainboard/google/zork/v... File src/mainboard/google/zork/variants/baseboard/gpio_baseboard_trembyle.c:
https://review.coreboot.org/c/coreboot/+/47308/2/src/mainboard/google/zork/v... PS2, Line 317: if (slp_typ != ACPI_S0) { braces {} are not necessary for single statement blocks