build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39663 )
Change subject: [WIP] soc/intel/braswell: Clean up ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39663/1/src/soc/intel/braswell/lpss... File src/soc/intel/braswell/lpss.c:
https://review.coreboot.org/c/coreboot/+/39663/1/src/soc/intel/braswell/lpss... PS1, Line 111: #define CASE_I2C(name_) case PCI_DEVFN(name_ ## _DEV, name_ ## _FUNC) Macros with complex values should be enclosed in parentheses