build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63560 )
Change subject: [VERY WIP]tests/cpu/x86/smm: Add unit tests ......................................................................
Patch Set 20: Verified-1
(10 comments)
File tests/cpu/x86/smm/smm_loader.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/78930b80_bf68675a PS20, Line 54: #define setup_rmodule(mod , c) \ space prohibited before that ',' (ctx:WxW)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/72f77377_da062ec0 PS20, Line 54: #define setup_rmodule(mod , c) \ macros should not use a trailing semicolon
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/23091483_e8ab72da PS20, Line 59: hdr->payload_begin_offset = (uintptr_t)(stub_module->program) - (uintptr_t)stub_module; \ line length of 97 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/c01aa03b_d15117a9 PS20, Line 66: hdr->parameters_begin = (uintptr_t)&stub_module->params - (uintptr_t)stub_module->program; \ line length of 100 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/782fdba1_b6c1e959 PS20, Line 80: /* hdr->payload_begin_offset = (uintptr_t)(stub_module->program) - (uintptr_t)stub_module; */ line length of 101 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/08b2618a_754cda80 PS20, Line 87: /* hdr->parameters_begin = (uintptr_t)&stub_module->params - (uintptr_t)stub_module->program ; */ line length of 105 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/1e072aec_340b2fa5 PS20, Line 93: /* stub_module->program[i] = stub_string[i % sizeof(stub_string)]; */ please, no space before tabs
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/c8653a18_6030a958 PS20, Line 106: hdr->payload_begin_offset = (uintptr_t)(handler_module->program) - (uintptr_t)handler_module; line length of 101 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/ea6592ec_f5288029 PS20, Line 107: hdr->payload_end_offset = (uintptr_t)(handler_module->program) - (uintptr_t)handler_module + sizeof(handler_module->program); line length of 133 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150281): https://review.coreboot.org/c/coreboot/+/63560/comment/9267ba23_1d90e29a PS20, Line 113: hdr->parameters_begin = (uintptr_t)&handler_module->params - (uintptr_t)handler_module->program; line length of 104 exceeds 96 columns
-
build bot (Jenkins) (Code Review)