build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43195 )
Change subject: soc/intel/baytrail/lpss.c: Align with Braswell ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/43195/1/src/soc/intel/baytrail/lpss... File src/soc/intel/baytrail/lpss.c:
https://review.coreboot.org/c/coreboot/+/43195/1/src/soc/intel/baytrail/lpss... PS1, Line 65: #define SET_IOSF_REG(name_) \ Macros with complex values should be enclosed in parentheses
https://review.coreboot.org/c/coreboot/+/43195/1/src/soc/intel/baytrail/lpss... PS1, Line 109: #define CASE_I2C(name_) case PCI_DEVFN(name_ ## _DEV, name_ ## _FUNC) Macros with complex values should be enclosed in parentheses