Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/29509 )
Change subject: soc/intel/icelake: Add PCIE ASL entry ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/#/c/29509/3/src/soc/intel/icelake/acpi/pcie.asl File src/soc/intel/icelake/acpi/pcie.asl:
https://review.coreboot.org/#/c/29509/3/src/soc/intel/icelake/acpi/pcie.asl@... PS3, Line 27: Package () { 0x0000ffff, 0, 0, 11 }, : Package () { 0x0000ffff, 1, 0, 10 }, : Package () { 0x0000ffff, 2, 0, 11 }, : Package () { 0x0000ffff, 3, 0, 11 } }) you are mapping multiple pins to the same link? Could you explain it in the commit message as the commit message does not seem to match at all what you are doing...