Martin Roth has posted comments on this change. ( https://review.coreboot.org/28469 )
Change subject: x86/acpi: Add APEI definitions ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/28469/1/src/arch/x86/include/arch/acpi.h File src/arch/x86/include/arch/acpi.h:
https://review.coreboot.org/#/c/28469/1/src/arch/x86/include/arch/acpi.h@693 PS1, Line 693: /* zero or more Generic Error Data Entries */
The "zero or more" part? That odd phrase came from the spec but I can make it better.
Yeah, maybe it will make more sense as I finish reviewing the rest of the patches. Just seeing it here makes me say why would we have a status block if there were no entries.