build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35031 )
Change subject: soc/intel/skylake: Add Lewisburg PCH GPIOs support ......................................................................
Patch Set 5:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35031/5/src/soc/intel/skylake/gpio.... File src/soc/intel/skylake/gpio.c:
https://review.coreboot.org/c/coreboot/+/35031/5/src/soc/intel/skylake/gpio.... PS5, Line 240: #else braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/35031/5/src/soc/intel/skylake/gpio.... PS5, Line 243: # if CONFIG(SKYLAKE_SOC_PCH_H) braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/35031/5/src/soc/intel/skylake/inclu... File src/soc/intel/skylake/include/soc/gpio_defs.h:
https://review.coreboot.org/c/coreboot/+/35031/5/src/soc/intel/skylake/inclu... PS5, Line 45: #define NUM_GPI_STATUS_REGS \ Macros with complex values should be enclosed in parentheses