mbox series

[00/11] Unified peripheral emulation for Renesas chips

Message ID 20210527052122.97103-1-ysato@users.sourceforge.jp (mailing list archive)
Headers show
Series Unified peripheral emulation for Renesas chips | expand

Message

Yoshinori Sato May 27, 2021, 5:21 a.m. UTC
Renesas SH4 and RX have similar peripherals.
Integrate the emulation code for these peripherals.

Migrate peripherals.
SH4 sh_timer(TMU) -> renesas_timer
SH4 sh_serial(SCI/SCIF) -> renesas_sci
RX renesas_cmt(CMT) -> renesas_timer
RX renesas_sci(SCIa) -> renesas_sci

I want to process the peripheral clocks of SH4 and RX in the same way,
so I added a new clock generator. This will generate a peripheral clock
from the master clock.

Yoshinori Sato (11):
  hw/char: Renesas SCI module.
  hw/char: remove sh_serial.
  hw/timer: Renesas TMU/CMT module.
  hw/timer: Remove sh_timer.
  hw/timer: Remove renesas_cmt.
  hw/rx: Add RX62N Clock generator
  hw/timer: Renesas 8bit timer.
  hw/rx: rx62n use new hw modules.
  hw/sh4: sh7750 Add CPG.
  hw/sh4: sh7750 use new hw modules.
  hw/rx: rx-gdbsim Add bootstrup for linux

 include/hw/char/renesas_sci.h    |  129 +++-
 include/hw/rx/rx62n-cpg.h        |   72 +++
 include/hw/rx/rx62n.h            |   34 +-
 include/hw/sh4/sh.h              |    8 -
 include/hw/sh4/sh7751-cpg.h      |   94 +++
 include/hw/timer/renesas_cmt.h   |   43 --
 include/hw/timer/renesas_timer.h |   96 +++
 include/hw/timer/renesas_tmr.h   |   58 --
 include/hw/timer/renesas_tmr8.h  |   67 ++
 include/hw/timer/tmu012.h        |   23 -
 hw/char/renesas_sci.c            | 1039 +++++++++++++++++++++++++-----
 hw/char/sh_serial.c              |  431 -------------
 hw/rx/rx-gdbsim.c                |   89 +--
 hw/rx/rx62n-cpg.c                |  344 ++++++++++
 hw/rx/rx62n.c                    |   64 +-
 hw/sh4/sh7750.c                  |  110 +++-
 hw/sh4/sh7751-cpg.c              |  457 +++++++++++++
 hw/timer/renesas_cmt.c           |  283 --------
 hw/timer/renesas_timer.c         |  644 ++++++++++++++++++
 hw/timer/renesas_tmr.c           |  493 --------------
 hw/timer/renesas_tmr8.c          |  540 ++++++++++++++++
 hw/timer/sh_timer.c              |  368 -----------
 MAINTAINERS                      |    8 +-
 hw/char/Kconfig                  |    3 -
 hw/char/meson.build              |    1 -
 hw/rx/Kconfig                    |    4 +-
 hw/rx/meson.build                |    2 +-
 hw/sh4/Kconfig                   |    4 +-
 hw/sh4/meson.build               |    1 +
 hw/timer/Kconfig                 |   12 +-
 hw/timer/meson.build             |    5 +-
 31 files changed, 3501 insertions(+), 2025 deletions(-)
 create mode 100644 include/hw/rx/rx62n-cpg.h
 create mode 100644 include/hw/sh4/sh7751-cpg.h
 delete mode 100644 include/hw/timer/renesas_cmt.h
 create mode 100644 include/hw/timer/renesas_timer.h
 delete mode 100644 include/hw/timer/renesas_tmr.h
 create mode 100644 include/hw/timer/renesas_tmr8.h
 delete mode 100644 include/hw/timer/tmu012.h
 delete mode 100644 hw/char/sh_serial.c
 create mode 100644 hw/rx/rx62n-cpg.c
 create mode 100644 hw/sh4/sh7751-cpg.c
 delete mode 100644 hw/timer/renesas_cmt.c
 create mode 100644 hw/timer/renesas_timer.c
 delete mode 100644 hw/timer/renesas_tmr.c
 create mode 100644 hw/timer/renesas_tmr8.c
 delete mode 100644 hw/timer/sh_timer.c

