[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl,icl}: Add provision to use PIRX PCI IRQ acpi package

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34710 )

Change subject: soc/intel/{cnl,icl}: Add provision to use PIRX PCI IRQ acpi package
......................................................................

Patch Set 4: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/34710/4/src/soc/intel/icelake/acpi/... 
File src/soc/intel/icelake/acpi/pci_irqs.asl:

https://review.coreboot.org/c/coreboot/+/34710/4/src/soc/intel/icelake/acpi/... 
PS4, Line 25: External(PIRX)
> So if you change the scope to _SB.PCI0, I think this declaration needs to include that as well.

Done