Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45562 )
Change subject: nb/intel/ironlake: Add more host bridge PCI IDs ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/45562/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/45562/2//COMMIT_MSG@18 PS2, Line 18: Maybe mention that only devices with broken ME firmware are affected? At least that is how I understood it.
https://review.coreboot.org/c/coreboot/+/45562/2/src/northbridge/intel/ironl... File src/northbridge/intel/ironlake/northbridge.c:
https://review.coreboot.org/c/coreboot/+/45562/2/src/northbridge/intel/ironl... PS2, Line 218: 0x0040, /* Clarkdale */ Why add Clarkdale? it's not supported by coreboot, or is it?