build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45434 )
Change subject: nb/intel/gm45/gm45.h: Clean up cosmetics ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/c/coreboot/+/45434/3/src/northbridge/intel/gm45/... File src/northbridge/intel/gm45/gm45.h:
https://review.coreboot.org/c/coreboot/+/45434/3/src/northbridge/intel/gm45/... PS3, Line 234: #define MCHBAR8(x) *((volatile u8 *)(DEFAULT_MCHBAR + x)) Macros with complex values should be enclosed in parentheses
https://review.coreboot.org/c/coreboot/+/45434/3/src/northbridge/intel/gm45/... PS3, Line 358: #define DMIBAR8(x) *((volatile u8 *)(DEFAULT_DMIBAR + x)) Macros with complex values should be enclosed in parentheses
https://review.coreboot.org/c/coreboot/+/45434/3/src/northbridge/intel/gm45/... PS3, Line 375: #define EPBAR8(x) *((volatile u8 *)(DEFAULT_EPBAR + x)) Macros with complex values should be enclosed in parentheses