build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44974 )
Change subject: soc/intel/xeon_sp/cpx: display FSP_PREV_BOOT_ERR_SRC_HOB ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44974/1/src/soc/intel/xeon_sp/cpx/h... File src/soc/intel/xeon_sp/cpx/hob_display.c:
https://review.coreboot.org/c/coreboot/+/44974/1/src/soc/intel/xeon_sp/cpx/h... PS1, Line 199: printk(BIOS_DEBUG, "\t\t McBankStatus: 0x%llx\n", mcbinfo->McBankStatus); line over 96 characters
https://review.coreboot.org/c/coreboot/+/44974/1/src/soc/intel/xeon_sp/cpx/h... PS1, Line 210: printk(BIOS_ERR, "\t illegal ERROR_ACCESS_TYPE:%d \n", *type); unnecessary whitespace before a quoted newline