Patchew URL: https://patchew.org/QEMU/20200616170844.13318-1-peter.mayd...@linaro.org/
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH 00/21] target/arm: Finish neon decodetree conversion Type: series Message-id: 20200616170844.13318-1-peter.mayd...@linaro.org === 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 Switched to a new branch 'test' f6e3d71 target/arm: Move some functions used only in translate-neon.inc.c to that file 4a2df60 target/arm: Convert Neon VTRN to decodetree 26f74d6 target/arm: Convert Neon VSWP to decodetree 7a0fa02 target/arm: Convert Neon 2-reg-misc VCVT insns to decodetree 73198e7 target/arm: Convert Neon 2-reg-misc VRINT insns to decodetree 0908381 target/arm: Convert Neon 2-reg-misc fp-compare-with-zero insns to decodetree f6d72da target/arm: Convert simple fp Neon 2-reg-misc insns 3ef1127 target/arm: Convert Neon VQABS, VQNEG to decodetree 624f399 target/arm: Convert remaining simple 2-reg-misc Neon ops 9bb8fa6 target/arm: Convert Neon 2-reg-misc VREV32 and VREV16 to decodetree 3535d27 target/arm: Make gen_swap_half() take separate src and dest 2eac819 target/arm: Fix capitalization in NeonGenTwo{Single, Double}OPFn typedefs e30825b target/arm: Rename NeonGenOneOpFn to NeonGenOne64OpFn 7d1109a target/arm: Convert Neon 2-reg-misc crypto operations to decodetree 9d87342 target/arm: Convert vectorised 2-reg-misc Neon ops to decodetree 605ae75 target/arm: Convert Neon VCVT f16/f32 insns to decodetree 5fb6c16 target/arm: Convert Neon 2-reg-misc VSHLL to decodetree e2e99ab target/arm: Convert Neon narrowing moves to decodetree 3dde5dd target/arm: Convert VZIP, VUZP to decodetree 3ed7eaf target/arm: Convert Neon 2-reg-misc pairwise ops to decodetree 37f7428 target/arm: Convert Neon 2-reg-misc VREV64 to decodetree === OUTPUT BEGIN === 1/21 Checking commit 37f7428534e5 (target/arm: Convert Neon 2-reg-misc VREV64 to decodetree) 2/21 Checking commit 3ed7eaff8f5f (target/arm: Convert Neon 2-reg-misc pairwise ops to decodetree) 3/21 Checking commit 3dde5ddb764b (target/arm: Convert VZIP, VUZP to decodetree) 4/21 Checking commit e2e99ab61e59 (target/arm: Convert Neon narrowing moves to decodetree) 5/21 Checking commit 5fb6c161af86 (target/arm: Convert Neon 2-reg-misc VSHLL to decodetree) 6/21 Checking commit 605ae75d2431 (target/arm: Convert Neon VCVT f16/f32 insns to decodetree) 7/21 Checking commit 9d87342857c8 (target/arm: Convert vectorised 2-reg-misc Neon ops to decodetree) 8/21 Checking commit 7d1109aae4db (target/arm: Convert Neon 2-reg-misc crypto operations to decodetree) 9/21 Checking commit e30825bbd0a4 (target/arm: Rename NeonGenOneOpFn to NeonGenOne64OpFn) 10/21 Checking commit 2eac8198e699 (target/arm: Fix capitalization in NeonGenTwo{Single, Double}OPFn typedefs) 11/21 Checking commit 3535d2721010 (target/arm: Make gen_swap_half() take separate src and dest) ERROR: trailing statements should be on next line #50: FILE: target/arm/translate.c:4963: + case 1: gen_swap_half(tmp, tmp); break; total: 1 errors, 0 warnings, 43 lines checked Patch 11/21 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/21 Checking commit 9bb8fa6a3658 (target/arm: Convert Neon 2-reg-misc VREV32 and VREV16 to decodetree) 13/21 Checking commit 624f399e8573 (target/arm: Convert remaining simple 2-reg-misc Neon ops) 14/21 Checking commit 3ef11275c669 (target/arm: Convert Neon VQABS, VQNEG to decodetree) 15/21 Checking commit f6d72da4c6fc (target/arm: Convert simple fp Neon 2-reg-misc insns) 16/21 Checking commit 090838197cf0 (target/arm: Convert Neon 2-reg-misc fp-compare-with-zero insns to decodetree) 17/21 Checking commit 73198e721bff (target/arm: Convert Neon 2-reg-misc VRINT insns to decodetree) 18/21 Checking commit 7a0fa02240e2 (target/arm: Convert Neon 2-reg-misc VCVT insns to decodetree) 19/21 Checking commit 26f74d6de184 (target/arm: Convert Neon VSWP to decodetree) 20/21 Checking commit 4a2df6081512 (target/arm: Convert Neon VTRN to decodetree) 21/21 Checking commit f6e3d7186b3e (target/arm: Move some functions used only in translate-neon.inc.c to that file) WARNING: Block comments use a leading /* on a separate line #28: FILE: target/arm/translate-neon.inc.c:57: +/* Return the offset of a 2**SIZE piece of a NEON register, at index ELE, WARNING: Block comments use a leading /* on a separate line #37: FILE: target/arm/translate-neon.inc.c:66: + /* Calculate the offset assuming fully little-endian, total: 0 errors, 2 warnings, 226 lines checked Patch 21/21 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200616170844.13318-1-peter.mayd...@linaro.org/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com