Patchew URL: https://patchew.org/QEMU/[email protected]/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: [email protected] Subject: [Qemu-devel] [PATCH v3 00/16] s390: vfio-ccw dasd ipl support === 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 t [tag update] patchew/[email protected] -> patchew/[email protected] Switched to a new branch 'test' 73739b0cc6 s390-bios: dasd-ipl: Use control unit type to customize error data 8733330492 s390-bios: Support booting from real dasd device 5a5c4b01ef s390-bios: Add channel command codes/structs needed for dasd-ipl d23ff47a3d s390-bios: Use control unit type to determine boot method 2af24581b4 s390-bios: Refactor virtio to run channel programs via cio 631002c644 s390-bios: cio error handling 44d5259870 s390-bios: Support for running format-0/1 channel programs 7244df5a79 s390-bios: ptr2u32 and u32toptr af222f5618 s390-bios: Map low core memory 281f64b37a s390-bios: Decouple channel i/o logic from virtio 4ac96c2c10 s390-bios: Clean up cio.h 06ea714695 s390-bios: Factor finding boot device out of virtio code path a99947c33d s390-bios: Extend find_dev() for non-virtio devices 9b82cabb3b s390-bios: decouple common boot logic from virtio 2e4c88a8ae s390-bios: decouple cio setup from virtio 648c3648c7 s390 vfio-ccw: Add bootindex property and IPLB data === OUTPUT BEGIN === 1/16 Checking commit 648c3648c73b (s390 vfio-ccw: Add bootindex property and IPLB data) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #205: new file mode 100644 total: 0 errors, 1 warnings, 181 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 2e4c88a8ae84 (s390-bios: decouple cio setup from virtio) 3/16 Checking commit 9b82cabb3b40 (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 a99947c33d4a (s390-bios: Extend find_dev() for non-virtio devices) 5/16 Checking commit 06ea71469510 (s390-bios: Factor finding boot device out of virtio code path) 6/16 Checking commit 4ac96c2c105e (s390-bios: Clean up cio.h) 7/16 Checking commit 281f64b37a69 (s390-bios: Decouple channel i/o logic from virtio) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #28: new file mode 100644 total: 0 errors, 1 warnings, 122 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 af222f561810 (s390-bios: Map low core memory) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #35: new file mode 100644 total: 0 errors, 1 warnings, 116 lines checked Patch 8/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 9/16 Checking commit 7244df5a79fa (s390-bios: ptr2u32 and u32toptr) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #16: new file mode 100644 total: 0 errors, 1 warnings, 16 lines checked Patch 9/16 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 10/16 Checking commit 44d52598706f (s390-bios: Support for running format-0/1 channel programs) 11/16 Checking commit 631002c64417 (s390-bios: cio error handling) 12/16 Checking commit 2af24581b441 (s390-bios: Refactor virtio to run channel programs via cio) WARNING: line over 80 characters #103: 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 #116: 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, 107 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 d23ff47a3d3c (s390-bios: Use control unit type to determine boot method) 14/16 Checking commit 5a5c4b01efbc (s390-bios: Add channel command codes/structs needed for dasd-ipl) 15/16 Checking commit 8733330492f4 (s390-bios: Support booting from real dasd device) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #25: new file mode 100644 total: 0 errors, 1 warnings, 446 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 73739b0cc6e1 (s390-bios: dasd-ipl: Use control unit type to customize error data) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/[email protected]/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to [email protected]
