build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43804 )
Change subject: /soc/amd/acpi Move ACPI IVRS generation to coreboot ......................................................................
Patch Set 7:
(2 comments)
https://review.coreboot.org/c/coreboot/+/43804/7/src/soc/amd/picasso/agesa_a... File src/soc/amd/picasso/agesa_acpi.c:
https://review.coreboot.org/c/coreboot/+/43804/7/src/soc/amd/picasso/agesa_a... PS7, Line 426: >> IOMMU_FEATURE_MSI_NUM_PPR_SHIFT:0) ; space prohibited before semicolon
https://review.coreboot.org/c/coreboot/+/43804/7/src/soc/amd/picasso/agesa_a... PS7, Line 451: if (pci_read_config32(iommu_dev, ivrs->ivhd.capability_offset) & EFR_FEATURE_SUP) line over 96 characters