[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Update Jasperlake Device IDs

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38849 )

Change subject: soc/intel/common: Update Jasperlake Device IDs
......................................................................

Patch Set 9:

(3 comments)

https://review.coreboot.org/c/coreboot/+/38849/6//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/38849/6//COMMIT_MSG@7 
PS6, Line 7: Jasperlake
> Jasper Lake

Please address this comment as well.

https://review.coreboot.org/c/coreboot/+/38849/5/src/include/device/pci_ids.... 
File src/include/device/pci_ids.h:

https://review.coreboot.org/c/coreboot/+/38849/5/src/include/device/pci_ids.... 
PS5, Line 3394: JSL
> The CPU core IPs use JSL and the PCH IPs use JSP. But thanks for the finding. […]

Done

https://review.coreboot.org/c/coreboot/+/38849/5/src/include/device/pci_ids.... 
PS5, Line 3453: #define PCI_DEVICE_ID_INTEL_JSL_EHL		0x4532
              : #define PCI_DEVICE_ID_INTEL_EHL_ID_1		0x4510
> Yes, these IDs have got nothing to do with JasperLake. They are EHL specific.

Done