[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 6:

(2 comments)

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
All the IDs use JSP and this ID uses JSL. Can you please keep it consistent with other IDs?

Also while you are at it, do you need to re-index GT1 to GT0 & GT2 to GT1. I am not an expert here. It is your call.

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
> Done

The confusion happened because of JSL_EHL. So I thought it has some relation with Jasper Lake. Can you please confirm that ID has got nothing to do with Jasper Lake.