Patchew URL: https://patchew.org/QEMU/20181213040126.6768-1-da...@gibson.dropbear.id.au/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20181213040126.6768-1-da...@gibson.dropbear.id.au Subject: [Qemu-devel] [PULL 00/27] ppc-for-4.0 queue 20181213 === 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' ad20d15 spapr/xive: use the VCPU id as a NVT identifier 936bcd5 spapr/xive: introduce a XIVE interrupt controller c13abc0 ppc/xive: notify the CPU when the interrupt priority is more privileged 58a0735 ppc/xive: introduce a simplified XIVE presenter f463196 ppc/xive: introduce the XIVE interrupt thread context b5b9c85 ppc/xive: add support for the END Event State Buffers 11d0460 Changes requirement for "vsubsbs" instruction 37e7503 spapr: export and rename the xics_max_server_number() routine 4d7a081 spapr: introduce a spapr_irq_init() routine 22e14d8 spapr: initialize VSMT before initializing the IRQ backend e85821f ppc/xive: introduce the XIVE Event Notification Descriptors 37adf84 ppc/xive: introduce the XiveRouter model b17c216 ppc/xive: introduce the XiveNotifier interface e9aa0c9 ppc/xive: add support for the LSI interrupt sources afa26a0 ppc/xive: introduce a XIVE interrupt source model 4052b1a e500: simplify IRQ wiring f01edd5 mac_newworld: simplify IRQ wiring dd09168 virtex_ml507: use g_new(T, n) instead of g_malloc(sizeof(T) * n) 6666001 sam460ex: use g_new(T, n) instead of g_malloc(sizeof(T) * n) ae12a4c ppc440_bamboo: use g_new(T, n) instead of g_malloc(sizeof(T) * n) 0126aee ppc405_uc: use g_new(T, n) instead of g_malloc(sizeof(T) * n) f9a1d51 ppc405_boards: use g_new(T, n) instead of g_malloc(sizeof(T) * n) 1dc1282 spapr: use g_new(T, n) instead of g_malloc(sizeof(T) * n) 61eb9c6 target/ppc: use g_new(T, n) instead of g_malloc(sizeof(T) * n) db6492f spapr: drop redundant statement in spapr_populate_drconf_memory() 61d6352 target/ppc: tcg: Implement addex instruction 17ee949 spapr: Fix ibm, max-associativity-domains property number of nodes === OUTPUT BEGIN === Checking PATCH 1/27: spapr: Fix ibm, max-associativity-domains property number of nodes... Checking PATCH 2/27: target/ppc: tcg: Implement addex instruction... ERROR: space required after that ',' (ctx:VxV) #42: FILE: disas/ppc.c:3737: +{ "addex", XO(31,170,0,0), XO_MASK, POWER9, { RT, RA, RB } }, ^ ERROR: space required after that ',' (ctx:VxV) #42: FILE: disas/ppc.c:3737: +{ "addex", XO(31,170,0,0), XO_MASK, POWER9, { RT, RA, RB } }, ^ ERROR: space required after that ',' (ctx:VxV) #42: FILE: disas/ppc.c:3737: +{ "addex", XO(31,170,0,0), XO_MASK, POWER9, { RT, RA, RB } }, ^ total: 3 errors, 0 warnings, 156 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/27: spapr: drop redundant statement in spapr_populate_drconf_memory()... Checking PATCH 4/27: target/ppc: use g_new(T, n) instead of g_malloc(sizeof(T) * n)... Checking PATCH 5/27: spapr: use g_new(T, n) instead of g_malloc(sizeof(T) * n)... Checking PATCH 6/27: ppc405_boards: use g_new(T, n) instead of g_malloc(sizeof(T) * n)... Checking PATCH 7/27: ppc405_uc: use g_new(T, n) instead of g_malloc(sizeof(T) * n)... Checking PATCH 8/27: ppc440_bamboo: use g_new(T, n) instead of g_malloc(sizeof(T) * n)... Checking PATCH 9/27: sam460ex: use g_new(T, n) instead of g_malloc(sizeof(T) * n)... Checking PATCH 10/27: virtex_ml507: use g_new(T, n) instead of g_malloc(sizeof(T) * n)... Checking PATCH 11/27: mac_newworld: simplify IRQ wiring... Checking PATCH 12/27: e500: simplify IRQ wiring... Checking PATCH 13/27: ppc/xive: introduce a XIVE interrupt source model... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #61: new file mode 100644 total: 0 errors, 1 warnings, 656 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/27: ppc/xive: add support for the LSI interrupt sources... Checking PATCH 15/27: ppc/xive: introduce the XiveNotifier interface... Checking PATCH 16/27: ppc/xive: introduce the XiveRouter model... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #170: new file mode 100644 total: 0 errors, 1 warnings, 179 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 17/27: ppc/xive: introduce the XIVE Event Notification Descriptors... Checking PATCH 18/27: spapr: initialize VSMT before initializing the IRQ backend... Checking PATCH 19/27: spapr: introduce a spapr_irq_init() routine... Checking PATCH 20/27: spapr: export and rename the xics_max_server_number() routine... Checking PATCH 21/27: Changes requirement for "vsubsbs" instruction... Checking PATCH 22/27: ppc/xive: add support for the END Event State Buffers... Checking PATCH 23/27: ppc/xive: introduce the XIVE interrupt thread context... Checking PATCH 24/27: ppc/xive: introduce a simplified XIVE presenter... Checking PATCH 25/27: ppc/xive: notify the CPU when the interrupt priority is more privileged... Checking PATCH 26/27: spapr/xive: introduce a XIVE interrupt controller... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #59: new file mode 100644 total: 0 errors, 1 warnings, 425 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 27/27: spapr/xive: use the VCPU id as a NVT identifier... === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20181213040126.6768-1-da...@gibson.dropbear.id.au/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-de...@redhat.com