mbox series

[00/15] *** ARC port for review ***

Message ID 20201111161758.9636-1-cupertinomiranda@gmail.com
Headers show
Series *** ARC port for review *** | expand

Message

Cupertino Miranda Nov. 11, 2020, 4:17 p.m. UTC
From: Cupertino Miranda <cmiranda@synopsys.com>

Hello everyone,

Here is a long due refresh of ARCv2 port patches.

List of done requested improvements:
 - Include acceptance tests for ARC (linux boot),
 - Copyright notices,
 - Remove osdep.h from header files,
 - Removed makefiles,
 - Cleared list of boards reducing for just 2 boards

Things requested but not done: 
 - f4bug: we did not change any of the mentioned variable types since those
   variables are used in global properties, and the property definition
   seem to enforce a size and signess, when using DEFINE_PROP_UNSIGNED.

Best regards,
Cupertino Miranda


| Hello everyone,
| 
| It is with utmost pleasure that on behalf of Synopsys I initiate the
| upstream process for our ARCv2 architecture QEMU port.
| 
| I am one of the main contributors to this port and member of the
| Synopsys GNU toolchain team, together with Claudiu Zissulescu and 
| Shahab Vahedi.
| 
| ARCv2 processors use RISC, and employ the 16-/32-bit instruction set
| architecture (ISA).
| ARC processors are configurable and extensible for a wide range of uses
| in system on a chip (SoC) devices.
| It is a common processor in deeply embedded systems, used in storage,
| digital home, mobile, automotive, and Internet of Things (IoT) 
| applications.
| 
| To facilitate reviewing, we split the files through several commits,
| attempting to keep file size to reviewable chunks.
| 
| The port as presented implements the system softmmu hardware emulation,
| capable to boot and run Linux kernel.
| It has also been integrated as a patch in Zephyr project, where it is
| being used.
| To validate and guarantee correctness of the port through development, 
| we have created TCG tests from the very beggining, covering the most 
| fundamental CPU features.
| For the remaining instructions we validate the port through compiler 
| testsuites, like DejaGNU, comparing results with real hardware 
| executions.
| 
| Would like also to thank our colleagues Vineet Gupta and Alexey Broadkin
| for their contributions, which due to the collapse of the development
| commits, got their contributions overcast by major ones.
| 
| Link for Synopsys ARC processor page:
|  - https://www.synopsys.com/designware-ip/processor-solutions.html
| 
| ARC PRM for both HS and EM processors can be found in:
|  - https://www.synopsys.com/dw/doc.php/ds/cc/programmers-reference-manual-ARC-EM.pdf
|  - https://www.synopsys.com/dw/doc.php/ds/cc/programmers-reference-manual-ARC-HS.pdf.
| 
| Looking forward to your comments and suggestions.
| 
| Best regards,
| Cupertino Miranda

Claudiu Zissulescu (5):
  arc: Decoder code
  arc: Opcode definitions table
  arc: Add IRQ and timer subsystem support
  arc: Add Synopsys ARC emulation boards
  tests/tcg: ARC: Add TCG instruction definition tests

Cupertino Miranda (7):
  arc: Add initial core cpu files
  arc: TCG and decoder glue code and helpers
  arc: TCG instruction generator and hand-definitions
  arc: TCG instruction definitions
  arc: Add BCR and AUX registers implementation
  arc: Add memory management unit (MMU) support
  tests/acceptance: ARC: Add linux boot testing.

