Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35985 )
Change subject: intel/skylake: Implement PCIe RP devicetree update based on LCAP ......................................................................
Patch Set 17:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35985/17/src/soc/intel/common/block... File src/soc/intel/common/block/pcie/pcie_rp.c:
https://review.coreboot.org/c/coreboot/+/35985/17/src/soc/intel/common/block... PS17, Line 102: dev->enabled = 0; I thought about unlinking these nodes from the bus. And now I'm convinced that this is even necessary (if we don't make assumptions about the reordering) because we don't know a new fn that we could assign.