[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Enable VT-d and generate DMAR ACPI table

[caveh jalali (Code Review)]

caveh jalali has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38165 )

Change subject: soc/intel/tigerlake: Enable VT-d and generate DMAR ACPI table
......................................................................

Patch Set 10:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38165/10/src/soc/intel/tigerlake/ac... 
File src/soc/intel/tigerlake/acpi.c:

https://review.coreboot.org/c/coreboot/+/38165/10/src/soc/intel/tigerlake/ac... 
PS10, Line 198: struct device *const
did you mean
const struct device *?

const struct device *const
would also be OK.

same with ipu_dev.

https://review.coreboot.org/c/coreboot/+/38165/10/src/soc/intel/tigerlake/ac... 
PS10, Line 243: 4
do we have a constant for the number of TCSS ports?
maybe add to tigerlake/chip.h?