Shahab Vahedi (3):
  arc: Add memory protection unit (MPU) support
  arc: Add gdbstub and XML for debugging support
  arc: Add support for ARCv2

 configure                               |     2 +
 default-configs/arc-softmmu.mak         |     5 +
 default-configs/devices/arc-softmmu.mak |     7 +
 default-configs/targets/arc-softmmu.mak |     2 +
 disas.c                                 |     2 +
 disas/arc.c                             |   461 +
 disas/meson.build                       |     1 +
 gdb-xml/arc-v2-aux.xml                  |    32 +
 gdb-xml/arc-v2-core.xml                 |    45 +
 gdb-xml/arc-v2-other.xml                |   235 +
 hw/Kconfig                              |     1 +
 hw/arc/Kconfig                          |     7 +
 hw/arc/arc_sim.c                        |   124 +
 hw/arc/boot.c                           |   100 +
 hw/arc/boot.h                           |    21 +
 hw/arc/meson.build                      |     9 +
 hw/arc/pic_cpu.c                        |   113 +
 hw/arc/virt.c                           |   107 +
 hw/meson.build                          |     1 +
 include/disas/dis-asm.h                 |    10 +-
 include/elf.h                           |     3 +
 include/exec/poison.h                   |     2 +
 include/hw/arc/cpudevs.h                |    30 +
 include/sysemu/arch_init.h              |     1 +
 meson.build                             |     5 +-
 softmmu/arch_init.c                     |     2 +
 target/arc/arc-common.h                 |    54 +
 target/arc/cache.c                      |   182 +
 target/arc/cache.h                      |    42 +
 target/arc/cpu-param.h                  |    32 +
 target/arc/cpu-qom.h                    |    52 +
 target/arc/cpu.c                        |   467 +
 target/arc/cpu.h                        |   531 +
 target/arc/decoder.c                    |  1277 ++
 target/arc/decoder.h                    |   349 +
 target/arc/extra_mapping.def            |    40 +
 target/arc/flags.def                    |    85 +
 target/arc/gdbstub.c                    |   420 +
 target/arc/helper.c                     |   293 +
 target/arc/helper.h                     |    46 +
 target/arc/internals.h                  |    38 +
 target/arc/irq.c                        |   658 +
 target/arc/irq.h                        |    37 +
 target/arc/meson.build                  |    20 +
 target/arc/mmu.c                        |   777 +
 target/arc/mmu.h                        |   166 +
 target/arc/mpu.c                        |   656 +
 target/arc/mpu.h                        |   142 +
 target/arc/op_helper.c                  |   749 +
 target/arc/opcodes.def                  | 19976 ++++++++++++++++++++++
 target/arc/operands.def                 |   123 +
 target/arc/regs-detail.def              |   538 +
 target/arc/regs.c                       |   139 +
 target/arc/regs.def                     |   396 +
 target/arc/regs.h                       |   118 +
 target/arc/semfunc-helper.c             |   493 +
 target/arc/semfunc-helper.h             |   280 +
 target/arc/semfunc.c                    |  8473 +++++++++
 target/arc/semfunc.h                    |    62 +
 target/arc/semfunc_mapping.def          |   329 +
 target/arc/timer.c                      |   454 +
 target/arc/timer.h                      |    30 +
 target/arc/translate.c                  |  1345 ++
 target/arc/translate.h                  |   201 +
 target/meson.build                      |     1 +
 tests/Makefile.include                  |     1 +
 tests/acceptance/boot_linux_console.py  |    55 +
 tests/tcg/arc/Makefile                  |   114 +
 tests/tcg/arc/Makefile.softmmu-target   |    43 +
 tests/tcg/arc/Makefile.target           |   101 +
 tests/tcg/arc/check_add.S               |    11 +
 tests/tcg/arc/check_addx.S              |    71 +
 tests/tcg/arc/check_andx.S              |    36 +
 tests/tcg/arc/check_aslx.S              |    57 +
 tests/tcg/arc/check_asrx.S              |    86 +
 tests/tcg/arc/check_basic1.S            |    30 +
 tests/tcg/arc/check_basic2.S            |    26 +
 tests/tcg/arc/check_beq.S               |    14 +
 tests/tcg/arc/check_beqx.S              |    26 +
 tests/tcg/arc/check_bi.S                |    32 +
 tests/tcg/arc/check_big_tb.S            |   173 +
 tests/tcg/arc/check_bih.S               |    29 +
 tests/tcg/arc/check_bnex.S              |    26 +
 tests/tcg/arc/check_breqx.S             |    26 +
 tests/tcg/arc/check_brgex.S             |    26 +
 tests/tcg/arc/check_brhsx.S             |    27 +
 tests/tcg/arc/check_brlox.S             |    26 +
 tests/tcg/arc/check_brltx.S             |    26 +
 tests/tcg/arc/check_brnex.S             |    26 +
 tests/tcg/arc/check_bta.S               |   294 +
 tests/tcg/arc/check_carry.S             |    15 +
 tests/tcg/arc/check_enter_leave.S       |   715 +
 tests/tcg/arc/check_excp.S              |    17 +
 tests/tcg/arc/check_excp_1.c            |    15 +
 tests/tcg/arc/check_excp_jumpdl_mmu.S   |    44 +
 tests/tcg/arc/check_excp_mmu.S          |    69 +
 tests/tcg/arc/check_flags.S             |    23 +
 tests/tcg/arc/check_ldaw_mmu.S          |    71 +
 tests/tcg/arc/check_ldstx.S             |    37 +
 tests/tcg/arc/check_lp.S                |    12 +
 tests/tcg/arc/check_lp02.S              |    72 +
 tests/tcg/arc/check_lp03.S              |    49 +
 tests/tcg/arc/check_lp04.S              |    48 +
 tests/tcg/arc/check_lp05.S              |    23 +
 tests/tcg/arc/check_lp06.S              |   163 +
 tests/tcg/arc/check_lsrx.S              |    33 +
 tests/tcg/arc/check_mac.S               |   228 +
 tests/tcg/arc/check_manip_10_mmu.S      |   173 +
 tests/tcg/arc/check_manip_4_mmu.S       |   158 +
 tests/tcg/arc/check_manip_5_mmu.S       |   166 +
 tests/tcg/arc/check_manip_mmu.S         |   565 +
 tests/tcg/arc/check_mmu.S               |    59 +
 tests/tcg/arc/check_mpu.S               |   703 +
 tests/tcg/arc/check_mpyd.S              |   543 +
 tests/tcg/arc/check_mpyw.S              |    41 +
 tests/tcg/arc/check_norm.S              |    40 +
 tests/tcg/arc/check_orx.S               |    34 +
 tests/tcg/arc/check_prefetch.S          |    37 +
 tests/tcg/arc/check_rolx.S              |    47 +
 tests/tcg/arc/check_rorx.S              |    64 +
 tests/tcg/arc/check_rtc.S               |    29 +
 tests/tcg/arc/check_rtie_user.S         |    30 +
 tests/tcg/arc/check_stld.S              |    10 +
 tests/tcg/arc/check_subf.S              |    67 +
 tests/tcg/arc/check_subx.S              |    43 +
 tests/tcg/arc/check_swi.S               |   115 +
 tests/tcg/arc/check_swirq.S             |    27 +
 tests/tcg/arc/check_swirq1.S            |    31 +
 tests/tcg/arc/check_swirq3.S            |    49 +
 tests/tcg/arc/check_t01.S               |    12 +
 tests/tcg/arc/check_t02.S               |     9 +
 tests/tcg/arc/check_timer0.S            |    36 +
 tests/tcg/arc/check_timer0_loop.S       |    34 +
 tests/tcg/arc/check_timer0_loop3.S      |    46 +
 tests/tcg/arc/check_timer0_retrig.S     |    29 +
 tests/tcg/arc/check_timer0_sleep.S      |    33 +
 tests/tcg/arc/check_timerX_freq.S       |    87 +
 tests/tcg/arc/check_vadd.S              |   510 +
 tests/tcg/arc/check_vsub.S              |   510 +
 tests/tcg/arc/check_xorx.S              |    32 +
 tests/tcg/arc/ivt.S                     |    38 +
 tests/tcg/arc/macros.inc                |   261 +
 tests/tcg/arc/memory.x                  |    12 +
 tests/tcg/arc/mmu.inc                   |   132 +
 tests/tcg/arc/mpu.inc                   |   269 +
 tests/tcg/arc/tarc.ld                   |    15 +
 tests/tcg/arc/tarc_mmu.ld               |    15 +
 tests/tcg/arc/test_macros.h             |   257 +
 tests/tcg/configure.sh                  |     3 +-
 149 files changed, 49782 insertions(+), 4 deletions(-)
 create mode 100644 default-configs/arc-softmmu.mak
 create mode 100644 default-configs/devices/arc-softmmu.mak
 create mode 100644 default-configs/targets/arc-softmmu.mak
 create mode 100644 disas/arc.c
 create mode 100644 gdb-xml/arc-v2-aux.xml
 create mode 100644 gdb-xml/arc-v2-core.xml
 create mode 100644 gdb-xml/arc-v2-other.xml
 create mode 100644 hw/arc/Kconfig
 create mode 100644 hw/arc/arc_sim.c
 create mode 100644 hw/arc/boot.c
 create mode 100644 hw/arc/boot.h
 create mode 100644 hw/arc/meson.build
 create mode 100644 hw/arc/pic_cpu.c
 create mode 100644 hw/arc/virt.c
 create mode 100644 include/hw/arc/cpudevs.h
 create mode 100644 target/arc/arc-common.h
 create mode 100644 target/arc/cache.c
 create mode 100644 target/arc/cache.h
 create mode 100644 target/arc/cpu-param.h
 create mode 100644 target/arc/cpu-qom.h
 create mode 100644 target/arc/cpu.c
 create mode 100644 target/arc/cpu.h
 create mode 100644 target/arc/decoder.c
 create mode 100644 target/arc/decoder.h
 create mode 100644 target/arc/extra_mapping.def
 create mode 100644 target/arc/flags.def
 create mode 100644 target/arc/gdbstub.c
 create mode 100644 target/arc/helper.c
 create mode 100644 target/arc/helper.h
 create mode 100644 target/arc/internals.h
 create mode 100644 target/arc/irq.c
 create mode 100644 target/arc/irq.h
 create mode 100644 target/arc/meson.build
 create mode 100644 target/arc/mmu.c
 create mode 100644 target/arc/mmu.h
 create mode 100644 target/arc/mpu.c
 create mode 100644 target/arc/mpu.h
 create mode 100644 target/arc/op_helper.c
 create mode 100644 target/arc/opcodes.def
 create mode 100644 target/arc/operands.def
 create mode 100644 target/arc/regs-detail.def
 create mode 100644 target/arc/regs.c
 create mode 100644 target/arc/regs.def
 create mode 100644 target/arc/regs.h
 create mode 100644 target/arc/semfunc-helper.c
 create mode 100644 target/arc/semfunc-helper.h
 create mode 100644 target/arc/semfunc.c
 create mode 100644 target/arc/semfunc.h
 create mode 100644 target/arc/semfunc_mapping.def
 create mode 100644 target/arc/timer.c
 create mode 100644 target/arc/timer.h
 create mode 100644 target/arc/translate.c
 create mode 100644 target/arc/translate.h
 create mode 100644 tests/tcg/arc/Makefile
 create mode 100644 tests/tcg/arc/Makefile.softmmu-target
 create mode 100644 tests/tcg/arc/Makefile.target
 create mode 100644 tests/tcg/arc/check_add.S
 create mode 100644 tests/tcg/arc/check_addx.S
 create mode 100644 tests/tcg/arc/check_andx.S
 create mode 100644 tests/tcg/arc/check_aslx.S
 create mode 100644 tests/tcg/arc/check_asrx.S
 create mode 100644 tests/tcg/arc/check_basic1.S
 create mode 100644 tests/tcg/arc/check_basic2.S
 create mode 100644 tests/tcg/arc/check_beq.S
 create mode 100644 tests/tcg/arc/check_beqx.S
 create mode 100644 tests/tcg/arc/check_bi.S
 create mode 100644 tests/tcg/arc/check_big_tb.S
 create mode 100644 tests/tcg/arc/check_bih.S
 create mode 100644 tests/tcg/arc/check_bnex.S
 create mode 100644 tests/tcg/arc/check_breqx.S
 create mode 100644 tests/tcg/arc/check_brgex.S
 create mode 100644 tests/tcg/arc/check_brhsx.S
 create mode 100644 tests/tcg/arc/check_brlox.S
 create mode 100644 tests/tcg/arc/check_brltx.S
 create mode 100644 tests/tcg/arc/check_brnex.S
 create mode 100644 tests/tcg/arc/check_bta.S
 create mode 100644 tests/tcg/arc/check_carry.S
 create mode 100644 tests/tcg/arc/check_enter_leave.S
 create mode 100644 tests/tcg/arc/check_excp.S
 create mode 100644 tests/tcg/arc/check_excp_1.c
 create mode 100644 tests/tcg/arc/check_excp_jumpdl_mmu.S
 create mode 100644 tests/tcg/arc/check_excp_mmu.S
 create mode 100644 tests/tcg/arc/check_flags.S
 create mode 100644 tests/tcg/arc/check_ldaw_mmu.S
 create mode 100644 tests/tcg/arc/check_ldstx.S
 create mode 100644 tests/tcg/arc/check_lp.S
 create mode 100644 tests/tcg/arc/check_lp02.S
 create mode 100644 tests/tcg/arc/check_lp03.S
 create mode 100644 tests/tcg/arc/check_lp04.S
 create mode 100644 tests/tcg/arc/check_lp05.S
 create mode 100644 tests/tcg/arc/check_lp06.S
 create mode 100644 tests/tcg/arc/check_lsrx.S
 create mode 100644 tests/tcg/arc/check_mac.S
 create mode 100644 tests/tcg/arc/check_manip_10_mmu.S
 create mode 100644 tests/tcg/arc/check_manip_4_mmu.S
 create mode 100644 tests/tcg/arc/check_manip_5_mmu.S
 create mode 100644 tests/tcg/arc/check_manip_mmu.S
 create mode 100644 tests/tcg/arc/check_mmu.S
 create mode 100644 tests/tcg/arc/check_mpu.S
 create mode 100644 tests/tcg/arc/check_mpyd.S
 create mode 100644 tests/tcg/arc/check_mpyw.S
 create mode 100644 tests/tcg/arc/check_norm.S
 create mode 100644 tests/tcg/arc/check_orx.S
 create mode 100644 tests/tcg/arc/check_prefetch.S
 create mode 100644 tests/tcg/arc/check_rolx.S
 create mode 100644 tests/tcg/arc/check_rorx.S
 create mode 100644 tests/tcg/arc/check_rtc.S
 create mode 100644 tests/tcg/arc/check_rtie_user.S
 create mode 100644 tests/tcg/arc/check_stld.S
 create mode 100644 tests/tcg/arc/check_subf.S
 create mode 100644 tests/tcg/arc/check_subx.S
 create mode 100644 tests/tcg/arc/check_swi.S
 create mode 100644 tests/tcg/arc/check_swirq.S
 create mode 100644 tests/tcg/arc/check_swirq1.S
 create mode 100644 tests/tcg/arc/check_swirq3.S
 create mode 100644 tests/tcg/arc/check_t01.S
 create mode 100644 tests/tcg/arc/check_t02.S
 create mode 100644 tests/tcg/arc/check_timer0.S
 create mode 100644 tests/tcg/arc/check_timer0_loop.S
 create mode 100644 tests/tcg/arc/check_timer0_loop3.S
 create mode 100644 tests/tcg/arc/check_timer0_retrig.S
 create mode 100644 tests/tcg/arc/check_timer0_sleep.S
 create mode 100644 tests/tcg/arc/check_timerX_freq.S
 create mode 100644 tests/tcg/arc/check_vadd.S
 create mode 100644 tests/tcg/arc/check_vsub.S
 create mode 100644 tests/tcg/arc/check_xorx.S
 create mode 100644 tests/tcg/arc/ivt.S
 create mode 100644 tests/tcg/arc/macros.inc
 create mode 100644 tests/tcg/arc/memory.x
 create mode 100644 tests/tcg/arc/mmu.inc
 create mode 100644 tests/tcg/arc/mpu.inc
 create mode 100644 tests/tcg/arc/tarc.ld
 create mode 100644 tests/tcg/arc/tarc_mmu.ld
 create mode 100644 tests/tcg/arc/test_macros.h

Comments

no-reply@patchew.org Nov. 11, 2020, 4:43 p.m. UTC | #1
Patchew URL: https://patchew.org/QEMU/20201111161758.9636-1-cupertinomiranda@gmail.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Message-id: 20201111161758.9636-1-cupertinomiranda@gmail.com
Subject: [PATCH 00/15] *** ARC port for review ***
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
 - [tag update]      patchew/1605104763-118687-1-git-send-email-zhengchuan@huawei.com -> patchew/1605104763-118687-1-git-send-email-zhengchuan@huawei.com
 * [new tag]         patchew/20201111161758.9636-1-cupertinomiranda@gmail.com -> patchew/20201111161758.9636-1-cupertinomiranda@gmail.com
