[coreboot-gerrit] [XS] Change in coreboot[main]: soc/intel/common/block/acpi/exclude DMI fixed memory range if no DMI

[Paul Menzel (Code Review)]

Attention is currently required from: Cliff Huang, Jérémy Compostella.

Paul Menzel has posted comments on this change by Cliff Huang. ( https://review.coreboot.org/c/coreboot/+/84349?usp=email )

Change subject: soc/intel/common/block/acpi/exclude DMI fixed memory range if no DMI
......................................................................

Patch Set 3:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84349/comment/4ff7d96e_3973e924?usp... :
PS3, Line 7: soc/intel/common/block/acpi/exclude DMI fixed memory range if no DMI
I think the colon is missing to separate the prefix from the summary.

https://review.coreboot.org/c/coreboot/+/84349/comment/e716a49e_176c5b8d?usp... :
PS3, Line 9: exclude DMI in northbridge.asl if DMI_BASE_SIZE is '0'
Why?

https://review.coreboot.org/c/coreboot/+/84349/comment/0ec34e92_d32f8814?usp... :
PS3, Line 9: exclude
Exclude

https://review.coreboot.org/c/coreboot/+/84349/comment/0a7c4210_4b39be59?usp... :
PS3, Line 12: Verified on Intel® Simics® Pre Silicon Simulation platform
            :      for PTL using google/fatcat mainboard.
            : 
How? Was booting not possible before, and now it is?