Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29668 )
Change subject: nb/intel/sandybridge/acpi: Add RMRR entry for iGPU ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/#/c/29668/7/src/northbridge/intel/sandybridge/ac... File src/northbridge/intel/sandybridge/acpi.c:
https://review.coreboot.org/#/c/29668/7/src/northbridge/intel/sandybridge/ac... PS7, Line 101: current += acpi_create_igfx_rmrr(current); what happens on error if acpi_create_igfx_rmrr returns 0 ? Does acpi_dmar_rmrr_fixup handle that properly ?