build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/28956 )
Change subject: src: Standardize PCI_DEV(0, 0x1f, 0) name ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/28956/1/src/soc/intel/fsp_baytrail/include/s... File src/soc/intel/fsp_baytrail/include/soc/lpc.h:
https://review.coreboot.org/#/c/28956/1/src/soc/intel/fsp_baytrail/include/s... PS1, Line 20: #define FADT_LPC_DEV 0, PCI_DEVFN(0x1f, 0) Macros with complex values should be enclosed in parentheses
https://review.coreboot.org/#/c/28956/1/src/southbridge/intel/fsp_rangeley/s... File src/southbridge/intel/fsp_rangeley/soc.h:
https://review.coreboot.org/#/c/28956/1/src/southbridge/intel/fsp_rangeley/s... PS1, Line 96: #define LPC_DEVFN 0, PCI_DEVFN(0x1f, 0) Macros with complex values should be enclosed in parentheses