Paul Menzel 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 8:
(4 comments)
https://review.coreboot.org/c/coreboot/+/43804/8//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/43804/8//COMMIT_MSG@7 PS8, Line 7: /soc/amd/acpi
soc/amd/acpi: …
https://review.coreboot.org/c/coreboot/+/43804/8//COMMIT_MSG@11 PS8, Line 11: binary. Please do not wrap lines after sentences, if it’s not the end of a paragraph. Paragraphs should be separate by a blank line.
https://review.coreboot.org/c/coreboot/+/43804/8//COMMIT_MSG@12 PS8, Line 12: How much time does this add to the total boot time?
https://review.coreboot.org/c/coreboot/+/43804/8//COMMIT_MSG@14 PS8, Line 14: TEST=Boot trembyle to shell and extract and compare IVRS tables Are they the same?