build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34157 )
Change subject: soc/intel/cannonlake: Add device Ids for new CFL SKUs support ......................................................................
Patch Set 12:
(4 comments)
https://review.coreboot.org/c/coreboot/+/34157/12/src/include/device/pci_ids... File src/include/device/pci_ids.h:
https://review.coreboot.org/c/coreboot/+/34157/12/src/include/device/pci_ids... PS12, Line 3095: #define PCI_DEVICE_ID_INTEL_SKL_ID_H_2 0x1900 please, no space before tabs
https://review.coreboot.org/c/coreboot/+/34157/12/src/include/device/pci_ids... PS12, Line 3097: #define PCI_DEVICE_ID_INTEL_SKL_ID_S_2 0x190f please, no space before tabs
https://review.coreboot.org/c/coreboot/+/34157/12/src/include/device/pci_ids... PS12, Line 3098: #define PCI_DEVICE_ID_INTEL_SKL_ID_S_4 0x191f please, no space before tabs
https://review.coreboot.org/c/coreboot/+/34157/12/src/include/device/pci_ids... PS12, Line 3106: #define PCI_DEVICE_ID_INTEL_KBL_ID_DT_2 0x5918 please, no space before tabs