Patchew URL: https://patchew.org/QEMU/20210208023752.270606-1-richard.hender...@linaro.org/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20210208023752.270606-1-richard.hender...@linaro.org Subject: [PATCH v3 00/70] TCI fixes and cleanups === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu - [tag update] patchew/20210202132103.1096654-1-ppan...@redhat.com -> patchew/20210202132103.1096654-1-ppan...@redhat.com * [new tag] patchew/20210208023752.270606-1-richard.hender...@linaro.org -> patchew/20210208023752.270606-1-richard.hender...@linaro.org Switched to a new branch 'test' 130a759 gitlab: Enable cross-i386 builds of TCI 537da5c tests/tcg: Increase timeout for TCI 657c9d3 tcg/tci: Implement add2, sub2 1723413 tcg/tci: Implement mulu2, muls2 a939076 tcg/tci: Implement clz, ctz, ctpop 7209e1f tcg/tci: Implement extract, sextract 4d08848 tcg/tci: Implement andc, orc, eqv, nand, nor acf9fa4 tcg/tci: Implement movcond e2c5d5b tcg/tci: Implement goto_ptr 4888fb2 tcg/tci: Change encoding to uint32_t units c8a6e41 tcg/tci: Remove tci_write_reg c6f6fb3 tcg/tci: Emit setcond before brcond b5e61eb tcg/tci: Reserve r13 for a temporary 737c1a6 tcg/tci: Split out tcg_out_op_r[iI] 1d5e3b6 tcg/tci: Split out tcg_out_op_np 878bc25 tcg/tci: Split out tcg_out_op_v 7715c25 tcg/tci: Split out tcg_out_op_{rrm,rrrm,rrrrm} 0639600 tcg/tci: Split out tcg_out_op_rrrrcl 80d6564 tcg/tci: Split out tcg_out_op_rrrr 28e84b8 tcg/tci: Split out tcg_out_op_rrrrrr ba38a14 tcg/tci: Split out tcg_out_op_rrcl dfc698a tcg/tci: Split out tcg_out_op_rrrbb 4e349f3 tcg/tci: Split out tcg_out_op_rrrrrc c624d63 tcg/tci: Split out tcg_out_op_rrrc 7495322 tcg/tci: Split out tcg_out_op_rrr 9d981cb tcg/tci: Split out tcg_out_op_rr 4e89f8e tcg/tci: Split out tcg_out_op_p 43b03a7 tcg/tci: Split out tcg_out_op_l 891f923 tcg/tci: Split out tcg_out_op_rrs e0f64b1 tcg/tci: Push opcode emit into each case a8cdf4b tcg/tci: Move call-return regs to end of tcg_target_reg_alloc_order b658bdf tcg/tci: Improve tcg_target_call_clobber_regs 7061224 tcg/tci: Use ffi for calls 8f5fa28 tcg: Build ffi data structures for helpers a5aeb2f tcg/tci: Implement the disassembler properly cff39b3 tcg/tci: Remove tci_disas 436c626 tcg/tci: Hoist op_size checking into tci_args_* 3f97fe3 tcg/tci: Split out tci_args_{rrm,rrrm,rrrrm} 09c4b02 tcg/tci: Reduce qemu_ld/st TCGMemOpIdx operand to 32-bits d6ea609 tcg/tci: Clean up deposit operations 790a1a4 tcg/tci: Split out tci_args_rrrr 07179d8 tcg/tci: Split out tci_args_rrrrrr 2564979 tcg/tci: Reuse tci_args_l for goto_tb 94b2744 tcg/tci: Reuse tci_args_l for exit_tb fd26dc8 tcg/tci: Reuse tci_args_l for calls. 5578fd9 tcg/tci: Split out tci_args_ri and tci_args_rI e2b9810 tcg/tci: Split out tci_args_rrcl and tci_args_rrrrcl cfbdf1d tcg/tci: Split out tci_args_rrrrrc e4d84d0 tcg/tci: Split out tci_args_l 91a9a07 tcg/tci: Split out tci_args_rrrc b03a2ac tcg/tci: Split out tci_args_rrr fcc3c7a tcg/tci: Split out tci_args_rr d6ae8fc tcg/tci: Split out tci_args_rrs 243857a tcg/tci: Rename tci_read_r to tci_read_rval 3329020 tcg/tci: Merge mov, not and neg operations dbb2896 tcg/tci: Merge bswap operations 1c440bc tcg/tci: Remove ifdefs for TCG_TARGET_HAS_ext32[us]_i64 beabf85 tcg/tci: Merge extension operations 8d5959b tcg/tci: Merge basic arithmetic operations e22c5a6 tcg/tci: Reduce use of tci_read_r64 a8c774f tcg/tci: Remove tci_read_r32s 468e805 tcg/tci: Remove tci_read_r32 fddbd22 tcg/tci: Remove tci_read_r16s 187a012 tcg/tci: Remove tci_read_r16 5ce27c4 tcg/tci: Remove tci_read_r8s 8a4f80e tcg/tci: Remove tci_read_r8 6a28b59 tcg/tci: Merge identical cases in generation 36f0810 tcg: Manage splitwx in tc_ptr_to_region_tree by hand 1ae9d2d tcg: Split out tcg_raise_tb_overflow 955191f gdbstub: Fix handle_query_xfer_auxv === OUTPUT BEGIN === 1/70 Checking commit 955191ffda89 (gdbstub: Fix handle_query_xfer_auxv) 2/70 Checking commit 1ae9d2de47fc (tcg: Split out tcg_raise_tb_overflow) 3/70 Checking commit 36f08102a470 (tcg: Manage splitwx in tc_ptr_to_region_tree by hand) 4/70 Checking commit 6a28b596aec1 (tcg/tci: Merge identical cases in generation) 5/70 Checking commit 8a4f80e01c93 (tcg/tci: Remove tci_read_r8) 6/70 Checking commit 5ce27c4af25a (tcg/tci: Remove tci_read_r8s) 7/70 Checking commit 187a0127a7ac (tcg/tci: Remove tci_read_r16) 8/70 Checking commit fddbd22d85f0 (tcg/tci: Remove tci_read_r16s) 9/70 Checking commit 468e805759cd (tcg/tci: Remove tci_read_r32) 10/70 Checking commit a8c774f174e6 (tcg/tci: Remove tci_read_r32s) 11/70 Checking commit e22c5a6d4fd6 (tcg/tci: Reduce use of tci_read_r64) 12/70 Checking commit 8d5959bfd605 (tcg/tci: Merge basic arithmetic operations) 13/70 Checking commit beabf85d0bfc (tcg/tci: Merge extension operations) 14/70 Checking commit 1c440bc8bd0d (tcg/tci: Remove ifdefs for TCG_TARGET_HAS_ext32[us]_i64) 15/70 Checking commit dbb289624f02 (tcg/tci: Merge bswap operations) 16/70 Checking commit 3329020e9328 (tcg/tci: Merge mov, not and neg operations) 17/70 Checking commit 243857adadc3 (tcg/tci: Rename tci_read_r to tci_read_rval) 18/70 Checking commit d6ae8fc71f71 (tcg/tci: Split out tci_args_rrs) 19/70 Checking commit fcc3c7a34083 (tcg/tci: Split out tci_args_rr) 20/70 Checking commit b03a2ac0ea6b (tcg/tci: Split out tci_args_rrr) 21/70 Checking commit 91a9a076678a (tcg/tci: Split out tci_args_rrrc) 22/70 Checking commit e4d84d057d1d (tcg/tci: Split out tci_args_l) 23/70 Checking commit cfbdf1d2d888 (tcg/tci: Split out tci_args_rrrrrc) 24/70 Checking commit e2b9810902ac (tcg/tci: Split out tci_args_rrcl and tci_args_rrrrcl) 25/70 Checking commit 5578fd9a2f17 (tcg/tci: Split out tci_args_ri and tci_args_rI) 26/70 Checking commit fd26dc87058e (tcg/tci: Reuse tci_args_l for calls.) 27/70 Checking commit 94b274444d21 (tcg/tci: Reuse tci_args_l for exit_tb) 28/70 Checking commit 2564979b3403 (tcg/tci: Reuse tci_args_l for goto_tb) 29/70 Checking commit 07179d882605 (tcg/tci: Split out tci_args_rrrrrr) 30/70 Checking commit 790a1a4efd6e (tcg/tci: Split out tci_args_rrrr) 31/70 Checking commit d6ea609b4a77 (tcg/tci: Clean up deposit operations) 32/70 Checking commit 09c4b028c61f (tcg/tci: Reduce qemu_ld/st TCGMemOpIdx operand to 32-bits) 33/70 Checking commit 3f97fe31b87b (tcg/tci: Split out tci_args_{rrm,rrrm,rrrrm}) 34/70 Checking commit 436c6262ac2f (tcg/tci: Hoist op_size checking into tci_args_*) 35/70 Checking commit cff39b3b98da (tcg/tci: Remove tci_disas) 36/70 Checking commit a5aeb2fa96a7 (tcg/tci: Implement the disassembler properly) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #21: deleted file mode 100644 total: 0 errors, 1 warnings, 304 lines checked Patch 36/70 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 37/70 Checking commit 8f5fa28c2116 (tcg: Build ffi data structures for helpers) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #25: new file mode 100644 ERROR: Macros with complex values should be enclosed in parenthesis #40: FILE: include/exec/helper-ffi.h:11: +#define dh_ffitype_i32 &ffi_type_uint32 ERROR: Macros with complex values should be enclosed in parenthesis #41: FILE: include/exec/helper-ffi.h:12: +#define dh_ffitype_s32 &ffi_type_sint32 ERROR: Macros with complex values should be enclosed in parenthesis #43: FILE: include/exec/helper-ffi.h:14: +#define dh_ffitype_i64 &ffi_type_uint64 ERROR: Macros with complex values should be enclosed in parenthesis #44: FILE: include/exec/helper-ffi.h:15: +#define dh_ffitype_s64 &ffi_type_sint64 ERROR: Macros with complex values should be enclosed in parenthesis #45: FILE: include/exec/helper-ffi.h:16: +#define dh_ffitype_f16 &ffi_type_uint32 ERROR: Macros with complex values should be enclosed in parenthesis #46: FILE: include/exec/helper-ffi.h:17: +#define dh_ffitype_f32 &ffi_type_uint32 ERROR: Macros with complex values should be enclosed in parenthesis #47: FILE: include/exec/helper-ffi.h:18: +#define dh_ffitype_f64 &ffi_type_uint64 ERROR: Macros with complex values should be enclosed in parenthesis #50: FILE: include/exec/helper-ffi.h:21: +# define dh_ffitype_tl &ffi_type_uint32 ERROR: Macros with complex values should be enclosed in parenthesis #52: FILE: include/exec/helper-ffi.h:23: +# define dh_ffitype_tl &ffi_type_uint64 ERROR: Macros with complex values should be enclosed in parenthesis #55: FILE: include/exec/helper-ffi.h:26: +#define dh_ffitype_ptr &ffi_type_pointer ERROR: Macros with complex values should be enclosed in parenthesis #56: FILE: include/exec/helper-ffi.h:27: +#define dh_ffitype_cptr &ffi_type_pointer ERROR: Macros with complex values should be enclosed in parenthesis #59: FILE: include/exec/helper-ffi.h:30: +#define dh_ffitype_env &ffi_type_pointer ERROR: space required after that ',' (ctx:VxV) #63: FILE: include/exec/helper-ffi.h:34: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: Macros with multiple statements should be enclosed in a do - while loop #67: FILE: include/exec/helper-ffi.h:38: +#define DEF_HELPER_FLAGS_1(NAME, FLAGS, ret, t1) \ + static ffi_type *glue(cif_args_,NAME)[1] = { dh_ffitype(t1) }; \ + static ffi_cif glue(cif_,NAME) = { \ + .rtype = dh_ffitype(ret), .nargs = 1, \ + .arg_types = glue(cif_args_,NAME), \ + }; ERROR: spaces required around that '*' (ctx:WxV) #68: FILE: include/exec/helper-ffi.h:39: + static ffi_type *glue(cif_args_,NAME)[1] = { dh_ffitype(t1) }; \ ^ ERROR: space required after that ',' (ctx:VxV) #68: FILE: include/exec/helper-ffi.h:39: + static ffi_type *glue(cif_args_,NAME)[1] = { dh_ffitype(t1) }; \ ^ ERROR: open brace '{' following function declarations go on the next line #69: FILE: include/exec/helper-ffi.h:40: + static ffi_cif glue(cif_,NAME) = { \ ERROR: space required after that ',' (ctx:VxV) #69: FILE: include/exec/helper-ffi.h:40: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: space required after that ',' (ctx:VxV) #71: FILE: include/exec/helper-ffi.h:42: + .arg_types = glue(cif_args_,NAME), \ ^ ERROR: Macros with multiple statements should be enclosed in a do - while loop #74: FILE: include/exec/helper-ffi.h:45: +#define DEF_HELPER_FLAGS_2(NAME, FLAGS, ret, t1, t2) \ + static ffi_type *glue(cif_args_,NAME)[2] = { \ + dh_ffitype(t1), dh_ffitype(t2) \ + }; \ + static ffi_cif glue(cif_,NAME) = { \ + .rtype = dh_ffitype(ret), .nargs = 2, \ + .arg_types = glue(cif_args_,NAME), \ + }; ERROR: spaces required around that '*' (ctx:WxV) #75: FILE: include/exec/helper-ffi.h:46: + static ffi_type *glue(cif_args_,NAME)[2] = { \ ^ ERROR: space required after that ',' (ctx:VxV) #75: FILE: include/exec/helper-ffi.h:46: + static ffi_type *glue(cif_args_,NAME)[2] = { \ ^ ERROR: open brace '{' following function declarations go on the next line #78: FILE: include/exec/helper-ffi.h:49: + static ffi_cif glue(cif_,NAME) = { \ ERROR: space required after that ',' (ctx:VxV) #78: FILE: include/exec/helper-ffi.h:49: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: space required after that ',' (ctx:VxV) #80: FILE: include/exec/helper-ffi.h:51: + .arg_types = glue(cif_args_,NAME), \ ^ ERROR: Macros with multiple statements should be enclosed in a do - while loop #83: FILE: include/exec/helper-ffi.h:54: +#define DEF_HELPER_FLAGS_3(NAME, FLAGS, ret, t1, t2, t3) \ + static ffi_type *glue(cif_args_,NAME)[3] = { \ + dh_ffitype(t1), dh_ffitype(t2), dh_ffitype(t3) \ + }; \ + static ffi_cif glue(cif_,NAME) = { \ + .rtype = dh_ffitype(ret), .nargs = 3, \ + .arg_types = glue(cif_args_,NAME), \ + }; ERROR: spaces required around that '*' (ctx:WxV) #84: FILE: include/exec/helper-ffi.h:55: + static ffi_type *glue(cif_args_,NAME)[3] = { \ ^ ERROR: space required after that ',' (ctx:VxV) #84: FILE: include/exec/helper-ffi.h:55: + static ffi_type *glue(cif_args_,NAME)[3] = { \ ^ ERROR: open brace '{' following function declarations go on the next line #87: FILE: include/exec/helper-ffi.h:58: + static ffi_cif glue(cif_,NAME) = { \ ERROR: space required after that ',' (ctx:VxV) #87: FILE: include/exec/helper-ffi.h:58: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: space required after that ',' (ctx:VxV) #89: FILE: include/exec/helper-ffi.h:60: + .arg_types = glue(cif_args_,NAME), \ ^ ERROR: Macros with multiple statements should be enclosed in a do - while loop #92: FILE: include/exec/helper-ffi.h:63: +#define DEF_HELPER_FLAGS_4(NAME, FLAGS, ret, t1, t2, t3, t4) \ + static ffi_type *glue(cif_args_,NAME)[4] = { \ + dh_ffitype(t1), dh_ffitype(t2), dh_ffitype(t3), dh_ffitype(t4) \ + }; \ + static ffi_cif glue(cif_,NAME) = { \ + .rtype = dh_ffitype(ret), .nargs = 4, \ + .arg_types = glue(cif_args_,NAME), \ + }; ERROR: spaces required around that '*' (ctx:WxV) #93: FILE: include/exec/helper-ffi.h:64: + static ffi_type *glue(cif_args_,NAME)[4] = { \ ^ ERROR: space required after that ',' (ctx:VxV) #93: FILE: include/exec/helper-ffi.h:64: + static ffi_type *glue(cif_args_,NAME)[4] = { \ ^ ERROR: open brace '{' following function declarations go on the next line #96: FILE: include/exec/helper-ffi.h:67: + static ffi_cif glue(cif_,NAME) = { \ ERROR: space required after that ',' (ctx:VxV) #96: FILE: include/exec/helper-ffi.h:67: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: space required after that ',' (ctx:VxV) #98: FILE: include/exec/helper-ffi.h:69: + .arg_types = glue(cif_args_,NAME), \ ^ ERROR: Macros with multiple statements should be enclosed in a do - while loop #101: FILE: include/exec/helper-ffi.h:72: +#define DEF_HELPER_FLAGS_5(NAME, FLAGS, ret, t1, t2, t3, t4, t5) \ + static ffi_type *glue(cif_args_,NAME)[5] = { \ + dh_ffitype(t1), dh_ffitype(t2), dh_ffitype(t3), \ + dh_ffitype(t4), dh_ffitype(t5) \ + }; \ + static ffi_cif glue(cif_,NAME) = { \ + .rtype = dh_ffitype(ret), .nargs = 5, \ + .arg_types = glue(cif_args_,NAME), \ + }; ERROR: spaces required around that '*' (ctx:WxV) #102: FILE: include/exec/helper-ffi.h:73: + static ffi_type *glue(cif_args_,NAME)[5] = { \ ^ ERROR: space required after that ',' (ctx:VxV) #102: FILE: include/exec/helper-ffi.h:73: + static ffi_type *glue(cif_args_,NAME)[5] = { \ ^ ERROR: open brace '{' following function declarations go on the next line #106: FILE: include/exec/helper-ffi.h:77: + static ffi_cif glue(cif_,NAME) = { \ ERROR: space required after that ',' (ctx:VxV) #106: FILE: include/exec/helper-ffi.h:77: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: space required after that ',' (ctx:VxV) #108: FILE: include/exec/helper-ffi.h:79: + .arg_types = glue(cif_args_,NAME), \ ^ ERROR: Macros with multiple statements should be enclosed in a do - while loop #111: FILE: include/exec/helper-ffi.h:82: +#define DEF_HELPER_FLAGS_6(NAME, FLAGS, ret, t1, t2, t3, t4, t5, t6) \ + static ffi_type *glue(cif_args_,NAME)[6] = { \ + dh_ffitype(t1), dh_ffitype(t2), dh_ffitype(t3), \ + dh_ffitype(t4), dh_ffitype(t5), dh_ffitype(t6) \ + }; \ + static ffi_cif glue(cif_,NAME) = { \ + .rtype = dh_ffitype(ret), .nargs = 6, \ + .arg_types = glue(cif_args_,NAME), \ + }; ERROR: spaces required around that '*' (ctx:WxV) #112: FILE: include/exec/helper-ffi.h:83: + static ffi_type *glue(cif_args_,NAME)[6] = { \ ^ ERROR: space required after that ',' (ctx:VxV) #112: FILE: include/exec/helper-ffi.h:83: + static ffi_type *glue(cif_args_,NAME)[6] = { \ ^ ERROR: open brace '{' following function declarations go on the next line #116: FILE: include/exec/helper-ffi.h:87: + static ffi_cif glue(cif_,NAME) = { \ ERROR: space required after that ',' (ctx:VxV) #116: FILE: include/exec/helper-ffi.h:87: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: space required after that ',' (ctx:VxV) #118: FILE: include/exec/helper-ffi.h:89: + .arg_types = glue(cif_args_,NAME), \ ^ ERROR: Macros with multiple statements should be enclosed in a do - while loop #121: FILE: include/exec/helper-ffi.h:92: +#define DEF_HELPER_FLAGS_7(NAME, FLAGS, ret, t1, t2, t3, t4, t5, t6, t7) \ + static ffi_type *glue(cif_args_,NAME)[7] = { \ + dh_ffitype(t1), dh_ffitype(t2), dh_ffitype(t3), \ + dh_ffitype(t4), dh_ffitype(t5), dh_ffitype(t6), dh_ffitype(t7) \ + }; \ + static ffi_cif glue(cif_,NAME) = { \ + .rtype = dh_ffitype(ret), .nargs = 7, \ + .arg_types = glue(cif_args_,NAME), \ + }; ERROR: spaces required around that '*' (ctx:WxV) #122: FILE: include/exec/helper-ffi.h:93: + static ffi_type *glue(cif_args_,NAME)[7] = { \ ^ ERROR: space required after that ',' (ctx:VxV) #122: FILE: include/exec/helper-ffi.h:93: + static ffi_type *glue(cif_args_,NAME)[7] = { \ ^ ERROR: open brace '{' following function declarations go on the next line #126: FILE: include/exec/helper-ffi.h:97: + static ffi_cif glue(cif_,NAME) = { \ ERROR: space required after that ',' (ctx:VxV) #126: FILE: include/exec/helper-ffi.h:97: + static ffi_cif glue(cif_,NAME) = { \ ^ ERROR: space required after that ',' (ctx:VxV) #128: FILE: include/exec/helper-ffi.h:99: + .arg_types = glue(cif_args_,NAME), \ ^ total: 55 errors, 1 warnings, 309 lines checked Patch 37/70 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 38/70 Checking commit 7061224572bd (tcg/tci: Use ffi for calls) ERROR: spaces required around that '+' (ctx:VxV) #93: FILE: tcg/tcg.c:2085: + bool is_64bit = sizemask & (1 << (i+1)*2); ^ ERROR: spaces required around that '*' (ctx:VxV) #93: FILE: tcg/tcg.c:2085: + bool is_64bit = sizemask & (1 << (i+1)*2); ^ ERROR: suspect code indent for conditional statements (8, 11) #122: FILE: tcg/tcg.c:2104: + if (TCG_TARGET_REG_BITS < 64 && is_64bit) { + /* total: 3 errors, 0 warnings, 380 lines checked Patch 38/70 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 39/70 Checking commit b658bdf748c2 (tcg/tci: Improve tcg_target_call_clobber_regs) 40/70 Checking commit a8cdf4b59504 (tcg/tci: Move call-return regs to end of tcg_target_reg_alloc_order) 41/70 Checking commit e0f64b1f53b0 (tcg/tci: Push opcode emit into each case) 42/70 Checking commit 891f923a5e8d (tcg/tci: Split out tcg_out_op_rrs) 43/70 Checking commit 43b03a7a482a (tcg/tci: Split out tcg_out_op_l) 44/70 Checking commit 4e89f8e94f32 (tcg/tci: Split out tcg_out_op_p) 45/70 Checking commit 9d981cb696ce (tcg/tci: Split out tcg_out_op_rr) 46/70 Checking commit 749532228b16 (tcg/tci: Split out tcg_out_op_rrr) 47/70 Checking commit c624d631f834 (tcg/tci: Split out tcg_out_op_rrrc) 48/70 Checking commit 4e349f3024c8 (tcg/tci: Split out tcg_out_op_rrrrrc) 49/70 Checking commit dfc698a5e4a7 (tcg/tci: Split out tcg_out_op_rrrbb) 50/70 Checking commit ba38a147fdbb (tcg/tci: Split out tcg_out_op_rrcl) 51/70 Checking commit 28e84b83dd6d (tcg/tci: Split out tcg_out_op_rrrrrr) 52/70 Checking commit 80d6564e8034 (tcg/tci: Split out tcg_out_op_rrrr) 53/70 Checking commit 0639600d9269 (tcg/tci: Split out tcg_out_op_rrrrcl) 54/70 Checking commit 7715c258a91e (tcg/tci: Split out tcg_out_op_{rrm,rrrm,rrrrm}) 55/70 Checking commit 878bc25538ce (tcg/tci: Split out tcg_out_op_v) 56/70 Checking commit 1d5e3b6e64a6 (tcg/tci: Split out tcg_out_op_np) 57/70 Checking commit 737c1a64f096 (tcg/tci: Split out tcg_out_op_r[iI]) 58/70 Checking commit b5e61ebe266e (tcg/tci: Reserve r13 for a temporary) 59/70 Checking commit c6f6fb337877 (tcg/tci: Emit setcond before brcond) 60/70 Checking commit c8a6e415cc88 (tcg/tci: Remove tci_write_reg) 61/70 Checking commit 4888fb2af0de (tcg/tci: Change encoding to uint32_t units) 62/70 Checking commit e2c5d5b3d328 (tcg/tci: Implement goto_ptr) 63/70 Checking commit acf9fa4f54a5 (tcg/tci: Implement movcond) 64/70 Checking commit 4d08848ba871 (tcg/tci: Implement andc, orc, eqv, nand, nor) 65/70 Checking commit 7209e1f21448 (tcg/tci: Implement extract, sextract) 66/70 Checking commit a939076bac17 (tcg/tci: Implement clz, ctz, ctpop) 67/70 Checking commit 1723413e3f8e (tcg/tci: Implement mulu2, muls2) 68/70 Checking commit 657c9d38d025 (tcg/tci: Implement add2, sub2) 69/70 Checking commit 537da5c6c20d (tests/tcg: Increase timeout for TCI) 70/70 Checking commit 130a759ec343 (gitlab: Enable cross-i386 builds of TCI) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20210208023752.270606-1-richard.hender...@linaro.org/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com