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