Message-id: 20190514061458.125225-1-ysato@users.sourceforge.jp Type: series Subject: [Qemu-devel] [PATCH v12 00/12] Add RX archtecture 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 * [new tag] patchew/20190514061458.125225-1-ysato@users.sourceforge.jp -> patchew/20190514061458.125225-1-ysato@users.sourceforge.jp Switched to a new branch 'test' 12830ba804 MAINTAINERS: Add RX 3ea6789003 qemu/bitops.h: Add extract8 and extract16 29c7c8ab07 hw/registerfields.h: Add 8bit and 16bit register macros. 3a56cd61de Add rx-softmmu f76f967d52 hw/rx: RX Target hardware definition b4f6bd0b28 hw/char: RX62N serial communication interface (SCI) fcba250528 hw/timer: RX62N internal timer modules 986ac86aed hw/intc: RX62N interrupt controller (ICUa) 3de4d6a599 target/rx: RX disassembler 2914d6d902 target/rx: CPU definition 3ed2f8ac67 target/rx: TCG helper aa8a665253 target/rx: TCG translation === OUTPUT BEGIN === 1/12 Checking commit aa8a665253ef (target/rx: TCG translation) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #15: new file mode 100644 total: 0 errors, 1 warnings, 3065 lines checked Patch 1/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 2/12 Checking commit 3ed2f8ac67f6 (target/rx: TCG helper) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #11: new file mode 100644 total: 0 errors, 1 warnings, 660 lines checked Patch 2/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 3/12 Checking commit 2914d6d90296 (target/rx: CPU definition) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #11: new file mode 100644 ERROR: code indent should never use tabs #380: FILE: target/rx/cpu.h:137: + qemu_irq ack;^I^I/* Interrupt acknowledge */$ total: 1 errors, 1 warnings, 599 lines checked Patch 3/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 4/12 Checking commit 3de4d6a5996e (target/rx: RX disassembler) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #34: new file mode 100644 total: 0 errors, 1 warnings, 1497 lines checked Patch 4/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 5/12 Checking commit 986ac86aed6a (hw/intc: RX62N interrupt controller (ICUa)) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #35: new file mode 100644 ERROR: code indent should never use tabs #430: FILE: include/hw/intc/rx_icu.h:9: + TRG_NEDGE = 1,^I/* Falling */$ ERROR: code indent should never use tabs #431: FILE: include/hw/intc/rx_icu.h:10: + TRG_PEDGE = 2,^I/* Raising */$ ERROR: code indent should never use tabs #432: FILE: include/hw/intc/rx_icu.h:11: + TRG_BEDGE = 3,^I/* Both */$ total: 3 errors, 1 warnings, 443 lines checked Patch 5/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/12 Checking commit fcba2505282c (hw/timer: RX62N internal timer modules) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #45: new file mode 100644 total: 0 errors, 1 warnings, 836 lines checked Patch 6/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/12 Checking commit b4f6bd0b28ee (hw/char: RX62N serial communication interface (SCI)) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #38: new file mode 100644 total: 0 errors, 1 warnings, 398 lines checked Patch 7/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 8/12 Checking commit f76f967d52bb (hw/rx: RX Target hardware definition) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #14: new file mode 100644 total: 0 errors, 1 warnings, 460 lines checked Patch 8/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 9/12 Checking commit 3a56cd61debb (Add rx-softmmu) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #50: new file mode 100644 total: 0 errors, 1 warnings, 45 lines checked Patch 9/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Use of uninitialized value in concatenation (.) or string at ./scripts/checkpatch.pl line 2466. 10/12 Checking commit 29c7c8ab076b (hw/registerfields.h: Add 8bit and 16bit register macros.) ERROR: Macros with multiple statements should be enclosed in a do - while loop #21: FILE: include/hw/registerfields.h:25: +#define REG8(reg, addr) \ + enum { A_ ## reg = (addr) }; \ + enum { R_ ## reg = (addr) }; ERROR: Macros with multiple statements should be enclosed in a do - while loop #25: FILE: include/hw/registerfields.h:29: +#define REG16(reg, addr) \ + enum { A_ ## reg = (addr) }; \ + enum { R_ ## reg = (addr) / 2 }; total: 2 errors, 0 warnings, 56 lines checked Patch 10/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 11/12 Checking commit 3ea6789003e6 (qemu/bitops.h: Add extract8 and extract16) 12/12 Checking commit 12830ba8040d (MAINTAINERS: Add RX) === OUTPUT END === Test command exited with code: 1