Hi, This series seems to have some coding style problems. See output below for more information:
Message-id: 20170426070034.10727-1-da...@gibson.dropbear.id.au Subject: [Qemu-devel] [PULL 00/48] ppc-for-2.10 queue 20170426 Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 # Useful git options git config --local diff.renamelimit 0 git config --local diff.renames True 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 - [tag update] patchew/1493179907-22516-1-git-send-email-th...@redhat.com -> patchew/1493179907-22516-1-git-send-email-th...@redhat.com - [tag update] patchew/20170426073247.7441-2-quint...@redhat.com -> patchew/20170426073247.7441-2-quint...@redhat.com Switched to a new branch 'test' c14ea42 MAINTAINERS: Remove myself from e500 cda1bcc target/ppc: Style fixes 2687a22 e500, book3s: mfspr 259: Register mapped/aliased SPRG3 user read 4d80bea target/ppc: Flush TLB on write to PIDR 803a452 spapr-cpu-core: Release ICPState object during CPU unrealization dad1411 ppc/pnv: generate an OEM SEL event on shutdown 81fc8dc ppc/pnv: add initial IPMI sensors for the BMC simulator 59b6269 ppc/pnv: populate device tree for IPMI BT devices 9631be0 ppc/pnv: populate device tree for serial devices 0aa29fe ppc/pnv: populate device tree for RTC devices 38cf1ba ppc/pnv: scan ISA bus to populate device tree 77a98a6 ppc/pnv: enable only one LPC bus a4448e0 ppc/pnv: Add support for POWER8+ LPC Controller 052c6f8 spapr: remove the 'nr_servers' field from the machine 930053d target/ppc: Fix size of struct PPCElfPrstatus 75df6b3 ipmi: introduce an ipmi_bmc_gen_event() API 235eb1b ipmi: introduce an ipmi_bmc_sdr_find() API 6c4b8b9 ipmi: provide support for FRUs 375940e ipmi: use a file to load SDRs 5d70630 ppc: add IPMI support 8e07169 ppc/pnv: Add OCC model stub with interrupt support 8f049fd ppc/pnv: Add cut down PSI bridge model and hookup external interrupt 8b50fc4 ppc/pnv: add memory regions for the ICP registers afea407 ppc/pnv: add a helper to calculate MMIO addresses registers 053cc96 ppc/pnv: create the ICP object under PnvCore 77454c6 ppc/pnv: extend the machine with a InterruptStatsProvider interface 27496fd ppc/pnv: extend the machine with a XICSFabric interface cb60b41 ppc/pnv: add a PnvICPState object b3e42b8 ppc/xics: add a realize() handler to ICPStateClass 43fa3d7 spapr: allocate the ICPState object from under sPAPRCPUCore 55519e0 spapr: move the IRQ server number mapping under the machine 59bca08 ppc/xics: introduce an 'intc' backlink under PowerPCCPU caae2a9 target/ppc: Add ibm, processor-radix-AP-encodings for TCG 4469965 spapr_pci: Removed unused include dfa9faf spapr_pci: Warn when RAM page size is not enabled in IOMMU page mask 05faf4f target-ppc/kvm: Enable in-kernel TCE acceleration for multi-tce 808e8c1 spapr: Workaround for broken radix guests 7c423a7 spapr: Enable ISA 3.0 MMU mode selection via CAS 17d2c76 spapr: move spapr_populate_pa_features() a9df275 target/ppc: Implement H_REGISTER_PROCESS_TABLE H_CALL b3a9436 target/ppc: Add new H-CALL shells for in memory table translation 29faef3 target-ppc: support KVM_CAP_PPC_MMU_RADIX, KVM_CAP_PPC_MMU_HASH_V3 b8e28f8 spapr: Add ibm, processor-radix-AP-encodings to the device tree e8e32b8 target-ppc: kvm: make use of KVM_CREATE_SPAPR_TCE_64 b2f1c56 hw/ppc/pnv: Classify the "PowerNV Chip" devices as CPU devices 4d69a70 ppc/spapr: QOM'ify sPAPRRTCState d7091e1 pseries: Add pseries-2.10 machine type 1bc859f target/ppc: Improve accuracy of guest HTM availability on P8s === OUTPUT BEGIN === Checking PATCH 1/48: target/ppc: Improve accuracy of guest HTM availability on P8s... Checking PATCH 2/48: pseries: Add pseries-2.10 machine type... Checking PATCH 3/48: ppc/spapr: QOM'ify sPAPRRTCState... Checking PATCH 4/48: hw/ppc/pnv: Classify the "PowerNV Chip" devices as CPU devices... Checking PATCH 5/48: target-ppc: kvm: make use of KVM_CREATE_SPAPR_TCE_64... Checking PATCH 6/48: spapr: Add ibm, processor-radix-AP-encodings to the device tree... Checking PATCH 7/48: target-ppc: support KVM_CAP_PPC_MMU_RADIX, KVM_CAP_PPC_MMU_HASH_V3... Checking PATCH 8/48: target/ppc: Add new H-CALL shells for in memory table translation... Checking PATCH 9/48: target/ppc: Implement H_REGISTER_PROCESS_TABLE H_CALL... Checking PATCH 10/48: spapr: move spapr_populate_pa_features()... Checking PATCH 11/48: spapr: Enable ISA 3.0 MMU mode selection via CAS... Checking PATCH 12/48: spapr: Workaround for broken radix guests... Checking PATCH 13/48: target-ppc/kvm: Enable in-kernel TCE acceleration for multi-tce... Checking PATCH 14/48: spapr_pci: Warn when RAM page size is not enabled in IOMMU page mask... Checking PATCH 15/48: spapr_pci: Removed unused include... Checking PATCH 16/48: target/ppc: Add ibm, processor-radix-AP-encodings for TCG... Checking PATCH 17/48: ppc/xics: introduce an 'intc' backlink under PowerPCCPU... Checking PATCH 18/48: spapr: move the IRQ server number mapping under the machine... Checking PATCH 19/48: spapr: allocate the ICPState object from under sPAPRCPUCore... Checking PATCH 20/48: ppc/xics: add a realize() handler to ICPStateClass... Checking PATCH 21/48: ppc/pnv: add a PnvICPState object... Checking PATCH 22/48: ppc/pnv: extend the machine with a XICSFabric interface... Checking PATCH 23/48: ppc/pnv: extend the machine with a InterruptStatsProvider interface... Checking PATCH 24/48: ppc/pnv: create the ICP object under PnvCore... Checking PATCH 25/48: ppc/pnv: add a helper to calculate MMIO addresses registers... Checking PATCH 26/48: ppc/pnv: add memory regions for the ICP registers... Checking PATCH 27/48: ppc/pnv: Add cut down PSI bridge model and hookup external interrupt... Checking PATCH 28/48: ppc/pnv: Add OCC model stub with interrupt support... Checking PATCH 29/48: ppc: add IPMI support... Checking PATCH 30/48: ipmi: use a file to load SDRs... Checking PATCH 31/48: ipmi: provide support for FRUs... Checking PATCH 32/48: ipmi: introduce an ipmi_bmc_sdr_find() API... Checking PATCH 33/48: ipmi: introduce an ipmi_bmc_gen_event() API... Checking PATCH 34/48: target/ppc: Fix size of struct PPCElfPrstatus... Checking PATCH 35/48: spapr: remove the 'nr_servers' field from the machine... Checking PATCH 36/48: ppc/pnv: Add support for POWER8+ LPC Controller... Checking PATCH 37/48: ppc/pnv: enable only one LPC bus... Checking PATCH 38/48: ppc/pnv: scan ISA bus to populate device tree... Checking PATCH 39/48: ppc/pnv: populate device tree for RTC devices... Checking PATCH 40/48: ppc/pnv: populate device tree for serial devices... Checking PATCH 41/48: ppc/pnv: populate device tree for IPMI BT devices... Checking PATCH 42/48: ppc/pnv: add initial IPMI sensors for the BMC simulator... Checking PATCH 43/48: ppc/pnv: generate an OEM SEL event on shutdown... Checking PATCH 44/48: spapr-cpu-core: Release ICPState object during CPU unrealization... Checking PATCH 45/48: target/ppc: Flush TLB on write to PIDR... Checking PATCH 46/48: e500, book3s: mfspr 259: Register mapped/aliased SPRG3 user read... ERROR: space prohibited between function name and open parenthesis '(' #51: FILE: target/ppc/translate_init.c:1643: +static void gen_spr_usprg3 (CPUPPCState *env) total: 1 errors, 0 warnings, 28 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 47/48: target/ppc: Style fixes... Checking PATCH 48/48: MAINTAINERS: Remove myself from e500... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-de...@freelists.org