Switched to a new branch 'test'
dc09f35 tests/acceptance: ARC: Add linux boot testing.
482f8c4 tests/tcg: ARC: Add TCG instruction definition tests
bafbb71 arc: Add support for ARCv2
2fb7984 arc: Add Synopsys ARC emulation boards
6e52aab1 arc: Add gdbstub and XML for debugging support
c052fe6 arc: Add memory protection unit (MPU) support
df0adea arc: Add memory management unit (MMU) support
7f70667 arc: Add IRQ and timer subsystem support
46999ef arc: Add BCR and AUX registers implementation
08846bb arc: TCG instruction definitions
710758d arc: TCG instruction generator and hand-definitions
21c6742 arc: TCG and decoder glue code and helpers
e9b9883 arc: Opcode definitions table
50f88cc arc: Decoder code
0c1f209 arc: Add initial core cpu files

=== OUTPUT BEGIN ===
1/15 Checking commit 0c1f20962105 (arc: Add initial core cpu files)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#11: 
new file mode 100644

WARNING: line over 80 characters
#285: FILE: target/arc/cpu.c:114:
+    DEFINE_PROP_BOOL("slc-datahalf", ARCCPU, cfg.slc_data_halfcycle_steal, false),

WARNING: line over 80 characters
#286: FILE: target/arc/cpu.c:115:
+    DEFINE_PROP_BOOL("slc-dataadd", ARCCPU, cfg.slc_data_add_pre_pipeline, false),

ERROR: Use of volatile is usually wrong, please add a comment
#894: FILE: target/arc/cpu.h:250:
+    volatile uint32_t T_Limit;

ERROR: Use of volatile is usually wrong, please add a comment
#895: FILE: target/arc/cpu.h:251:
+    volatile uint64_t last_clk;

total: 2 errors, 3 warnings, 1194 lines checked

Patch 1/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

2/15 Checking commit 50f88ccf9ad1 (arc: Decoder code)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#13: 
new file mode 100644

ERROR: Don't use '#' flag of printf format ('%#') in format strings, use '0x' prefix instead
#276: FILE: disas/arc.c:259:
+                (*info->fprintf_func)(info->stream, "%#x", value);

ERROR: Don't use '#' flag of printf format ('%#') in format strings, use '0x' prefix instead
#310: FILE: disas/arc.c:293:
+                    (*info->fprintf_func)(info->stream, "%#x", value);

ERROR: "foo * bar" should be "foo *bar"
#1173: FILE: target/arc/decoder.c:689:
+                                    bfd_boolean * invalid  ATTRIBUTE_UNUSED)

ERROR: "foo * bar" should be "foo *bar"
#1179: FILE: target/arc/decoder.c:695:
+                                  bfd_boolean * invalid  ATTRIBUTE_UNUSED)

ERROR: "foo * bar" should be "foo *bar"
#1185: FILE: target/arc/decoder.c:701:
+                                     bfd_boolean * invalid  ATTRIBUTE_UNUSED)

ERROR: "foo * bar" should be "foo *bar"
#1191: FILE: target/arc/decoder.c:707:
+                                   bfd_boolean * invalid  ATTRIBUTE_UNUSED)

ERROR: Macros with complex values should be enclosed in parenthesis
#1253: FILE: target/arc/decoder.c:769:
+#define ARC_OPERAND(NAME, BITS, SHIFT, RELO, FLAGS, FUN) OPERAND_##NAME,

ERROR: Macros with complex values should be enclosed in parenthesis
#1276: FILE: target/arc/decoder.c:792:
+#define ARC_FLAG(NAME, MNEMONIC, CODE, BITS, SHIFT, AVAIL) F_##NAME,

total: 8 errors, 1 warnings, 2295 lines checked

Patch 2/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

3/15 Checking commit e9b9883582ca (arc: Opcode definitions table)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#11: 
new file mode 100644

total: 0 errors, 1 warnings, 19976 lines checked

Patch 3/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/15 Checking commit 21c674282d28 (arc: TCG and decoder glue code and helpers)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#11: 
new file mode 100644

total: 0 errors, 1 warnings, 1457 lines checked

Patch 4/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
5/15 Checking commit 710758dec714 (arc: TCG instruction generator and hand-definitions)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#17: 
new file mode 100644

ERROR: Macros with multiple statements should be enclosed in a do - while loop
#178: FILE: target/arc/translate.c:157:
+#define CC_EPILOGUE          \
+    if (ctx->insn.cc) {      \
+        gen_set_label(done); \
+    }                        \
+    tcg_temp_free(cc)

ERROR: externs should be avoided in .c files
#372: FILE: target/arc/translate.c:351:
+extern bool enabled_interrupts;

ERROR: Macros with complex values should be enclosed in parenthesis
#485: FILE: target/arc/translate.c:464:
+#define MAPPING(MNEMONIC, NAME, NOPS, ...) MAP_##MNEMONIC##_##NAME,

ERROR: Macros with complex values should be enclosed in parenthesis
#498: FILE: target/arc/translate.c:477:
+#define MAPPING(MNEMONIC, NAME, NOPS, ...) NOPS,

ERROR: do not initialise statics to 0 or NULL
#1064: FILE: target/arc/translate.c:1043:
+    static bool initialized = false;

ERROR: Macros with multiple statements should be enclosed in a do - while loop
#1099: FILE: target/arc/translate.c:1078:
+#define MAPPING(MNEMONIC, NAME, NOPS, ...)                              \
+            case MAP_##MNEMONIC##_##NAME:                               \
+                ret = SEMANTIC_FUNCTION_CALL_##NOPS(NAME, __VA_ARGS__); \
+                break;

total: 6 errors, 1 warnings, 1546 lines checked

Patch 5/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

6/15 Checking commit 08846bb66dec (arc: TCG instruction definitions)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#14: 
new file mode 100644

ERROR: externs should be avoided in .c files
#251: FILE: target/arc/semfunc-helper.c:233:
+extern bool enabled_interrupts;

ERROR: do not initialise statics to 0 or NULL
#255: FILE: target/arc/semfunc-helper.c:237:
+    static int in_delay_slot = false;

WARNING: architecture specific defines should be avoided
#9304: FILE: target/arc/semfunc.h:22:
+#ifndef __ARC_SEMFUNC_H__

total: 2 errors, 2 warnings, 9308 lines checked

Patch 6/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

7/15 Checking commit 46999efd8b8a (arc: Add BCR and AUX registers implementation)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#15: 
new file mode 100644

WARNING: architecture specific defines should be avoided
#226: FILE: target/arc/cache.h:19:
+#ifndef __ARC_CACHE_H__

ERROR: Macros with complex values should be enclosed in parenthesis
#858: FILE: target/arc/regs.c:59:
+#define AUX_REG(NAME, GET, SET) #NAME,

ERROR: Macros with complex values should be enclosed in parenthesis
#1390: FILE: target/arc/regs.h:44:
+#define AUX_REG(NAME, GET, SET) AUX_ID_##NAME,

ERROR: Macros with complex values should be enclosed in parenthesis
#1398: FILE: target/arc/regs.h:52:
+#define DEF(NUM, CPU, SUB, NAME) CPU##_##NUM,

total: 3 errors, 2 warnings, 1415 lines checked

Patch 7/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

