[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Fix DMAR when no iGPU is present

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43994 )

Change subject: soc/intel/cannonlake: Fix DMAR when no iGPU is present
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43994/2/src/soc/intel/cannonlake/ac... 
File src/soc/intel/cannonlake/acpi.c:

https://review.coreboot.org/c/coreboot/+/43994/2/src/soc/intel/cannonlake/ac... 
PS2, Line 297: 		/* Add RMRR entry */
> Wait... Apparently VT-d spec says that RMRRs need to be added after all DRHDs have been added. […]

You are right. Checked Document D51397-011.
Then the following platforms are wrong: APL, SDY, BDW, SKL