[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/skylake: Add PCI Id for Kabylake DT

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32850 )

Change subject: soc/intel/skylake: Add PCI Id for Kabylake DT
......................................................................

Patch Set 10:

(1 comment)
> Patch Set 10:
> > Patch Set 9:
> >
> > What is DT? If it's supposed to mean Desktop, wouldn't that be Kabylake-S?
>
> Kabylake DT is an "entry level server" variant.

Thanks.

https://review.coreboot.org/#/c/32850/10/src/include/device/pci_ids.h 
File src/include/device/pci_ids.h:

https://review.coreboot.org/#/c/32850/10/src/include/device/pci_ids.h@3085 
PS10, Line 3085: #define PCI_DEVICE_ID_INTEL_KBL_ID_DT	0x591f
Now that I learned about DT, this 0x591f one is documented as S/X in
doc 567957.

Again, I guess we could have better lists if Intel would publish the
stepping and id guides.