Attention is currently required from: Saurabh Mishra, Subrata Banik.
Angel Pons has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/83506?usp=email )
Change subject: device/pci_ids: Add new Intel PTL device IDs for eSPI/LPC ......................................................................
Patch Set 1:
(1 comment)
File src/soc/intel/common/block/lpc/lpc.c:
https://review.coreboot.org/c/coreboot/+/83506/comment/fa01d886_219ce89a?usp... : PS1, Line 143: static const unsigned short pci_device_ids[] = {
For another patch: bind the LPC/eSPI ops through the chipset devicetree instead of having such a l […]
Yeah, we'd still need the IDs there. Fortunately, `report_platform.c` is SoC-specific, so the lists are not as long. It's also non-critical to function, so it would only impact the log output (which most non-developers won't see anyway).
The main idea is to avoid having to potentially iterate over 300 IDs (worst-case) to try binding the ops to a device we always know is at PCI B:D.F `00:1f.0`.