Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34137 )
Change subject: soc/intel/common: Add function to check if PCI device is a LPSS controller ......................................................................
Patch Set 4:
(3 comments)
https://review.coreboot.org/c/coreboot/+/34137/4/src/soc/intel/common/block/... File src/soc/intel/common/block/include/intelblocks/lpss.h:
https://review.coreboot.org/c/coreboot/+/34137/4/src/soc/intel/common/block/... PS4, Line 35: soc nit: All the soc specific functions have the format with soc_*. Prefer to maintain that format. Again my personal opinion.
https://review.coreboot.org/c/coreboot/+/34137/4/src/soc/intel/common/block/... File src/soc/intel/common/block/lpss/lpss.c:
https://review.coreboot.org/c/coreboot/+/34137/4/src/soc/intel/common/block/... PS4, Line 83: static const int *lpss_devices; Please leave a blank line between declaration block and function body.
https://review.coreboot.org/c/coreboot/+/34137/4/src/soc/intel/common/block/... PS4, Line 87: dev->path.pci.devfn It is also better to ensure that the device path is a PCI type in addition to checking the devfn.