8/15 Checking commit 7f70667a3510 (arc: Add IRQ and timer subsystem support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#11: 
new file mode 100644

ERROR: do not initialise globals to 0 or NULL
#57: FILE: target/arc/irq.c:42:
+bool enabled_interrupts = false;

WARNING: architecture specific defines should be avoided
#700: FILE: target/arc/irq.h:21:
+#ifndef __IRQ_H__

WARNING: architecture specific defines should be avoided
#1203: FILE: target/arc/timer.h:21:
+#ifndef __ARC_TIMER_H__

total: 1 errors, 3 warnings, 1179 lines checked

Patch 8/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

9/15 Checking commit df0adeadbaee (arc: Add memory management unit (MMU) support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#14: 
new file mode 100644

ERROR: do not initialise statics to 0 or NULL
#680: FILE: target/arc/mmu.c:662:
+    static bool is_initialized = false;

total: 1 errors, 1 warnings, 943 lines checked

Patch 9/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

10/15 Checking commit c052fe6e32bc (arc: Add memory protection unit (MPU) support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#15: 
new file mode 100644

total: 0 errors, 1 warnings, 798 lines checked

Patch 10/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
11/15 Checking commit 6e52aab1c1f3 (arc: Add gdbstub and XML for debugging support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#13: 
new file mode 100644

WARNING: line over 80 characters
#754: FILE: target/arc/gdbstub.c:407:
+                             GDB_AUX_MIN_REG_LAST,        /* number of registers */

WARNING: line over 80 characters
#756: FILE: target/arc/gdbstub.c:409:
+                             0);                          /* position in g packet */

total: 0 errors, 3 warnings, 732 lines checked

Patch 11/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
12/15 Checking commit 2fb7984a73bc (arc: Add Synopsys ARC emulation boards)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#15: 
new file mode 100644

total: 0 errors, 1 warnings, 529 lines checked

Patch 12/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
13/15 Checking commit bafbb7176f2d (arc: Add support for ARCv2)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#29: 
new file mode 100644

ERROR: space prohibited between function name and open parenthesis '('
#192: FILE: include/disas/dis-asm.h:469:
+int print_insn_arc              (bfd_vma, disassemble_info*);

total: 1 errors, 1 warnings, 154 lines checked

Patch 13/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

14/15 Checking commit 482f8c4587f4 (tests/tcg: ARC: Add TCG instruction definition tests)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#28: 
new file mode 100644

WARNING: line over 80 characters
#368: FILE: tests/tcg/arc/check_addx.S:43:
+  TEST_RR_3OP( 20,  add, 0xffffffffffff8000, 0x0000000000000000, 0xffffffffffff8000 )

WARNING: line over 80 characters
#370: FILE: tests/tcg/arc/check_addx.S:45:
+  TEST_RR_3OP( 22,  add, 0xffffffff7fff8000, 0xffffffff80000000, 0xffffffffffff8000 )

WARNING: line over 80 characters
#372: FILE: tests/tcg/arc/check_addx.S:47:
+  TEST_RR_3OP( 23,  add, 0x0000000000007fff, 0x0000000000000000, 0x0000000000007fff )

WARNING: line over 80 characters
#373: FILE: tests/tcg/arc/check_addx.S:48:
+  TEST_RR_3OP( 24,  add, 0x000000007fffffff, 0x000000007fffffff, 0x0000000000000000 )

WARNING: line over 80 characters
#374: FILE: tests/tcg/arc/check_addx.S:49:
+  TEST_RR_3OP( 25,  add, 0x0000000080007ffe, 0x000000007fffffff, 0x0000000000007fff )

WARNING: line over 80 characters
#376: FILE: tests/tcg/arc/check_addx.S:51:
+  TEST_RR_3OP( 26,  add, 0xffffffff80007fff, 0xffffffff80000000, 0x0000000000007fff )

WARNING: line over 80 characters
#377: FILE: tests/tcg/arc/check_addx.S:52:
+  TEST_RR_3OP( 27,  add, 0x000000007fff7fff, 0x000000007fffffff, 0xffffffffffff8000 )

WARNING: line over 80 characters
#379: FILE: tests/tcg/arc/check_addx.S:54:
+  TEST_RR_3OP( 28,  add, 0xffffffffffffffff, 0x0000000000000000, 0xffffffffffffffff )

WARNING: line over 80 characters
#380: FILE: tests/tcg/arc/check_addx.S:55:
+  TEST_RR_3OP( 29,  add, 0x0000000000000000, 0xffffffffffffffff, 0x0000000000000001 )

WARNING: line over 80 characters
#381: FILE: tests/tcg/arc/check_addx.S:56:
+  TEST_RR_3OP( 30,  add, 0xfffffffffffffffe, 0xffffffffffffffff, 0xffffffffffffffff )

WARNING: line over 80 characters
#383: FILE: tests/tcg/arc/check_addx.S:58:
+  TEST_RR_3OP( 31,  add, 0x0000000080000000, 0x0000000000000001, 0x000000007fffffff )

ERROR: space prohibited between function name and open parenthesis '('
#2264: FILE: tests/tcg/arc/check_excp_1.c:3:
+void main (void)

ERROR: space prohibited between function name and open parenthesis '('
#2266: FILE: tests/tcg/arc/check_excp_1.c:5:
+  __builtin_arc_trap_s (0);

ERROR: space prohibited between function name and open parenthesis '('
#2267: FILE: tests/tcg/arc/check_excp_1.c:6:
+  printf ("[PASS] TRAPC:1\n");

ERROR: space prohibited between function name and open parenthesis '('
#2268: FILE: tests/tcg/arc/check_excp_1.c:7:
+  __builtin_arc_trap_s (1);

ERROR: space prohibited between function name and open parenthesis '('
#2269: FILE: tests/tcg/arc/check_excp_1.c:8:
+  printf ("[PASS] TRAPC:2\n");

ERROR: space prohibited between function name and open parenthesis '('
#2275: FILE: tests/tcg/arc/check_excp_1.c:14:
+  printf ("[PASS] TRAPC:IRQ\n");

ERROR: line over 90 characters
#2534: FILE: tests/tcg/arc/check_ldstx.S:21:
+       TEST_CASE(9, r0, 0xcafebabe,  "ld:9", mov r2, 5` mov r1, @tdat` ld_s.as r0,[r1, r2])

ERROR: line over 90 characters
#2931: FILE: tests/tcg/arc/check_lp06.S:141:
+; Timer interrupt and a single insn ZOL. We need to check if indeed we get multiple interrupts, while in ZOL.

ERROR: line over 90 characters
#2932: FILE: tests/tcg/arc/check_lp06.S:142:
+; Timer interrupt and CLRI/SETI body ZOL. The same as above, 2 tests with seti/clri and clri/seti instruction order.

ERROR: line over 90 characters
#3042: FILE: tests/tcg/arc/check_mac.S:44:
+.macro mac_test acch=NOTSET, accl=NOTSET, val1, val2, res, racch=NOTSET, raccl=NOTSET, n=0, v=0, test_num

ERROR: line over 90 characters
#3056: FILE: tests/tcg/arc/check_mac.S:58:
+.macro macu_test acch=NOTSET, accl=NOTSET, val1, val2, res, racch=NOTSET, raccl=NOTSET, v=0, test_num

ERROR: line over 90 characters
#3071: FILE: tests/tcg/arc/check_mac.S:73:
+.macro macd_test acch=NOTSET, accl=NOTSET, val1, val2, resh, resl, racch=NOTSET, raccl=NOTSET, n=0, v=0, test_num

ERROR: line over 90 characters
#3086: FILE: tests/tcg/arc/check_mac.S:88:
+.macro macdu_test acch=NOTSET, accl=NOTSET, val1, val2, resh, resl, racch=NOTSET, raccl=NOTSET, v=0, test_num

ERROR: line over 90 characters
#3111: FILE: tests/tcg/arc/check_mac.S:113:
+mac_test val1=0xFFFFFFFD, val2=0x09, res=0xFFFFFFF9, racch=0xFFFFFFFF, raccl=0xFFFFFFF9, n=1, test_num=4

ERROR: line over 90 characters
#3113: FILE: tests/tcg/arc/check_mac.S:115:
+mac_test acch=0, accl=0, val1=0x7FFFFFFF, val2=0x7FFFFFFF, res=1, racch=0x3FFFFFFF, raccl=0x01, n=0, v=0, test_num=5

ERROR: line over 90 characters
#3115: FILE: tests/tcg/arc/check_mac.S:117:
+mac_test val1=0x80000000, val2=0x80000000, res=1, racch=0x7FFFFFFF, raccl=0x01, n=0, v=0, test_num=6

ERROR: line over 90 characters
#3117: FILE: tests/tcg/arc/check_mac.S:119:
+mac_test val1=0x12344321, val2=0x56788654, res=0xE1C14CD5, racch=0x86262098, raccl=0xE1C14CD5, n=1, v=1, test_num=7

WARNING: line over 80 characters
#3119: FILE: tests/tcg/arc/check_mac.S:121:
+mac_test acch=0, accl=0, val1=0, val2=0, res=0, racch=0, raccl=0, n=0, v=1, test_num=8

ERROR: line over 90 characters
#3131: FILE: tests/tcg/arc/check_mac.S:133:
+macu_test val1=0xFFFFFFFD, val2=0x09, res=0xFFFFFFF9, racch=0x0D, raccl=0xFFFFFFF9, test_num=12

ERROR: line over 90 characters
#3133: FILE: tests/tcg/arc/check_mac.S:135:
+macu_test acch=0, accl=0, val1=0x7FFFFFFF, val2=0x7FFFFFFF, res=1, racch=0x3FFFFFFF, raccl=0x01, v=0, test_num=13

ERROR: line over 90 characters
#3135: FILE: tests/tcg/arc/check_mac.S:137:
+macu_test val1=0x80000000, val2=0x80000000, res=1, racch=0x7FFFFFFF, raccl=0x01, v=0, test_num=14

WARNING: line over 80 characters
#3136: FILE: tests/tcg/arc/check_mac.S:138:
+; acc is 0x7FFFFFFF00000001; line below still will not trigger an overflow for MACU

ERROR: line over 90 characters
#3137: FILE: tests/tcg/arc/check_mac.S:139:
+macu_test val1=0x12344321, val2=0x56788654, res=0xE1C14CD5, racch=0x86262098, raccl=0xE1C14CD5, v=0, test_num=15

ERROR: line over 90 characters
#3139: FILE: tests/tcg/arc/check_mac.S:141:
+macu_test acch=0xFFFFFFFF, accl=0xFFFFFFFF, val1=1, val2=1, res=0, racch=0, raccl=0, v=1, test_num=16

WARNING: line over 80 characters
#3141: FILE: tests/tcg/arc/check_mac.S:143:
+macu_test acch=0, accl=0, val1=0, val2=0, res=0, racch=0, raccl=0, v=1, test_num=17

ERROR: line over 90 characters
#3150: FILE: tests/tcg/arc/check_mac.S:152:
+macd_test val1=0xFFFFFFFF, val2=0x5, resh=0, resl=20, racch=0x0, raccl=20, n=0, test_num=20

ERROR: line over 90 characters
#3152: FILE: tests/tcg/arc/check_mac.S:154:
+macd_test val1=0xFFFFFFFD, val2=0x09, resh = 0xFFFFFFFF, resl=0xFFFFFFF9, racch=0xFFFFFFFF, raccl=0xFFFFFFF9, n=1, test_num=21

ERROR: line over 90 characters
#3154: FILE: tests/tcg/arc/check_mac.S:156:
+macd_test acch=0, accl=0, val1=0x7FFFFFFF, val2=0x7FFFFFFF, resh=0x3FFFFFFF, resl=0x01, racch=0x3FFFFFFF, raccl=0x01, v=0, test_num=22

ERROR: line over 90 characters
#3156: FILE: tests/tcg/arc/check_mac.S:158:
+macd_test val1=0x80000000, val2=0x80000000, resh=0x7FFFFFFF, resl=0x01, racch=0x7FFFFFFF, raccl=0x01, v=0, test_num=23

ERROR: line over 90 characters
#3158: FILE: tests/tcg/arc/check_mac.S:160:
+macd_test val1=0x12344321, val2=0x56788654, resh=0x86262098, resl=0xE1C14CD5, racch=0x86262098, raccl=0xE1C14CD5, n=1, v=1, test_num=24

ERROR: line over 90 characters
#3160: FILE: tests/tcg/arc/check_mac.S:162:
+macd_test acch=0, accl=0, val1=0, val2=0, resh=0, resl=0, racch=0, raccl=0, n=0, v=1, test_num=25

WARNING: line over 80 characters
#3169: FILE: tests/tcg/arc/check_mac.S:171:
+macdu_test val1=0xFFFFFFFF, val2=0x5, resh=5, resl=20, racch=5, raccl=20, test_num=28

ERROR: line over 90 characters
#3171: FILE: tests/tcg/arc/check_mac.S:173:
+macdu_test val1=0xFFFFFFFD, val2=0x09, resh=0x0D, resl=0xFFFFFFF9, racch=0x0D, raccl=0xFFFFFFF9, test_num=29

ERROR: line over 90 characters
#3173: FILE: tests/tcg/arc/check_mac.S:175:
+macdu_test acch=0, accl=0, val1=0x7FFFFFFF, val2=0x7FFFFFFF, resh=0x3FFFFFFF, resl=0x01, racch=0x3FFFFFFF, raccl=0x01, v=0, test_num=30

ERROR: line over 90 characters
#3175: FILE: tests/tcg/arc/check_mac.S:177:
+macdu_test val1=0x80000000, val2=0x80000000, resh=0x7FFFFFFF, resl=0x01, racch=0x7FFFFFFF, raccl=0x01, v=0, test_num=31

WARNING: line over 80 characters
#3176: FILE: tests/tcg/arc/check_mac.S:178:
+; acc is 0x7FFFFFFF00000001; line below still will not trigger an overflow for MACU

ERROR: line over 90 characters
#3177: FILE: tests/tcg/arc/check_mac.S:179:
+macdu_test val1=0x12344321, val2=0x56788654, resh=0x86262098, resl=0xE1C14CD5, racch=0x86262098, raccl=0xE1C14CD5, v=0, test_num=32

ERROR: line over 90 characters
#3179: FILE: tests/tcg/arc/check_mac.S:181:
+macdu_test acch=0xFFFFFFFF, accl=0xFFFFFFFF, val1=1, val2=1, resh=0, resl=0, racch=0, raccl=0, v=1, test_num=33

ERROR: line over 90 characters
#3181: FILE: tests/tcg/arc/check_mac.S:183:
+macdu_test acch=0, accl=0, val1=0, val2=0, resh=0, resl=0, racch=0, raccl=0, v=1, test_num=34

ERROR: line over 90 characters
#3314: FILE: tests/tcg/arc/check_manip_10_mmu.S:82:
+  .equ T5_PD0, ((T5_VIRT_ADDR+T5_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#3315: FILE: tests/tcg/arc/check_manip_10_mmu.S:83:
+  .equ T5_PD1, ((T5_PHYS_ADDR+T5_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R | REG_PD1_KRNL_E)

ERROR: line over 90 characters
#3492: FILE: tests/tcg/arc/check_manip_4_mmu.S:81:
+  .equ T4_PD0, ((T4_VIRT_ADDR+T4_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#3493: FILE: tests/tcg/arc/check_manip_4_mmu.S:82:
+  .equ T4_PD1, ((T4_PHYS_ADDR+T4_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R | REG_PD1_KRNL_E)

ERROR: line over 90 characters
#3657: FILE: tests/tcg/arc/check_manip_5_mmu.S:82:
+  .equ T5_PD0, ((T5_VIRT_ADDR+T5_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#3658: FILE: tests/tcg/arc/check_manip_5_mmu.S:83:
+  .equ T5_PD1, ((T5_PHYS_ADDR+T5_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R | REG_PD1_KRNL_E)

WARNING: line over 80 characters
#3830: FILE: tests/tcg/arc/check_manip_mmu.S:83:
+  .equ T1_PHYS_ADDR, 0x73311334     ; the physical address (same page offset as VA)

WARNING: line over 80 characters
#3833: FILE: tests/tcg/arc/check_manip_mmu.S:86:
+  .equ T1_PD0   , ((T1_VIRT_ADDR & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#3905: FILE: tests/tcg/arc/check_manip_mmu.S:158:
+  .equ T2_PD0, ((T2_VIRT_ADDR+T2_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#3969: FILE: tests/tcg/arc/check_manip_mmu.S:222:
+  .equ T3_PD0_ENT1, ((T3_VIRT_ADDR_1+T3_ADDR_1_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

WARNING: line over 80 characters
#3970: FILE: tests/tcg/arc/check_manip_mmu.S:223:
+  .equ T3_PD1_ENT1, ((T3_PHYS_ADDR_1               & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R)

ERROR: line over 90 characters
#3971: FILE: tests/tcg/arc/check_manip_mmu.S:224:
+  .equ T3_PD0_ENT2, ((T3_VIRT_ADDR_2               & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

WARNING: line over 80 characters
#3972: FILE: tests/tcg/arc/check_manip_mmu.S:225:
+  .equ T3_PD1_ENT2, ((T3_PHYS_ADDR_2               & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R)

WARNING: line over 80 characters
#3978: FILE: tests/tcg/arc/check_manip_mmu.S:231:
+  sth     r1, [T3_PHYS_ADDR_2]                ; 0x56 0x34 at the beginning of 2nd page

WARNING: line over 80 characters
#3980: FILE: tests/tcg/arc/check_manip_mmu.S:233:
+  stb     r1, [T3_PHYS_ADDR_2+2]              ; 0x12 The 3rd byte on the 2nd page

ERROR: line over 90 characters
#4032: FILE: tests/tcg/arc/check_manip_mmu.S:285:
+  .equ T4_PD0, ((T4_VIRT_ADDR+T4_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#4033: FILE: tests/tcg/arc/check_manip_mmu.S:286:
+  .equ T4_PD1, ((T4_PHYS_ADDR+T4_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R | REG_PD1_KRNL_E)

ERROR: line over 90 characters
#4109: FILE: tests/tcg/arc/check_manip_mmu.S:362:
+  .equ T5_PD0, ((T5_VIRT_ADDR+T5_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#4110: FILE: tests/tcg/arc/check_manip_mmu.S:363:
+  .equ T5_PD1, ((T5_PHYS_ADDR+T5_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R | REG_PD1_KRNL_E)

ERROR: line over 90 characters
#4243: FILE: tests/tcg/arc/check_manip_mmu.S:496:
+  .equ T9_PD0, ((T9_VIRT_ADDR+T9_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD0_GLOBAL | REG_PD0_VALID)

ERROR: line over 90 characters
#4244: FILE: tests/tcg/arc/check_manip_mmu.S:497:
+  .equ T9_PD1, ((T9_PHYS_ADDR+T9_ADDR_OFS & PAGE_NUMBER_MSK) | REG_PD1_KRNL_R | REG_PD1_KRNL_E)

WARNING: line over 80 characters
#4279: FILE: tests/tcg/arc/check_manip_mmu.S:532:
+  cmp     r11, @T9_VIRT_ADDR+0x2000-4 ; Last instruction of the first page (blne.d)?

WARNING: line over 80 characters
#5788: FILE: tests/tcg/arc/check_prefetch.S:14:
+  TEST_CASE( 2, r0, 0x00000000, "prefetch:2", prefetch    [0x12]`      mov r0, 0x0)

WARNING: line over 80 characters
#5789: FILE: tests/tcg/arc/check_prefetch.S:15:
+  TEST_CASE( 3, r0, 0x00000000, "prefetch:3", prefetchw   [0x12]`      mov r0, 0x0)

WARNING: line over 80 characters
#5790: FILE: tests/tcg/arc/check_prefetch.S:16:
+  TEST_CASE( 4, r0, 0x00000000, "prefetch:4", prefetchw   [r1, r2]`    mov r0, 0x0)

WARNING: line over 80 characters
#5791: FILE: tests/tcg/arc/check_prefetch.S:17:
+  TEST_CASE( 5, r0, 0x00000000, "prefetch:5", prefetchw   [0x12, 0x1]` mov r0, 0x0)

WARNING: line over 80 characters
#5792: FILE: tests/tcg/arc/check_prefetch.S:18:
+  TEST_CASE( 6, r0, 0x00000000, "prefetch:6", prefetch    [r1, r2]`    mov r0, 0x0)

WARNING: line over 80 characters
#5793: FILE: tests/tcg/arc/check_prefetch.S:19:
+  TEST_CASE( 7, r0, 0x00000000, "prefetch:7", prefetch    [0x12, 0x1]` mov r0, 0x0)

WARNING: line over 80 characters
#5796: FILE: tests/tcg/arc/check_prefetch.S:22:
+       TEST_CASE( 8, r0, 0x00000004, "prefetch:8", prefetch    [r13]`       ld r0,[r13])

WARNING: line over 80 characters
#5797: FILE: tests/tcg/arc/check_prefetch.S:23:
+       TEST_CASE( 9, r0, 0x40000000, "prefetch:9", prefetch.aw [r13,4]`     ld r0,[r13])

WARNING: line over 80 characters
#5798: FILE: tests/tcg/arc/check_prefetch.S:24:
+       TEST_CASE(10, r0, 0x40400000, "prefetch:10", prefetch.ab [r13,4]`     ld r0,[r13])

WARNING: line over 80 characters
#6069: FILE: tests/tcg/arc/check_subf.S:42:
+sub0_flags_test 0xA0000000, 0xB0000000, 0xF0000000, z=0, n=1, c=1, v=0, test_num=0x01

WARNING: line over 80 characters
#6070: FILE: tests/tcg/arc/check_subf.S:43:
+sub1_flags_test 0xA0000000, 0x58000000, 0xF0000000, z=0, n=1, c=1, v=0, test_num=0x02

WARNING: line over 80 characters
#6071: FILE: tests/tcg/arc/check_subf.S:44:
+sub2_flags_test 0xA0000000, 0x2C000000, 0xF0000000, z=0, n=1, c=1, v=0, test_num=0x03

WARNING: line over 80 characters
#6072: FILE: tests/tcg/arc/check_subf.S:45:
+sub3_flags_test 0xA0000000, 0x16000000, 0xF0000000, z=0, n=1, c=1, v=0, test_num=0x04

WARNING: line over 80 characters
#6074: FILE: tests/tcg/arc/check_subf.S:47:
+sub0_flags_test 0xFFFFFF80, 0xF0000000, 0x0FFFFF80, z=0, n=0, c=0, v=0, test_num=0x05

WARNING: line over 80 characters
#6075: FILE: tests/tcg/arc/check_subf.S:48:
+sub1_flags_test 0xFFFFFF80, 0x78000000, 0x0FFFFF80, z=0, n=0, c=0, v=0, test_num=0x06

WARNING: line over 80 characters
#6076: FILE: tests/tcg/arc/check_subf.S:49:
+sub2_flags_test 0xFFFFFF80, 0x3C000000, 0x0FFFFF80, z=0, n=0, c=0, v=0, test_num=0x07

WARNING: line over 80 characters
#6077: FILE: tests/tcg/arc/check_subf.S:50:
+sub3_flags_test 0xFFFFFF80, 0x1E000000, 0x0FFFFF80, z=0, n=0, c=0, v=0, test_num=0x08

WARNING: line over 80 characters
#6079: FILE: tests/tcg/arc/check_subf.S:52:
+sub0_flags_test 0x80000000, 0x80000000, 0x00000000, z=1, n=0, c=0, v=0, test_num=0x09

WARNING: line over 80 characters
#6080: FILE: tests/tcg/arc/check_subf.S:53:
+sub1_flags_test 0x80000000, 0x40000000, 0x00000000, z=1, n=0, c=0, v=0, test_num=0x10

WARNING: line over 80 characters
#6081: FILE: tests/tcg/arc/check_subf.S:54:
+sub2_flags_test 0x80000000, 0x20000000, 0x00000000, z=1, n=0, c=0, v=0, test_num=0x11

WARNING: line over 80 characters
#6082: FILE: tests/tcg/arc/check_subf.S:55:
+sub3_flags_test 0x80000000, 0x10000000, 0x00000000, z=1, n=0, c=0, v=0, test_num=0x12

WARNING: line over 80 characters
#6084: FILE: tests/tcg/arc/check_subf.S:57:
+sub0_flags_test 0x80000000, 0xC0000000, 0xC0000000, z=0, n=1, c=1, v=0, test_num=0x13

WARNING: line over 80 characters
#6085: FILE: tests/tcg/arc/check_subf.S:58:
+sub1_flags_test 0x80000000, 0x60000000, 0xC0000000, z=0, n=1, c=1, v=0, test_num=0x14

WARNING: line over 80 characters
#6086: FILE: tests/tcg/arc/check_subf.S:59:
+sub2_flags_test 0x80000000, 0x30000000, 0xC0000000, z=0, n=1, c=1, v=0, test_num=0x15

WARNING: line over 80 characters
#6087: FILE: tests/tcg/arc/check_subf.S:60:
+sub3_flags_test 0x80000000, 0x18000000, 0xC0000000, z=0, n=1, c=1, v=0, test_num=0x16

WARNING: line over 80 characters
#6089: FILE: tests/tcg/arc/check_subf.S:62:
+sub0_flags_test 0x80000000, 0x00000008, 0x7FFFFFF8, z=0, n=0, c=0, v=1, test_num=0x17

WARNING: line over 80 characters
#6090: FILE: tests/tcg/arc/check_subf.S:63:
+sub1_flags_test 0x80000000, 0x00000004, 0x7FFFFFF8, z=0, n=0, c=0, v=1, test_num=0x18

WARNING: line over 80 characters
#6091: FILE: tests/tcg/arc/check_subf.S:64:
+sub2_flags_test 0x80000000, 0x00000002, 0x7FFFFFF8, z=0, n=0, c=0, v=1, test_num=0x19

WARNING: line over 80 characters
#6092: FILE: tests/tcg/arc/check_subf.S:65:
+sub3_flags_test 0x80000000, 0x00000001, 0x7FFFFFF8, z=0, n=0, c=0, v=1, test_num=0x20

WARNING: line over 80 characters
#6117: FILE: tests/tcg/arc/check_subx.S:17:
+  TEST_RR_3OP( 2,  sub, 0x0000000000000000, 0x0000000000000000, 0x0000000000000000 );

WARNING: line over 80 characters
#6118: FILE: tests/tcg/arc/check_subx.S:18:
+  TEST_RR_3OP( 3,  sub, 0x0000000000000000, 0x0000000000000001, 0x0000000000000001 );

WARNING: line over 80 characters
#6119: FILE: tests/tcg/arc/check_subx.S:19:
+  TEST_RR_3OP( 4,  sub, 0xfffffffffffffffc, 0x0000000000000003, 0x0000000000000007 );

WARNING: line over 80 characters
#6121: FILE: tests/tcg/arc/check_subx.S:21:
+  TEST_RR_3OP( 5,  sub, 0x0000000000008000, 0x0000000000000000, 0xffffffffffff8000 );

WARNING: line over 80 characters
#6122: FILE: tests/tcg/arc/check_subx.S:22:
+  TEST_RR_3OP( 6,  sub, 0xffffffff80000000, 0xffffffff80000000, 0x0000000000000000 );

WARNING: line over 80 characters
#6123: FILE: tests/tcg/arc/check_subx.S:23:
+  TEST_RR_3OP( 7,  sub, 0xffffffff80008000, 0xffffffff80000000, 0xffffffffffff8000 );

WARNING: line over 80 characters
#6125: FILE: tests/tcg/arc/check_subx.S:25:
+  TEST_RR_3OP( 8,  sub, 0xffffffffffff8001, 0x0000000000000000, 0x0000000000007fff );

WARNING: line over 80 characters
#6126: FILE: tests/tcg/arc/check_subx.S:26:
+  TEST_RR_3OP( 9,  sub, 0x000000007fffffff, 0x000000007fffffff, 0x0000000000000000 );

WARNING: line over 80 characters
#6127: FILE: tests/tcg/arc/check_subx.S:27:
+  TEST_RR_3OP( 10, sub, 0x000000007fff8000, 0x000000007fffffff, 0x0000000000007fff );

WARNING: line over 80 characters
#6129: FILE: tests/tcg/arc/check_subx.S:29:
+  TEST_RR_3OP( 11, sub, 0xffffffff7fff8001, 0xffffffff80000000, 0x0000000000007fff );

WARNING: line over 80 characters
#6130: FILE: tests/tcg/arc/check_subx.S:30:
+  TEST_RR_3OP( 12, sub, 0x0000000080007fff, 0x000000007fffffff, 0xffffffffffff8000 );

WARNING: line over 80 characters
#6132: FILE: tests/tcg/arc/check_subx.S:32:
+  TEST_RR_3OP( 13, sub, 0x0000000000000001, 0x0000000000000000, 0xffffffffffffffff );

WARNING: line over 80 characters
#6133: FILE: tests/tcg/arc/check_subx.S:33:
+  TEST_RR_3OP( 14, sub, 0xfffffffffffffffe, 0xffffffffffffffff, 0x0000000000000001 );

WARNING: line over 80 characters
#6134: FILE: tests/tcg/arc/check_subx.S:34:
+  TEST_RR_3OP( 15, sub, 0x0000000000000000, 0xffffffffffffffff, 0xffffffffffffffff );

WARNING: line over 80 characters
#7789: FILE: tests/tcg/arc/check_xorx.S:28:
+  TEST_IMM_SRC1_EQ_DEST( 10, xor, 0xffffffffff00f00f, 0xffffffffff00f700, 0x70f );

WARNING: architecture specific defines should be avoided
#8584: FILE: tests/tcg/arc/test_macros.h:1:
+#ifndef __TEST_MACROS_SCALAR_H

ERROR: space prohibited after that open parenthesis '('
#8599: FILE: tests/tcg/arc/test_macros.h:16:
+#define TEST_CASE( testnum, testreg, correctval, name, code... ) \

ERROR: space prohibited before that close parenthesis ')'
#8599: FILE: tests/tcg/arc/test_macros.h:16:
+#define TEST_CASE( testnum, testreg, correctval, name, code... ) \

ERROR: Macros with complex values should be enclosed in parenthesis
#8599: FILE: tests/tcg/arc/test_macros.h:16:
+#define TEST_CASE( testnum, testreg, correctval, name, code... ) \
+    test_ ## testnum:                                       \
+    code`                                                   \
+    mov  r12, testnum`                                      \
+    sub.f 0,testreg, correctval`                            \
+    bne  @fail`                                             \
+    PASS_TEST(name)

ERROR: spaces required around that ':' (ctx:VxE)
#8600: FILE: tests/tcg/arc/test_macros.h:17:
+    test_ ## testnum:                                       \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8603: FILE: tests/tcg/arc/test_macros.h:20:
+    sub.f 0,testreg, correctval`                            \
            ^

ERROR: space prohibited after that open parenthesis '('
#8607: FILE: tests/tcg/arc/test_macros.h:24:
+#define TEST_IMM_OP( testnum, inst, result, val1, imm ) \

ERROR: space prohibited before that close parenthesis ')'
#8607: FILE: tests/tcg/arc/test_macros.h:24:
+#define TEST_IMM_OP( testnum, inst, result, val1, imm ) \

ERROR: space prohibited after that open parenthesis '('
#8608: FILE: tests/tcg/arc/test_macros.h:25:
+    TEST_CASE( testnum, r0, result, xstr(inst) ":" xstr(testnum),       \

ERROR: space prohibited after that open parenthesis '('
#8613: FILE: tests/tcg/arc/test_macros.h:30:
+#define TEST_RR_3OP( testnum, inst, result, val1, val2 ) \

ERROR: space prohibited before that close parenthesis ')'
#8613: FILE: tests/tcg/arc/test_macros.h:30:
+#define TEST_RR_3OP( testnum, inst, result, val1, val2 ) \

ERROR: space prohibited after that open parenthesis '('
#8614: FILE: tests/tcg/arc/test_macros.h:31:
+    TEST_CASE( testnum, r0, result, xstr(inst) ":" xstr(testnum),  \

ERROR: space prohibited after that open parenthesis '('
#8620: FILE: tests/tcg/arc/test_macros.h:37:
+#define TEST_RR_2OP( testnum, inst, result, val)         \

ERROR: space prohibited after that open parenthesis '('
#8621: FILE: tests/tcg/arc/test_macros.h:38:
+    TEST_CASE( testnum, r0, result, xstr(inst) ":" xstr(testnum),  \

ERROR: space prohibited after that open parenthesis '('
#8626: FILE: tests/tcg/arc/test_macros.h:43:
+#define TEST_IMM_SRC1_EQ_DEST( testnum, inst, result, val1, imm ) \

ERROR: space prohibited before that close parenthesis ')'
#8626: FILE: tests/tcg/arc/test_macros.h:43:
+#define TEST_IMM_SRC1_EQ_DEST( testnum, inst, result, val1, imm ) \

ERROR: space prohibited after that open parenthesis '('
#8627: FILE: tests/tcg/arc/test_macros.h:44:
+    TEST_CASE( testnum, r1, result, xstr(inst) ":" xstr(testnum),           \

ERROR: space prohibited after that open parenthesis '('
#8632: FILE: tests/tcg/arc/test_macros.h:49:
+#define TEST_RR_SRC1_EQ_DEST( testnum, inst, result, val1, val2 ) \

ERROR: space prohibited before that close parenthesis ')'
#8632: FILE: tests/tcg/arc/test_macros.h:49:
+#define TEST_RR_SRC1_EQ_DEST( testnum, inst, result, val1, val2 ) \

ERROR: space prohibited after that open parenthesis '('
#8633: FILE: tests/tcg/arc/test_macros.h:50:
+    TEST_CASE( testnum, r1, result, xstr(inst) ":" xstr(testnum),           \

ERROR: space prohibited after that open parenthesis '('
#8639: FILE: tests/tcg/arc/test_macros.h:56:
+#define TEST_RR_2OP_SRC1_EQ_DEST( testnum, inst, result, val )     \

ERROR: space prohibited before that close parenthesis ')'
#8639: FILE: tests/tcg/arc/test_macros.h:56:
+#define TEST_RR_2OP_SRC1_EQ_DEST( testnum, inst, result, val )     \

ERROR: space prohibited after that open parenthesis '('
#8640: FILE: tests/tcg/arc/test_macros.h:57:
+    TEST_CASE( testnum, r1, result, xstr(inst) ":" xstr(testnum),            \

ERROR: space prohibited after that open parenthesis '('
#8645: FILE: tests/tcg/arc/test_macros.h:62:
+#define TEST_RR_SRC2_EQ_DEST( testnum, inst, result, val1, val2 )  \

ERROR: space prohibited before that close parenthesis ')'
#8645: FILE: tests/tcg/arc/test_macros.h:62:
+#define TEST_RR_SRC2_EQ_DEST( testnum, inst, result, val1, val2 )  \

ERROR: space prohibited after that open parenthesis '('
#8646: FILE: tests/tcg/arc/test_macros.h:63:
+    TEST_CASE( testnum, r2, result, xstr(inst) ":" xstr(testnum),            \

ERROR: space prohibited after that open parenthesis '('
#8652: FILE: tests/tcg/arc/test_macros.h:69:
+#define TEST_RR_SRC12_EQ_DEST( testnum, inst, result, val1 ) \

ERROR: space prohibited before that close parenthesis ')'
#8652: FILE: tests/tcg/arc/test_macros.h:69:
+#define TEST_RR_SRC12_EQ_DEST( testnum, inst, result, val1 ) \

ERROR: space prohibited after that open parenthesis '('
#8653: FILE: tests/tcg/arc/test_macros.h:70:
+    TEST_CASE( testnum, r1, result, xstr(inst) ":" xstr(testnum),      \

ERROR: space prohibited after that open parenthesis '('
#8658: FILE: tests/tcg/arc/test_macros.h:75:
+#define TEST_2OP_CARRY( testnum, inst, expected, val1, val2) \

ERROR: Macros with complex values should be enclosed in parenthesis
#8658: FILE: tests/tcg/arc/test_macros.h:75:
+#define TEST_2OP_CARRY( testnum, inst, expected, val1, val2) \
+    test_ ## testnum:                                        \
+    mov  r12, testnum`                                       \
+        mov  r1, MASK_XLEN(val1)`                            \
+        mov  r2, MASK_XLEN(val2)`                            \
+        inst.f   0, r1, r2`                                  \
+        mov.cs   r3,(~expected) & 0x01`                      \
+        mov.cc   r3, (expected) & 0x01`                      \
+        cmp      r3, 0`                                      \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8659: FILE: tests/tcg/arc/test_macros.h:76:
+    test_ ## testnum:                                        \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8664: FILE: tests/tcg/arc/test_macros.h:81:
+        mov.cs   r3,(~expected) & 0x01`                      \
                    ^

ERROR: space prohibited after that open parenthesis '('
#8669: FILE: tests/tcg/arc/test_macros.h:86:
+#define TEST_1OP_CARRY( testnum, inst, expected, val) \

ERROR: Macros with complex values should be enclosed in parenthesis
#8669: FILE: tests/tcg/arc/test_macros.h:86:
+#define TEST_1OP_CARRY( testnum, inst, expected, val) \
+    test_ ## testnum:                                 \
+    mov  r12, testnum`                                \
+        add.f    0, r0, r0`                           \
+        mov      r1, MASK_XLEN(val)`                  \
+        inst.f   0, r1`                               \
+        mov.cs   r3,(~expected) & 0x01`               \
+        mov.cc   r3, (expected) & 0x01`               \
+        cmp      r3, 0`                               \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8670: FILE: tests/tcg/arc/test_macros.h:87:
+    test_ ## testnum:                                 \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8675: FILE: tests/tcg/arc/test_macros.h:92:
+        mov.cs   r3,(~expected) & 0x01`               \
                    ^

ERROR: space prohibited after that open parenthesis '('
#8680: FILE: tests/tcg/arc/test_macros.h:97:
+#define TEST_2OP_ZERO( testnum, inst, expected, val1, val2)  \

ERROR: Macros with complex values should be enclosed in parenthesis
#8680: FILE: tests/tcg/arc/test_macros.h:97:
+#define TEST_2OP_ZERO( testnum, inst, expected, val1, val2)  \
+    test_ ## testnum:                                        \
+    mov  r12, testnum`                                       \
+        mov      r1, MASK_XLEN(val1)`                        \
+        mov      r2, MASK_XLEN(val2)`                        \
+        inst.f   0, r1, r2`                                  \
+        mov.eq   r3, (~expected) & 0x01`                     \
+        mov.ne   r3, (expected) & 0x01`                      \
+        cmp      r3, 0`                                      \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8681: FILE: tests/tcg/arc/test_macros.h:98:
+    test_ ## testnum:                                        \
                     ^

ERROR: space prohibited after that open parenthesis '('
#8691: FILE: tests/tcg/arc/test_macros.h:108:
+#define TEST_1OP_ZERO( testnum, inst, expected, val)  \

ERROR: Macros with complex values should be enclosed in parenthesis
#8691: FILE: tests/tcg/arc/test_macros.h:108:
+#define TEST_1OP_ZERO( testnum, inst, expected, val)  \
+    test_ ## testnum:                                 \
+    mov  r12, testnum`                                \
+        add.f    0, r0, r0`                           \
+        mov      r1, MASK_XLEN(val)`                  \
+        inst.f   0, r1`                               \
+        mov.eq   r3, (~expected) & 0x01`              \
+        mov.ne   r3, (expected) & 0x01`               \
+        cmp      r3, 0`                               \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8692: FILE: tests/tcg/arc/test_macros.h:109:
+    test_ ## testnum:                                 \
                     ^

ERROR: space prohibited after that open parenthesis '('
#8702: FILE: tests/tcg/arc/test_macros.h:119:
+#define TEST_2OP_OVERFLOW( testnum, inst, expected, val1, val2) \

ERROR: Macros with complex values should be enclosed in parenthesis
#8702: FILE: tests/tcg/arc/test_macros.h:119:
+#define TEST_2OP_OVERFLOW( testnum, inst, expected, val1, val2) \
+    test_ ## testnum:                                           \
+    mov  r12, testnum`                                          \
+        mov      r1, MASK_XLEN(val1)`                           \
+        mov      r2, MASK_XLEN(val2)`                           \
+        inst.f   0, r1, r2`                                     \
+        mov.vs   r3,(~expected) & 0x01`                         \
+        mov.vc   r3, (expected) & 0x01`                         \
+        cmp      r3, 0`                                         \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8703: FILE: tests/tcg/arc/test_macros.h:120:
+    test_ ## testnum:                                           \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8708: FILE: tests/tcg/arc/test_macros.h:125:
+        mov.vs   r3,(~expected) & 0x01`                         \
                    ^

ERROR: space prohibited after that open parenthesis '('
#8713: FILE: tests/tcg/arc/test_macros.h:130:
+#define TEST_1OP_OVERFLOW( testnum, inst, expected, val) \

ERROR: Macros with complex values should be enclosed in parenthesis
#8713: FILE: tests/tcg/arc/test_macros.h:130:
+#define TEST_1OP_OVERFLOW( testnum, inst, expected, val) \
+    test_ ## testnum:                                    \
+    mov  r12, testnum`                                   \
+        add.f    0, r0, r0`                              \
+        mov      r1, MASK_XLEN(val)`                     \
+        inst.f   0, r1`                                  \
+        mov.vs   r3,(~expected) & 0x01`                  \
+        mov.vc   r3, (expected) & 0x01`                  \
+        cmp      r3, 0`                                  \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8714: FILE: tests/tcg/arc/test_macros.h:131:
+    test_ ## testnum:                                    \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8719: FILE: tests/tcg/arc/test_macros.h:136:
+        mov.vs   r3,(~expected) & 0x01`                  \
                    ^

ERROR: space prohibited after that open parenthesis '('
#8724: FILE: tests/tcg/arc/test_macros.h:141:
+#define TEST_2OP_NEGATIVE( testnum, inst, expected, val1, val2)    \

ERROR: Macros with complex values should be enclosed in parenthesis
#8724: FILE: tests/tcg/arc/test_macros.h:141:
+#define TEST_2OP_NEGATIVE( testnum, inst, expected, val1, val2)    \
+    test_ ## testnum:                                              \
+    mov  r12, testnum`                                             \
+        mov      r1, MASK_XLEN(val1)`                              \
+        mov      r2, MASK_XLEN(val2)`                              \
+        inst.f   0, r1, r2`                                        \
+        mov.mi   r3,(~expected) & 0x01`                            \
+        mov.pl   r3, (expected) & 0x01`                            \
+        cmp      r3, 0`                                            \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8725: FILE: tests/tcg/arc/test_macros.h:142:
+    test_ ## testnum:                                              \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8730: FILE: tests/tcg/arc/test_macros.h:147:
+        mov.mi   r3,(~expected) & 0x01`                            \
                    ^

ERROR: space prohibited after that open parenthesis '('
#8735: FILE: tests/tcg/arc/test_macros.h:152:
+#define TEST_1OP_NEGATIVE( testnum, inst, expected, val)    \

ERROR: Macros with complex values should be enclosed in parenthesis
#8735: FILE: tests/tcg/arc/test_macros.h:152:
+#define TEST_1OP_NEGATIVE( testnum, inst, expected, val)    \
+    test_ ## testnum:                                       \
+    mov  r12, testnum`                                      \
+        add.f    0, r0, r0`                                 \
+        mov      r1, MASK_XLEN(val)`                        \
+        inst.f   0, r1`                                     \
+        mov.mi   r3,(~expected) & 0x01`                     \
+        mov.pl   r3, (expected) & 0x01`                     \
+        cmp      r3, 0`                                     \
+        bne      @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8736: FILE: tests/tcg/arc/test_macros.h:153:
+    test_ ## testnum:                                       \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8741: FILE: tests/tcg/arc/test_macros.h:158:
+        mov.mi   r3,(~expected) & 0x01`                     \
                    ^

ERROR: space prohibited after that open parenthesis '('
#8749: FILE: tests/tcg/arc/test_macros.h:166:
+#define TEST_BR2_OP_TAKEN( testnum, inst, val1, val2 )  \

ERROR: space prohibited before that close parenthesis ')'
#8749: FILE: tests/tcg/arc/test_macros.h:166:
+#define TEST_BR2_OP_TAKEN( testnum, inst, val1, val2 )  \

ERROR: Macros with complex values should be enclosed in parenthesis
#8749: FILE: tests/tcg/arc/test_macros.h:166:
+#define TEST_BR2_OP_TAKEN( testnum, inst, val1, val2 )  \
+    test_ ## testnum:`                                  \
+        mov  r12, testnum`                              \
+        mov  r1, val1`                                  \
+        mov  r2, val2`                                  \
+        sub.f 0,r1,r2`                                  \
+        inst 1f`                                        \
+        b @fail`                                        \
+        1:

ERROR: spaces required around that ':' (ctx:VxV)
#8750: FILE: tests/tcg/arc/test_macros.h:167:
+    test_ ## testnum:`                                  \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8754: FILE: tests/tcg/arc/test_macros.h:171:
+        sub.f 0,r1,r2`                                  \
                ^

ERROR: space required after that ',' (ctx:VxV)
#8754: FILE: tests/tcg/arc/test_macros.h:171:
+        sub.f 0,r1,r2`                                  \
                   ^

ERROR: spaces required around that ':' (ctx:VxE)
#8757: FILE: tests/tcg/arc/test_macros.h:174:
+        1:
          ^

ERROR: space prohibited after that open parenthesis '('
#8759: FILE: tests/tcg/arc/test_macros.h:176:
+#define TEST_BR2_OP_NOTTAKEN( testnum, inst, val1, val2 ) \

ERROR: space prohibited before that close parenthesis ')'
#8759: FILE: tests/tcg/arc/test_macros.h:176:
+#define TEST_BR2_OP_NOTTAKEN( testnum, inst, val1, val2 ) \

ERROR: Macros with complex values should be enclosed in parenthesis
#8759: FILE: tests/tcg/arc/test_macros.h:176:
+#define TEST_BR2_OP_NOTTAKEN( testnum, inst, val1, val2 ) \
+    test_ ## testnum:`                                    \
+    mov  r12,testnum`                                     \
+        mov  r1, val1`                                    \
+        mov  r2, val2`                                    \
+        sub.f 0,r1,r2`                                    \
+        inst @fail

ERROR: spaces required around that ':' (ctx:VxV)
#8760: FILE: tests/tcg/arc/test_macros.h:177:
+    test_ ## testnum:`                                    \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8761: FILE: tests/tcg/arc/test_macros.h:178:
+    mov  r12,testnum`                                     \
             ^

ERROR: space required after that ',' (ctx:VxV)
#8764: FILE: tests/tcg/arc/test_macros.h:181:
+        sub.f 0,r1,r2`                                    \
                ^

ERROR: space required after that ',' (ctx:VxV)
#8764: FILE: tests/tcg/arc/test_macros.h:181:
+        sub.f 0,r1,r2`                                    \
                   ^

ERROR: space prohibited after that open parenthesis '('
#8767: FILE: tests/tcg/arc/test_macros.h:184:
+#define TEST_BR_OP_TAKEN( testnum, inst, val1, val2 )  \

ERROR: space prohibited before that close parenthesis ')'
#8767: FILE: tests/tcg/arc/test_macros.h:184:
+#define TEST_BR_OP_TAKEN( testnum, inst, val1, val2 )  \

ERROR: Macros with complex values should be enclosed in parenthesis
#8767: FILE: tests/tcg/arc/test_macros.h:184:
+#define TEST_BR_OP_TAKEN( testnum, inst, val1, val2 )  \
+    test_ ## testnum:`                                  \
+        mov  r12, testnum`                              \
+        mov  r1, val1`                                  \
+        mov  r2, val2`                                  \
+        inst r1,r2,1f`                                  \
+        b @fail`                                        \
+        1:

ERROR: spaces required around that ':' (ctx:VxV)
#8768: FILE: tests/tcg/arc/test_macros.h:185:
+    test_ ## testnum:`                                  \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8772: FILE: tests/tcg/arc/test_macros.h:189:
+        inst r1,r2,1f`                                  \
                ^

ERROR: space required after that ',' (ctx:VxV)
#8772: FILE: tests/tcg/arc/test_macros.h:189:
+        inst r1,r2,1f`                                  \
                   ^

ERROR: spaces required around that ':' (ctx:VxE)
#8774: FILE: tests/tcg/arc/test_macros.h:191:
+        1:
          ^

ERROR: space prohibited after that open parenthesis '('
#8776: FILE: tests/tcg/arc/test_macros.h:193:
+#define TEST_BR_OP_NOTTAKEN( testnum, inst, val1, val2 ) \

ERROR: space prohibited before that close parenthesis ')'
#8776: FILE: tests/tcg/arc/test_macros.h:193:
+#define TEST_BR_OP_NOTTAKEN( testnum, inst, val1, val2 ) \

ERROR: Macros with complex values should be enclosed in parenthesis
#8776: FILE: tests/tcg/arc/test_macros.h:193:
+#define TEST_BR_OP_NOTTAKEN( testnum, inst, val1, val2 ) \
+    test_ ## testnum:`                                    \
+        mov  r12,testnum`                                 \
+        mov  r1, val1`                                    \
+        mov  r2, val2`                                    \
+        inst r1,r2,@fail

ERROR: spaces required around that ':' (ctx:VxV)
#8777: FILE: tests/tcg/arc/test_macros.h:194:
+    test_ ## testnum:`                                    \
                     ^

ERROR: space required after that ',' (ctx:VxV)
#8778: FILE: tests/tcg/arc/test_macros.h:195:
+        mov  r12,testnum`                                 \
                 ^

ERROR: space required after that ',' (ctx:VxV)
#8781: FILE: tests/tcg/arc/test_macros.h:198:
+        inst r1,r2,@fail
                ^

ERROR: space required after that ',' (ctx:VxV)
#8781: FILE: tests/tcg/arc/test_macros.h:198:
+        inst r1,r2,@fail
                   ^

ERROR: Macros with complex values should be enclosed in parenthesis
#8783: FILE: tests/tcg/arc/test_macros.h:200:
+#define ARCTEST_BEGIN \
+    .text`                                      \
+        .align 4 `                              \
+        .global main`                           \
+    main:                                       \
+        test_1:`                                \
+        mov r12,1`                              \
+        mov.f 0,0`                              \
+        bne @fail

ERROR: spaces required around that ':' (ctx:VxE)
#8787: FILE: tests/tcg/arc/test_macros.h:204:
+    main:                                       \
         ^

ERROR: spaces required around that ':' (ctx:VxV)
#8788: FILE: tests/tcg/arc/test_macros.h:205:
+        test_1:`                                \
               ^

ERROR: space required after that ',' (ctx:VxV)
#8789: FILE: tests/tcg/arc/test_macros.h:206:
+        mov r12,1`                              \
                ^

ERROR: space required after that ',' (ctx:VxV)
#8790: FILE: tests/tcg/arc/test_macros.h:207:
+        mov.f 0,0`                              \
                ^

ERROR: Macros with complex values should be enclosed in parenthesis
#8793: FILE: tests/tcg/arc/test_macros.h:210:
+#define ARCTEST_END \
+         .align 4 ` \
+1:`\
+        st 1,[0xf0000008]`\
+        b @1b`\
+fail:`\
+        mov     r2, '['`\
+        st      r2, [0x90000000]`\
+        mov     r2, 'F'`\
+        st      r2, [0x90000000]`\
+        mov     r2, 'a'`\
+        st      r2, [0x90000000]`\
+        mov     r2, 'i'`\
+        st      r2, [0x90000000]`\
+        mov     r2, 'l'`\
+        st      r2, [0x90000000]`\
+        mov     r2, ']'`\
+        st      r2, [0x90000000]`\
+        mov     r2, ' '`\
+        st      r2, [0x90000000]`\
+        mov r13, r12`\
+        mov r15, 0x30`\
+        mov r14, r12`\
+loop_z: `\
+        sub.f   r13, r13, 0x0A`\
+        add.pl  r15, r15, 1`\
+        mov.pl  r14, r13 `\
+        bpl     @loop_z`\
+        st      r15, [0x90000000]`\
+        add     r14, r14, 0x30`\
+        st      r14, [0x90000000]`\
+        mov     r2, '\n'`\
+        st      r2, [0x90000000]`\
+        b       1b`

ERROR: spaces required around that ':' (ctx:VxV)
#8795: FILE: tests/tcg/arc/test_macros.h:212:
+1:`\
  ^

ERROR: space required after that ',' (ctx:VxV)
#8796: FILE: tests/tcg/arc/test_macros.h:213:
+        st 1,[0xf0000008]`\
             ^

ERROR: spaces required around that ':' (ctx:VxV)
#8798: FILE: tests/tcg/arc/test_macros.h:215:
+fail:`\
     ^

ERROR: spaces required around that ':' (ctx:VxW)
#8816: FILE: tests/tcg/arc/test_macros.h:233:
+loop_z: `\
       ^

ERROR: Macros with complex values should be enclosed in parenthesis
#8828: FILE: tests/tcg/arc/test_macros.h:245:
+#define PASS_TEST(name)\
+    .data `            \
+2010:`\
+    .ascii "[PASS] ",name ,"\n\0"` \
+  .align 4`\
+  .text`\
+  mov_s     r11, @2010b`\
+  1010:`\
+    ldb.ab  r12, [r11, 1]`\
+    breq    r12, 0, @1011f`\
+    stb     r12, [0x90000000]`\
+    j       @1010b`\
+  1011:`

ERROR: spaces required around that ':' (ctx:VxV)
#8830: FILE: tests/tcg/arc/test_macros.h:247:
+2010:`\
     ^

ERROR: space required after that ',' (ctx:VxV)
#8831: FILE: tests/tcg/arc/test_macros.h:248:
+    .ascii "[PASS] ",name ,"\n\0"` \
                     ^

ERROR: space required after that ',' (ctx:WxV)
#8831: FILE: tests/tcg/arc/test_macros.h:248:
+    .ascii "[PASS] ",name ,"\n\0"` \
                           ^

ERROR: spaces required around that ':' (ctx:VxV)
#8835: FILE: tests/tcg/arc/test_macros.h:252:
+  1010:`\
       ^

ERROR: spaces required around that ':' (ctx:VxV)
#8840: FILE: tests/tcg/arc/test_macros.h:257:
+  1011:`
       ^

ERROR: line over 90 characters
#8858: FILE: tests/tcg/configure.sh:98:
+    arc|alpha|cris|hppa|i386|lm32|microblaze|microblazeel|m68k|openrisc|riscv64|s390x|sh4|sparc64)

total: 152 errors, 69 warnings, 8350 lines checked

Patch 14/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

15/15 Checking commit dc09f35a4b7e (tests/acceptance: ARC: Add linux boot testing.)
ERROR: line over 90 characters
#48: FILE: tests/acceptance/boot_linux_console.py:997:
+        tar_url = ('https://github.com/cupertinomiranda/arc-qemu-resources/archive/master.tar.gz')

ERROR: line over 90 characters
#84: FILE: tests/acceptance/boot_linux_console.py:1154:
+        self.vm.add_args('-netdev', 'user,id=net0,hostfwd=tcp::5558-:21,hostfwd=tcp::5557-:23')

total: 2 errors, 0 warnings, 70 lines checked

Patch 15/15 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20201111161758.9636-1-cupertinomiranda@gmail.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com