Hi, This series seems to have some coding style problems. See output below for more information:
Type: series Message-id: [email protected] Subject: [Qemu-devel] [PATCH 0/9] target/arm: Fixups for ARM_FEATURE_V8_FP16 === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram commits="$(git log --format=%H --reverse $BASE..)" for c in $commits; do echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..." if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then failed=1 echo fi n=$((n+1)) done exit $failed === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu t [tag update] patchew/1513160272-15921-1-git-send-email-christian.ehrha...@canonical.com -> patchew/1513160272-15921-1-git-send-email-christian.ehrha...@canonical.com t [tag update] patchew/[email protected] -> patchew/[email protected] t [tag update] patchew/[email protected] -> patchew/[email protected] t [tag update] patchew/[email protected] -> patchew/[email protected] t [tag update] patchew/[email protected] -> patchew/[email protected] * [new tag] patchew/[email protected] -> patchew/[email protected] Switched to a new branch 'test' 84efc5c3b1 target/arm: Implement FP data-processing (3 source) for fp16 10574670d9 target/arm: Implement FP data-processing (2 source) for fp16 2d4e187869 target/arm: Implement FCVT (scalar, fixed-point) for fp16 ec350065e4 target/arm: Implement FCVT (scalar, integer) for fp16 d157aaa589 target/arm: Implement FMOV (general) for fp16 7eeb78179f target/arm: Clear SVE high bits for FMOV a7fed5d8a6 target/arm: Fix float16 to/from int16 72239133a0 target/arm: Implement vector shifted FCVT for fp16 5cb4ff2996 target/arm: Implement vector shifted SCVF/UCVF for fp16 === OUTPUT BEGIN === Checking PATCH 1/9: target/arm: Implement vector shifted SCVF/UCVF for fp16... Checking PATCH 2/9: target/arm: Implement vector shifted FCVT for fp16... Checking PATCH 3/9: target/arm: Fix float16 to/from int16... ERROR: spaces required around that '*' (ctx:WxV) #40: FILE: target/arm/helper.c:11423: +static float16 do_postscale_fp16(float64 f, int shift, float_status *fpst) ^ total: 1 errors, 0 warnings, 83 lines checked Your patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Checking PATCH 4/9: target/arm: Clear SVE high bits for FMOV... Checking PATCH 5/9: target/arm: Implement FMOV (general) for fp16... Checking PATCH 6/9: target/arm: Implement FCVT (scalar, integer) for fp16... Checking PATCH 7/9: target/arm: Implement FCVT (scalar, fixed-point) for fp16... Checking PATCH 8/9: target/arm: Implement FP data-processing (2 source) for fp16... Checking PATCH 9/9: target/arm: Implement FP data-processing (3 source) for fp16... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to [email protected]
