Patchew URL: https://patchew.org/QEMU/1554388475-18329-1-git-send-email-jjhe...@linux.ibm.com/
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 1554388475-18329-1-git-send-email-jjhe...@linux.ibm.com Subject: [Qemu-devel] [PATCH v6 00/16] s390: vfio-ccw dasd ipl support Type: series === 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/1554388475-18329-1-git-send-email-jjhe...@linux.ibm.com -> patchew/1554388475-18329-1-git-send-email-jjhe...@linux.ibm.com t [tag update] patchew/20190402161900.7374-1-arm...@redhat.com -> patchew/20190402161900.7374-1-arm...@redhat.com t [tag update] patchew/20190404091725.20595-1-dgilb...@redhat.com -> patchew/20190404091725.20595-1-dgilb...@redhat.com * [new tag] patchew/20190404112953.4058-1-be...@igalia.com -> patchew/20190404112953.4058-1-be...@igalia.com * [new tag] patchew/20190404121015.28634-1...@kamp.de -> patchew/20190404121015.28634-1...@kamp.de Switched to a new branch 'test' e794c6367e s390-bios: Use control unit type to find bootable devices aea63e7dfb s390-bios: Support booting from real dasd device 8f7c6a5a42 s390-bios: Add channel command codes/structs needed for dasd-ipl dd95e5ec50 s390-bios: Use control unit type to determine boot method e408cfc1f1 s390-bios: Refactor virtio to run channel programs via cio 8357bd089e s390-bios: Factor finding boot device out of virtio code path 05d2aa89c9 s390-bios: Extend find_dev() for non-virtio devices 54e72ca6e1 s390-bios: cio error handling 9577eecec1 s390-bios: Support for running format-0/1 channel programs 1274b880cc s390-bios: ptr2u32 and u32toptr 63ce0ae260 s390-bios: Map low core memory 995578a87b s390-bios: Decouple channel i/o logic from virtio c0c43849b8 s390-bios: Clean up cio.h 1b82e3b63f s390-bios: decouple common boot logic from virtio 88ddf9dcb2 s390-bios: decouple cio setup from virtio 1357e39998 s390 vfio-ccw: Add bootindex property and IPLB data === OUTPUT BEGIN === 1/16 Checking commit 1357e399982b (s390 vfio-ccw: Add bootindex property and IPLB data) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #222: new file mode 100644 total: 0 errors, 1 warnings, 199 lines checked Patch 1/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 2/16 Checking commit 88ddf9dcb222 (s390-bios: decouple cio setup from virtio) 3/16 Checking commit 1b82e3b63f0f (s390-bios: decouple common boot logic from virtio) ERROR: externs should be avoided in .c files #31: FILE: pc-bios/s390-ccw/main.c:19: +IplParameterBlock iplb __attribute__((__aligned__(PAGE_SIZE))); total: 1 errors, 0 warnings, 65 lines checked Patch 3/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 4/16 Checking commit c0c43849b8a6 (s390-bios: Clean up cio.h) 5/16 Checking commit 995578a87b7b (s390-bios: Decouple channel i/o logic from virtio) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #30: new file mode 100644 total: 0 errors, 1 warnings, 123 lines checked Patch 5/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/16 Checking commit 63ce0ae260c6 (s390-bios: Map low core memory) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #37: new file mode 100644 total: 0 errors, 1 warnings, 104 lines checked Patch 6/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/16 Checking commit 1274b880cc60 (s390-bios: ptr2u32 and u32toptr) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #17: new file mode 100644 total: 0 errors, 1 warnings, 31 lines checked Patch 7/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 8/16 Checking commit 9577eecec1f6 (s390-bios: Support for running format-0/1 channel programs) 9/16 Checking commit 54e72ca6e1dc (s390-bios: cio error handling) 10/16 Checking commit 05d2aa89c93e (s390-bios: Extend find_dev() for non-virtio devices) 11/16 Checking commit 8357bd089e30 (s390-bios: Factor finding boot device out of virtio code path) 12/16 Checking commit e408cfc1f1ce (s390-bios: Refactor virtio to run channel programs via cio) WARNING: line over 80 characters #105: FILE: pc-bios/s390-ccw/virtio.c:298: + run_ccw(vdev, CCW_CMD_READ_VQ_CONF, &config, sizeof(config), false) == 0, WARNING: line over 80 characters #118: FILE: pc-bios/s390-ccw/virtio.c:308: + run_ccw(vdev, CCW_CMD_WRITE_STATUS, &status, sizeof(status), false) == 0, total: 0 errors, 2 warnings, 115 lines checked Patch 12/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 13/16 Checking commit dd95e5ec5028 (s390-bios: Use control unit type to determine boot method) 14/16 Checking commit 8f7c6a5a42f6 (s390-bios: Add channel command codes/structs needed for dasd-ipl) 15/16 Checking commit aea63e7dfb1f (s390-bios: Support booting from real dasd device) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #26: new file mode 100644 total: 0 errors, 1 warnings, 433 lines checked Patch 15/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 16/16 Checking commit e794c6367eaa (s390-bios: Use control unit type to find bootable devices) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/1554388475-18329-1-git-send-email-jjhe...@linux.ibm.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com