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 2: Code-Review+1 (3 comments) https://review.coreboot.org/c/coreboot/+/34710/2/src/soc/intel/cannonlake/ac... File src/soc/intel/cannonlake/acpi/pci_irqs.asl: https://review.coreboot.org/c/coreboot/+/34710/2/src/soc/intel/cannonlake/ac... PS2, Line 149: Return nit: space after Return https://review.coreboot.org/c/coreboot/+/34710/2/src/soc/intel/icelake/acpi/... File src/soc/intel/icelake/acpi/pci_irqs.asl: https://review.coreboot.org/c/coreboot/+/34710/2/src/soc/intel/icelake/acpi/... PS2, Line 142: ( nit: space after if https://review.coreboot.org/c/coreboot/+/34710/2/src/soc/intel/icelake/acpi/... PS2, Line 143: Return nit: space after return -- To view, visit https://review.coreboot.org/c/coreboot/+/34710 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib23e57d3ce58406f8faff18e912b78a911e50d3d Gerrit-Change-Number: 34710 Gerrit-PatchSet: 2 Gerrit-Owner: Aamir Bohra <aamir.bohra@intel.com> Gerrit-Reviewer: Aamir Bohra <aamir.bohra@intel.com> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net> Gerrit-Comment-Date: Tue, 13 Aug 2019 13:59:34 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment