build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64873 )
Change subject: csb_patcher.sh: gets,checks,installs the coreboot and SeaBIOS patches ......................................................................
Patch Set 4:
(6 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-165188): https://review.coreboot.org/c/coreboot/+/64873/comment/2368b8c9_39745678 PS4, Line 20: Please share your feedback/suggestions and tell about other useful patches, Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-165188): https://review.coreboot.org/c/coreboot/+/64873/comment/85c9a94a_89bcd038 PS4, Line 23: 1) "board-specific": Lenovo G505S discrete GPU support, custom SeaBIOS options, Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-165188): https://review.coreboot.org/c/coreboot/+/64873/comment/28493291_97126604 PS4, Line 24: example configs for G505S / AM1I-A / A88XM-E and known good AtomBIOS ROMs, Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-165188): https://review.coreboot.org/c/coreboot/+/64873/comment/d20d3c28_d7698e74 PS4, Line 25: and also good IRQ routing for AMD Lenovo G505S, ASUS A88XM-E and AM1I-A boards. Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-165188): https://review.coreboot.org/c/coreboot/+/64873/comment/24eee12e_fef6186f PS4, Line 31: Run ./csb_patcher.sh help or ./csb_patcher.sh usage for more information. Possible unwrapped commit description (prefer a maximum 72 chars per line)
File csb_patcher.sh:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-165188): https://review.coreboot.org/c/coreboot/+/64873/comment/4eea6f53_366ab178 PS4, Line 752: printf "\n${byellow}WARNING${bend}: important README at the beginning of ./$1.diff\n" 'README' may be misspelled - perhaps 'README'?