Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/28470 )
Change subject: arch/x86: Add BERT region support functions ......................................................................
Patch Set 1:
(2 comments)
I'd meant to relook at the error reporting, i.e. make sure there aren't duplicates.
https://review.coreboot.org/#/c/28470/1/src/arch/x86/acpi_bert_storage.c File src/arch/x86/acpi_bert_storage.c:
https://review.coreboot.org/#/c/28470/1/src/arch/x86/acpi_bert_storage.c@4 PS1, Line 4: 2015 Google Inc. copy/paste fail
https://review.coreboot.org/#/c/28470/1/src/arch/x86/acpi_bert_storage.c@19 PS1, Line 19: todo remove todo. was left over from old location of this file