Attention is currently required from: Jonathan Zhang, Johnny Lin, Christian Walter, Arthur Heymans, Tim Chu.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69500 )
Change subject: src/soc/intel/xeon_sp/ras/hest.c: fix HEST table length ......................................................................
Patch Set 1:
(4 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-163370): https://review.coreboot.org/c/coreboot/+/69500/comment/106f9114_b3a90510 PS1, Line 8: Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-163370): https://review.coreboot.org/c/coreboot/+/69500/comment/684d91d3_1254f1e8 PS1, Line 8: 'begining' may be misspelled - perhaps 'beginning'?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-163370): https://review.coreboot.org/c/coreboot/+/69500/comment/54495d7a_356c296c PS1, Line 9: `current' is the very begining of table, so `next' - `current' is the total Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-163370): https://review.coreboot.org/c/coreboot/+/69500/comment/dea1d41c_5a102a39 PS1, Line 9: `current' is the very begining of table, so `next' - `current' is the total 'legnth' may be misspelled - perhaps 'length'?
-
build bot (Jenkins) (Code Review)