Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34658 )
Change subject: soc/intel/common/block/acpi: Add provision to inject PCI PIRQ ACPI mapping ......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/c/coreboot/+/34658/5/src/soc/intel/common/block/... File src/soc/intel/common/block/acpi/acpi.c:
https://review.coreboot.org/c/coreboot/+/34658/5/src/soc/intel/common/block/... PS5, Line 242: : acpigen_write_scope("\"); : acpigen_write_name("PIRX"); : acpigen_write_package(num_entries);
Seems like this could be better-scoped; maybe the same scope that _PRT is in?
I tried to scope it SB.PCI0, where the current PICN abd PICM packages are present, and had provided external reference for the same.(patch set 1)
However the kernel could not locate the PIRX package and hanged. Unless I set the scope to '\' same as GNVS