build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46880 )
Change subject: soc/intel/common/block/cse: Add MCHI MCA group functions to cse lib ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/c/coreboot/+/46880/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse.c:
https://review.coreboot.org/c/coreboot/+/46880/3/src/soc/intel/common/block/... PS3, Line 856: if(resp.data_size > *data_size){ space required before the open brace '{'
https://review.coreboot.org/c/coreboot/+/46880/3/src/soc/intel/common/block/... PS3, Line 856: if(resp.data_size > *data_size){ space required before the open parenthesis '('
https://review.coreboot.org/c/coreboot/+/46880/3/src/soc/intel/common/block/... PS3, Line 857: printk(BIOS_ERR, "HECI: Response data larger than requested data \n"); unnecessary whitespace before a quoted newline