Hi, This series seems to have some coding style problems. See output below for more information:
Type: series Message-id: 20180228203243.1413-1-mark.cave-ayl...@ilande.co.uk Subject: [Qemu-devel] [PATCHv3 00/12] macio: remove legacy macio_init() function === 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/1519709965-29833-1-git-send-email-c...@braap.org -> patchew/1519709965-29833-1-git-send-email-c...@braap.org * [new tag] patchew/20180228203243.1413-1-mark.cave-ayl...@ilande.co.uk -> patchew/20180228203243.1413-1-mark.cave-ayl...@ilande.co.uk Switched to a new branch 'test' 172de12fd3 macio: remove macio_init() function 0e9b5f92bd macio: move setting of CUDA timebase frequency to macio_common_realize() cb79b98479 mac_newworld: use object link to pass OpenPIC object to macio 9042b9cdfd openpic: move OpenPIC state and related definitions to openpic.h dbb8ecee13 openpic: move KVM-specific declarations into separate openpic_kvm.h file 62cc211353 mac_oldworld: use object link to pass heathrow PIC object to macio 1fe36332f8 macio: move macio related structures and defines into separate macio.h file 2e9a6f0518 heathrow: change heathrow_pic_init() to return the heathrow device 6dbf45d8e3 heathrow: convert to trace-events b109c28719 heathrow: QOMify heathrow PIC 5ee002a59c macio: move ESCC device within the macio device 4d426b15fd macio: embed DBDMA device directly within macio === OUTPUT BEGIN === Checking PATCH 1/12: macio: embed DBDMA device directly within macio... Checking PATCH 2/12: macio: move ESCC device within the macio device... Checking PATCH 3/12: heathrow: QOMify heathrow PIC... Checking PATCH 4/12: heathrow: convert to trace-events... Checking PATCH 5/12: heathrow: change heathrow_pic_init() to return the heathrow device... Checking PATCH 6/12: macio: move macio related structures and defines into separate macio.h file... Checking PATCH 7/12: mac_oldworld: use object link to pass heathrow PIC object to macio... Checking PATCH 8/12: openpic: move KVM-specific declarations into separate openpic_kvm.h file... Checking PATCH 9/12: openpic: move OpenPIC state and related definitions to openpic.h... ERROR: "foo * bar" should be "foo *bar" #249: FILE: include/hw/ppc/openpic.h:57: +#define RAVEN_DBL_IRQ (RAVEN_IPI_IRQ + (RAVEN_MAX_CPU * RAVEN_MAX_IPI)) total: 1 errors, 0 warnings, 353 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 10/12: mac_newworld: use object link to pass OpenPIC object to macio... Checking PATCH 11/12: macio: move setting of CUDA timebase frequency to macio_common_realize()... Checking PATCH 12/12: macio: remove macio_init() function... === 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