build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45392 )
Change subject: soc/intel/xeon_sp/cpx: generate ACPI BERT table ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/c/coreboot/+/45392/1/src/soc/intel/xeon_sp/cpx/K... File src/soc/intel/xeon_sp/cpx/Kconfig:
https://review.coreboot.org/c/coreboot/+/45392/1/src/soc/intel/xeon_sp/cpx/K... PS1, Line 104: records. trailing whitespace
https://review.coreboot.org/c/coreboot/+/45392/1/src/soc/intel/xeon_sp/cpx/a... File src/soc/intel/xeon_sp/cpx/acpi.c:
https://review.coreboot.org/c/coreboot/+/45392/1/src/soc/intel/xeon_sp/cpx/a... PS1, Line 900: if (!status){ space required before the open brace '{'
https://review.coreboot.org/c/coreboot/+/45392/1/src/soc/intel/xeon_sp/cpx/a... PS1, Line 956: if(!fill_bert_region()) space required before the open parenthesis '('
https://review.coreboot.org/c/coreboot/+/45392/1/src/soc/intel/xeon_sp/cpx/a... PS1, Line 962: printk(BIOS_INFO, "BERT region base: %p, error record size: 0x%lx\n", rgn, size); line over 96 characters