build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55074 )
Change subject: cbfstool/linux_trampoline.S: Fixup the e820 table ......................................................................
Patch Set 1:
(19 comments)
File util/cbfstool/linux_trampoline.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/26352015_52017210 PS1, Line 6: 0xcb, 0x0f, 0x84, 0xab, 0x00, 0x00, 0x00, 0x8b, 0x59, 0x04, 0x01, 0xcb, 0x8b, 0x49, 0x14, 0x83, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/08ef6daf_c2e31ccd PS1, Line 6: 0xcb, 0x0f, 0x84, 0xab, 0x00, 0x00, 0x00, 0x8b, 0x59, 0x04, 0x01, 0xcb, 0x8b, 0x49, 0x14, 0x83, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/c196412e_fba9c8a3 PS1, Line 7: 0x3b, 0x11, 0x75, 0x05, 0x8b, 0x4b, 0x08, 0xeb, 0xcf, 0x83, 0x3b, 0x01, 0x75, 0x54, 0x8b, 0x43, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/60997b76_ff30c869 PS1, Line 7: 0x3b, 0x11, 0x75, 0x05, 0x8b, 0x4b, 0x08, 0xeb, 0xcf, 0x83, 0x3b, 0x01, 0x75, 0x54, 0x8b, 0x43, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/6aa1e5b9_07cddaa0 PS1, Line 10: 0x00, 0x89, 0xf0, 0x91, 0x8d, 0x73, 0x08, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0xf3, 0xa5, 0x91, 0xa1, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/53ae1f3c_6b4af2a5 PS1, Line 10: 0x00, 0x89, 0xf0, 0x91, 0x8d, 0x73, 0x08, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0xf3, 0xa5, 0x91, 0xa1, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/80665979_0530043b PS1, Line 11: 0xe8, 0x01, 0x09, 0x00, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0x83, 0xf8, 0x00, 0x74, 0x19, 0x83, 0x7f, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/3799b3c2_4284d0b1 PS1, Line 11: 0xe8, 0x01, 0x09, 0x00, 0xbf, 0xd0, 0x02, 0x09, 0x00, 0x83, 0xf8, 0x00, 0x74, 0x19, 0x83, 0x7f, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/afb4cf66_b667cb1c PS1, Line 12: 0x10, 0x0c, 0x7e, 0x08, 0xba, 0x02, 0x00, 0x00, 0x00, 0x89, 0x57, 0x10, 0x48, 0x83, 0xc7, 0x14, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/5f3d8182_2109123d PS1, Line 12: 0x10, 0x0c, 0x7e, 0x08, 0xba, 0x02, 0x00, 0x00, 0x00, 0x89, 0x57, 0x10, 0x48, 0x83, 0xc7, 0x14, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/85ed62e0_6e730a0c PS1, Line 13: 0xeb, 0xe7, 0x83, 0x3b, 0x12, 0x75, 0x00, 0x03, 0x5b, 0x04, 0x49, 0x75, 0x92, 0xb8, 0x00, 0x00, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/14533cc9_2b692b32 PS1, Line 13: 0xeb, 0xe7, 0x83, 0x3b, 0x12, 0x75, 0x00, 0x03, 0x5b, 0x04, 0x49, 0x75, 0x92, 0xb8, 0x00, 0x00, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/91032a19_35c895ea PS1, Line 14: 0x04, 0x00, 0xc7, 0x40, 0x10, 0xff, 0xff, 0x00, 0x00, 0xc7, 0x40, 0x14, 0x00, 0x9b, 0xcf, 0x00, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/35fee0c8_615bdbd4 PS1, Line 14: 0x04, 0x00, 0xc7, 0x40, 0x10, 0xff, 0xff, 0x00, 0x00, 0xc7, 0x40, 0x14, 0x00, 0x9b, 0xcf, 0x00, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/6fe6ae1c_ef2dd5c1 PS1, Line 15: 0xc7, 0x40, 0x18, 0xff, 0xff, 0x00, 0x00, 0xc7, 0x40, 0x1c, 0x00, 0x93, 0xcf, 0x00, 0xc6, 0x00, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/62235129_64c4d5d3 PS1, Line 15: 0xc7, 0x40, 0x18, 0xff, 0xff, 0x00, 0x00, 0xc7, 0x40, 0x1c, 0x00, 0x93, 0xcf, 0x00, 0xc6, 0x00, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/c6fffe66_b3a73114 PS1, Line 16: 0x2b, 0x89, 0x40, 0x02, 0x0f, 0x01, 0x10, 0xbe, 0x00, 0x00, 0x09, 0x00, 0xff, 0x25, 0x14, 0x02, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/783130d6_134f89c1 PS1, Line 16: 0x2b, 0x89, 0x40, 0x02, 0x0f, 0x01, 0x10, 0xbe, 0x00, 0x00, 0x09, 0x00, 0xff, 0x25, 0x14, 0x02, please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120287): https://review.coreboot.org/c/coreboot/+/55074/comment/5d09fe5e_b9cb2a5d PS1, Line 17: 0x09, 0x00, 0xf4, 0xeb, 0xfd please, no spaces at the start of a line