[coreboot-gerrit] Change in ...coreboot[master]: inteltool: Add multiple device IDs of Intel GPUs

[Nico Huber (Code Review)]

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

Change subject: inteltool: Add multiple device IDs of Intel GPUs
......................................................................

Patch Set 12: Code-Review+2

(1 comment)

I'm curious: where is your list from?

https://review.coreboot.org/#/c/30610/12/util/inteltool/inteltool.h 
File util/inteltool/inteltool.h:

https://review.coreboot.org/#/c/30610/12/util/inteltool/inteltool.h@328 
PS12, Line 328: #define PCI_DEVICE_ID_INTEL_IRIS_PLUS_655	0x3EA5
If half of them are UHD 630, maybe we shouldn't use
the marketing names? The most elaborate list I know
is in the Linux sources (include/drm/i915_pciids.h),
with a little more distinguished names (less dis-
tinguished in the Iris case, though, *sigh*).

NB. most of this file seems to use lower-case hex