Nico Huber has posted comments on this change. ( https://review.coreboot.org/26232 )
Change subject: chipset_enable: Add PCI IDs for discrete Kaby Lake PCHs ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/#/c/26232/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/26232/1//COMMIT_MSG@21 PS1, Line 21: Revision 003
It seems you were looking at the same version of spec as me. […]
No, you were right initially. Device IDs are listed in volume 1 and that's where I was looking at revision 002 ;)
https://review.coreboot.org/#/c/26232/1/chipset_enable.c File chipset_enable.c:
https://review.coreboot.org/#/c/26232/1/chipset_enable.c@1952 PS1, Line 1952: {0x8086, 0xa2c8, NT, "Intel", "B250", enable_flash_pch100},
Revision 003 of the spec also mentions 0xa2c9 for the Z370.
Done