Re: [Qemu-devel] [PATCH v3 00/23] target/openrisc improvements

[no-reply]

Hi,

This series seems to have some coding style problems. See output below for
more information:

Type: series
Message-id: address@hidden
Subject: [Qemu-devel] [PATCH v3 00/23] target/openrisc improvements

=== 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
Switched to a new branch 'test'
d2fcddb810 linux-user: Fix struct sigaltstack for openrisc
54bf3816e0 linux-user: Implement signals for openrisc
afd05358f9 target/openrisc: Add support in scripts/qemu-binfmt-conf.sh
6c3691e32d target/openrisc: Reorg tlb lookup
f53e2facc7 target/openrisc: Increase the TLB size
0d21fc5c8c target/openrisc: Stub out handle_mmu_fault for softmmu
f62561d1b7 target/openrisc: Use identical sizes for ITLB and DTLB
836489970f target/openrisc: Fix cpu_mmu_index
82ec0e63d4 target/openrisc: Fix tlb flushing in mtspr
7657ccb683 target/openrisc: Reduce tlb to a single dimension
0dedf4e4b9 target/openrisc: Merge mmu_helper.c into mmu.c
55d39fdea4 target/openrisc: Remove indirect function calls for mmu
68faceb291 target/openrisc: Merge tlb allocation into CPUOpenRISCState
bb22ed4bd0 target/openrisc: Form the spr index from tcg
aee6c05885 target/openrisc: Exit the TB after l.mtspr
2fa87705e3 target/openrisc: Split out is_user
8ffdeef89c target/openrisc: Link more translation blocks
14800ca37d target/openrisc: Fix singlestep_enabled
d40522dde7 target/openrisc: Use exit_tb instead of CPU_INTERRUPT_EXITTB
3571c024ff target/openrisc: Remove DISAS_JUMP & DISAS_TB_JUMP
7ae2154034 target/openrisc: Log interrupts
f1f81972ff target/openrisc: Add print_insn_or1k
92ac734d19 target/openrisc: Fix mtspr shadow gprs

=== OUTPUT BEGIN ===
Checking PATCH 1/23: target/openrisc: Fix mtspr shadow gprs...
Checking PATCH 2/23: target/openrisc: Add print_insn_or1k...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#69: 
new file mode 100644

ERROR: Macros with complex values should be enclosed in parenthesis
#104: FILE: target/openrisc/disas.c:31:
+#define output(mnemonic, format, ...) \
+    info->fprintf_func(info->stream, "%-9s " format, \
+                       mnemonic, ##__VA_ARGS__)

ERROR: spaces required around that '*' (ctx:WxV)
#129: FILE: target/openrisc/disas.c:56:
+    arg_l_##opcode *a, uint32_t insn)                   \
                    ^

ERROR: spaces required around that '*' (ctx:WxV)
#224: FILE: target/openrisc/disas.c:151:
+    arg_lf_##opcode##_##suffix *a, uint32_t insn)                \
                                ^

total: 3 errors, 1 warnings, 923 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 3/23: target/openrisc: Log interrupts...
Checking PATCH 4/23: target/openrisc: Remove DISAS_JUMP & DISAS_TB_JUMP...
Checking PATCH 5/23: target/openrisc: Use exit_tb instead of 
CPU_INTERRUPT_EXITTB...
Checking PATCH 6/23: target/openrisc: Fix singlestep_enabled...
Checking PATCH 7/23: target/openrisc: Link more translation blocks...
Checking PATCH 8/23: target/openrisc: Split out is_user...
Checking PATCH 9/23: target/openrisc: Exit the TB after l.mtspr...
Checking PATCH 10/23: target/openrisc: Form the spr index from tcg...
Checking PATCH 11/23: target/openrisc: Merge tlb allocation into 
CPUOpenRISCState...
Checking PATCH 12/23: target/openrisc: Remove indirect function calls for mmu...
Checking PATCH 13/23: target/openrisc: Merge mmu_helper.c into mmu.c...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#49: 
deleted file mode 100644

total: 0 errors, 1 warnings, 23 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 14/23: target/openrisc: Reduce tlb to a single dimension...
Checking PATCH 15/23: target/openrisc: Fix tlb flushing in mtspr...
Checking PATCH 16/23: target/openrisc: Fix cpu_mmu_index...
Checking PATCH 17/23: target/openrisc: Use identical sizes for ITLB and DTLB...
Checking PATCH 18/23: target/openrisc: Stub out handle_mmu_fault for softmmu...
Checking PATCH 19/23: target/openrisc: Increase the TLB size...
Checking PATCH 20/23: target/openrisc: Reorg tlb lookup...
Checking PATCH 21/23: target/openrisc: Add support in 
scripts/qemu-binfmt-conf.sh...
WARNING: line over 80 characters
#32: FILE: scripts/qemu-binfmt-conf.sh:127:
+or1k_magic='\x7fELF\x01\x02\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02\x00\x5c'

ERROR: line over 90 characters
#33: FILE: scripts/qemu-binfmt-conf.sh:128:
+or1k_mask='\xff\xff\xff\xff\xff\xff\xff\x00\xff\xff\xff\xff\xff\xff\xff\xff\xff\xfe\xff\xff'

total: 1 errors, 1 warnings, 23 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 22/23: linux-user: Implement signals for openrisc...
Checking PATCH 23/23: linux-user: Fix struct sigaltstack for openrisc...
=== OUTPUT END ===

Test command exited with code: 1


---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to address@hidden