Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44741 )
Change subject: soc/intel/cnl: Allow enabling/disabling PEG devices ......................................................................
Patch Set 14:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44741/13/src/soc/intel/cannonlake/i... File src/soc/intel/cannonlake/include/soc/pci_devs.h:
https://review.coreboot.org/c/coreboot/+/44741/13/src/soc/intel/cannonlake/i... PS13, Line 29: #define SA_DEVFN_PEG3 PCI_DEVFN(SA_DEV_SLOT_PEG, 3) I wonder what this is? datasheet doesn't seem to mention it.
https://review.coreboot.org/c/coreboot/+/44741/13/src/soc/intel/cannonlake/r... File src/soc/intel/cannonlake/romstage/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/44741/13/src/soc/intel/cannonlake/r... PS13, Line 146: m_cfg->Peg3Enable = is_dev_enabled(dev); Is there any SKU that could make use of this?