Comments

no-reply@patchew.org May 27, 2021, 5:33 a.m. UTC | #1
Patchew URL: https://patchew.org/QEMU/20210527052122.97103-1-ysato@users.sourceforge.jp/



Hi,

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

Type: series
Message-id: 20210527052122.97103-1-ysato@users.sourceforge.jp
Subject: [PATCH 00/11] Unified peripheral emulation for Renesas chips

=== 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/20210527052122.97103-1-ysato@users.sourceforge.jp -> patchew/20210527052122.97103-1-ysato@users.sourceforge.jp
Switched to a new branch 'test'
0d61629 hw/rx: rx-gdbsim Add bootstrup for linux
5595fd4 hw/sh4: sh7750 use new hw modules.
f85d3d2 hw/sh4: sh7750 Add CPG.
9422b37 hw/rx: rx62n use new hw modules.
88d7ca2 hw/timer: Renesas 8bit timer.
afe51ff hw/rx: Add RX62N Clock generator
87c0b3e hw/timer: Remove renesas_cmt.
e0a4816 hw/timer: Remove sh_timer.
0f983ff hw/timer: Renesas TMU/CMT module.
a5ba8bb hw/char: remove sh_serial.
872e5de hw/char: Renesas SCI module.

=== OUTPUT BEGIN ===
1/11 Checking commit 872e5de1daf1 (hw/char: Renesas SCI module.)
2/11 Checking commit a5ba8bb0a18d (hw/char: remove sh_serial.)
3/11 Checking commit 0f983ff94436 (hw/timer: Renesas TMU/CMT module.)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#44: 
new file mode 100644

total: 0 errors, 1 warnings, 752 lines checked

Patch 3/11 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/11 Checking commit e0a48164d14f (hw/timer: Remove sh_timer.)
5/11 Checking commit 87c0b3e06524 (hw/timer: Remove renesas_cmt.)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#30: 
deleted file mode 100644

total: 0 errors, 1 warnings, 7 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 afe51ff3750d (hw/rx: Add RX62N Clock generator)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#31: 
new file mode 100644

total: 0 errors, 1 warnings, 561 lines checked

Patch 6/11 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
7/11 Checking commit 88d7ca270dfb (hw/timer: Renesas 8bit timer.)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#50: 
deleted file mode 100644

total: 0 errors, 1 warnings, 626 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 9422b37a5c14 (hw/rx: rx62n use new hw modules.)
9/11 Checking commit f85d3d235566 (hw/sh4: sh7750 Add CPG.)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#98: 
new file mode 100644

total: 0 errors, 1 warnings, 616 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 5595fd490cbc (hw/sh4: sh7750 use new hw modules.)
ERROR: space required before the open parenthesis '('
#88: FILE: hw/sh4/sh7750.c:819:
+    switch(feat) {

ERROR: braces {} are necessary for all arms of this statement
#105: FILE: hw/sh4/sh7750.c:836:
+    if (tei_source)
[...]

ERROR: braces {} are necessary for all arms of this statement
#107: FILE: hw/sh4/sh7750.c:838:
+    if (bri_source)
[...]

ERROR: code indent should never use tabs
#142: FILE: hw/sh4/sh7750.c:916:
+^I^Is->intc.irqs[TMU2_TICPI], cpg);$

ERROR: code indent should never use tabs
#157: FILE: hw/sh4/sh7750.c:937:
+^I^I    NULL, NULL, cpg);$

total: 5 errors, 0 warnings, 153 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 0d616299f2f2 (hw/rx: rx-gdbsim Add bootstrup for linux)
=== OUTPUT END ===

Test command exited with code: 1


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