Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34941 )
Change subject: inteltool: add Lewisburg C62x chipset family PCI IDs ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/34941/4/util/inteltool/inteltool.h File util/inteltool/inteltool.h:
https://review.coreboot.org/c/coreboot/+/34941/4/util/inteltool/inteltool.h@... PS4, Line 173: #define PCI_DEVICE_ID_INTEL_C621 0xa1c1 Add "PROD" here to make the difference clear.
https://review.coreboot.org/c/coreboot/+/34941/4/util/inteltool/inteltool.h@... PS4, Line 182: #define PCI_DEVICE_ID_INTEL_C624_SUPER_SKU 0xa242 Remove "SKU", since this ID is the SKU. No need for extra mentioning it.