build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39713 )
Change subject: soc/intel/xeon_sp: Add basic Cooperlake-SP support ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39713/3/src/soc/intel/xeon_sp/cpx/c... File src/soc/intel/xeon_sp/cpx/chip.c:
https://review.coreboot.org/c/coreboot/+/39713/3/src/soc/intel/xeon_sp/cpx/c... PS3, Line 57: if (dev->path.type == DEVICE_PATH_DOMAIN) { braces {} are not necessary for any arm of this statement