Patchew URL: https://patchew.org/QEMU/20191028181643.5143-1-laur...@vivier.eu/
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PULL 00/11] Q800 branch patches Type: series Message-id: 20191028181643.5143-1-laur...@vivier.eu === 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 === Switched to a new branch 'test' 0aafe33 BootLinuxConsoleTest: Test the Quadra 800 6578d05 hw/m68k: define Macintosh Quadra 800 8d92411 hw/m68k: add a dummy SWIM floppy controller 33e127e hw/m68k: add Nubus macfb video card 9e5413f hw/m68k: add Nubus support 777b85a hw/m68k: implement ADB bus support for via d23cb98 hw/m68k: add VIA support 477391a dp8393x: manage big endian bus a5915a5 esp: add pseudo-DMA as used by Macintosh 324dd11 esp: move get_cmd() post-DMA code to get_cmd_cb() 447e0b9 esp: move handle_ti_cmd() cleanup code to esp_do_dma(). === OUTPUT BEGIN === 1/11 Checking commit 447e0b9e7e82 (esp: move handle_ti_cmd() cleanup code to esp_do_dma().) 2/11 Checking commit 324dd11bd843 (esp: move get_cmd() post-DMA code to get_cmd_cb()) 3/11 Checking commit a5915a587841 (esp: add pseudo-DMA as used by Macintosh) 4/11 Checking commit 477391a3299a (dp8393x: manage big endian bus) 5/11 Checking commit d23cb98ca175 (hw/m68k: add VIA support) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #84: new file mode 100644 ERROR: space prohibited after that '&&' (ctx:WxW) #456: FILE: hw/misc/mac_via.c:368: + if (!(v1s->last_b & VIA1B_vRTCClk) && (s->b & VIA1B_vRTCClk)) { ^ total: 1 errors, 1 warnings, 912 lines checked Patch 5/11 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/11 Checking commit 777b85a4403d (hw/m68k: implement ADB bus support for via) 7/11 Checking commit 9e5413fe652c (hw/m68k: add Nubus support) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #65: new file mode 100644 total: 0 errors, 1 warnings, 531 lines checked Patch 7/11 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 8/11 Checking commit 33e127ea8c90 (hw/m68k: add Nubus macfb video card) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #76: new file mode 100644 total: 0 errors, 1 warnings, 597 lines checked Patch 8/11 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 9/11 Checking commit 8d9241120f8f (hw/m68k: add a dummy SWIM floppy controller) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #63: new file mode 100644 total: 0 errors, 1 warnings, 593 lines checked Patch 9/11 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 10/11 Checking commit 6578d05b2deb (hw/m68k: define Macintosh Quadra 800) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #72: new file mode 100644 total: 0 errors, 1 warnings, 537 lines checked Patch 10/11 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 11/11 Checking commit 0aafe3328f0b (BootLinuxConsoleTest: Test the Quadra 800) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20191028181643.5143-1-laur...@vivier.eu/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com