Patchew URL: https://patchew.org/QEMU/20201021045149.1582203-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: 20201021045149.1582203-1-richard.hender...@linaro.org Subject: [RFC PATCH 00/15] softfloat: alternate conversion of float128_addsub === 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 * [new tag] patchew/20201021045149.1582203-1-richard.hender...@linaro.org -> patchew/20201021045149.1582203-1-richard.hender...@linaro.org Switched to a new branch 'test' ad120c1 softfloat: Improve subtraction of equal exponent 12eb5a4 softfloat: Use float_cmask for addsub_floats a04ff7d Test float128_addsub fc9537e softfloat: Streamline FloatFmt 979beb6 Test split to softfloat-parts.c.inc 4317991 softfloat: Inline float_raise 4689bd2 softfloat: Add float_cmask and constants b1141ee softfloat: Tidy a * b + inf return 197273c softfloat: Use int128.h for some operations aa4afa2 softfloat: Use mulu64 for mul64To128 017d276 qemu/int128: Add int128_geu 71dd5f1 qemu/int128: Add int128_shr 9144df9 qemu/int128: Rename int128_rshift, int128_lshift b6c9afb qemu/int128: Add int128_clz, int128_ctz 0ceff9a qemu/int128: Add int128_or === OUTPUT BEGIN === 1/15 Checking commit 0ceff9a14aa6 (qemu/int128: Add int128_or) 2/15 Checking commit b6c9afb58357 (qemu/int128: Add int128_clz, int128_ctz) 3/15 Checking commit 9144df990b17 (qemu/int128: Rename int128_rshift, int128_lshift) 4/15 Checking commit 71dd5f157a39 (qemu/int128: Add int128_shr) 5/15 Checking commit 017d27627112 (qemu/int128: Add int128_geu) 6/15 Checking commit aa4afa22ee78 (softfloat: Use mulu64 for mul64To128) 7/15 Checking commit 197273c0aeda (softfloat: Use int128.h for some operations) 8/15 Checking commit b1141eecc368 (softfloat: Tidy a * b + inf return) 9/15 Checking commit 4689bd26fd66 (softfloat: Add float_cmask and constants) 10/15 Checking commit 4317991dcbd8 (softfloat: Inline float_raise) 11/15 Checking commit 979beb676e89 (Test split to softfloat-parts.c.inc) WARNING: Block comments use a leading /* on a separate line #557: FILE: fpu/softfloat.c:685: + /* Note that this check is after pickNaNMulAdd so that function ERROR: Missing Signed-off-by: line(s) total: 1 errors, 1 warnings, 786 lines checked Patch 11/15 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/15 Checking commit fc9537e66b73 (softfloat: Streamline FloatFmt) 13/15 Checking commit a04ff7dcb003 (Test float128_addsub) ERROR: Missing Signed-off-by: line(s) total: 1 errors, 0 warnings, 405 lines checked Patch 13/15 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 14/15 Checking commit 12eb5a486720 (softfloat: Use float_cmask for addsub_floats) 15/15 Checking commit ad120c1d1ae8 (softfloat: Improve subtraction of equal exponent) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20201021045149.1582203-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
