Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43657 )
Change subject: soc/intel/tigerlake: Disable VT-d and no DMAR table for ES platform ......................................................................
Patch Set 2:
(5 comments)
https://review.coreboot.org/c/coreboot/+/43657/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/43657/2//COMMIT_MSG@9 PS2, Line 9: pre-QS silicon Please mention ES platform somewhere. Maybe:
pre-QS silicon (ES platform)
https://review.coreboot.org/c/coreboot/+/43657/2//COMMIT_MSG@9 PS2, Line 9: Enabling VT-d on pre-QS silicon may have issues like rendering the : Thunderbolt driver useless. Please reference the datasheet/errata, where this is documented.
https://review.coreboot.org/c/coreboot/+/43657/2//COMMIT_MSG@14 PS2, Line 14: ES(cpu:0x806c0) Please add a space before the (.
https://review.coreboot.org/c/coreboot/+/43657/2//COMMIT_MSG@15 PS2, Line 15: QS(cpu:0x806c1) Please add a space before the (.
https://review.coreboot.org/c/coreboot/+/43657/2/src/soc/intel/tigerlake/rom... File src/soc/intel/tigerlake/romstage/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/43657/2/src/soc/intel/tigerlake/rom... PS2, Line 188: Vt-D VT-d