qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v6 00/30] More SH4 clean ups (including code style series)
@ 2021-10-29 21:02 BALATON Zoltan
  2021-10-29 21:02 ` [PATCH v6 15/30] hw/intc/sh_intc: Use existing macro instead of local one BALATON Zoltan
                   ` (31 more replies)
  0 siblings, 32 replies; 53+ messages in thread
From: BALATON Zoltan @ 2021-10-29 21:02 UTC (permalink / raw)
  To: qemu-devel; +Cc: Peter Maydell, Richard Henderson, Magnus Damm, Yoshinori Sato

This now includes the previous code style clean up series so
everything is in the series now. This got big enough for now so I'll
wait until these are merged (hopefully before the freeze) before going
on so I don't have to carry them in my tree. Only plan to submit
another version if needed from review otherwise that's it for now,
hopefully this is the last version.

v6: Found some lines with wrong indentation missed in code style clean
up and one line missed removing sh_intc_source from header so I'm
posting the whole series with these two small fixes, otherwise same as
v5 but rebased on fixed code style series which is now also included.

v5: Do not use hw_error, back to qemu_log_mask; split up QOM-ify patch
mode and drop one memory alias for sh_serial

v4: Drop changes from fprintf before abort() as Philippe said, only
change sh_serial now which can use hw_error instead; missed two more
debug printfs in sh_timer that I've also added now; some more clean ups

v3: Correct mistakes found in review, drop size change of sh_intc
iomem as that was wrong so only rename it, more clean ups

v2: separate sh_serial trace events, split QOM-ify patch for easier
review and some more patches to clean up sh_intc a bit

Regards,

BALATON Zoltan (30):
  hw/sh4: Coding style: Remove tabs
  hw/sh4: Coding style: Fix multi-line comments
  hw/sh4: Coding style: White space fixes
  hw/sh4: Coding style: Add missing braces
  hw/sh4: Coding style: Remove unnecessary casts
  hw/sh4: Fix typos in a comment
  hw/sh4: Change debug printfs to traces
  hw/sh4/r2d: Use error_report instead of fprintf to stderr
  hw/char/sh_serial: Do not abort on invalid access
  hw/char/sh_serial: Rename type sh_serial_state to SHSerialState
  hw/char/sh_serial: Embed QEMUTimer in state struct
  hw/char/sh_serial: Split off sh_serial_reset() from sh_serial_init()
  hw/char/sh_serial: QOM-ify
  hw/char/sh_serial: Add device id to trace output
  hw/intc/sh_intc: Use existing macro instead of local one
  hw/intc/sh_intc: Turn some defines into an enum
  hw/intc/sh_intc: Rename iomem region
  hw/intc/sh_intc: Drop another useless macro
  hw/intc/sh_intc: Move sh_intc_register() closer to its only user
  hw/intc/sh_intc: Remove excessive parenthesis
  hw/intc/sh_intc: Use array index instead of pointer arithmetics
  hw/intc/sh_intc: Inline and drop sh_intc_source() function
  hw/intc/sh_intc: Replace abort() with g_assert_not_reached()
  hw/intc/sh_intc: Avoid using continue in loops
  hw/intc/sh_intc: Simplify allocating sources array
  hw/intc/sh_intc: Remove unneeded local variable initialisers
  hw/timer/sh_timer: Rename sh_timer_state to SHTimerState
  hw/timer/sh_timer: Do not wrap lines that are not too long
  hw/timer/sh_timer: Fix timer memory region size
  hw/timer/sh_timer: Remove use of hw_error

 hw/char/sh_serial.c      |  236 +++---
 hw/char/trace-events     |    4 +
 hw/intc/sh_intc.c        |  546 ++++++-------
 hw/intc/trace-events     |    8 +
 hw/pci-host/sh_pci.c     |   10 +-
 hw/sh4/r2d.c             |   69 +-
 hw/sh4/sh7750.c          |  637 ++++++++-------
 hw/sh4/sh7750_regnames.c |  148 ++--
 hw/sh4/sh7750_regs.h     | 1600 +++++++++++++++++++-------------------
 hw/sh4/shix.c            |   12 +-
 hw/sh4/trace-events      |    3 +
 hw/sh4/trace.h           |    1 +
 hw/timer/sh_timer.c      |  147 ++--
 hw/timer/trace-events    |    5 +
 include/hw/sh4/sh.h      |   19 +-
 include/hw/sh4/sh_intc.h |    2 +-
 meson.build              |    1 +
 17 files changed, 1739 insertions(+), 1709 deletions(-)
 create mode 100644 hw/sh4/trace-events
 create mode 100644 hw/sh4/trace.h

-- 
2.21.4



^ permalink raw reply	[flat|nested] 53+ messages in thread

end of thread, other threads:[~2021-10-30 14:12 UTC | newest]

Thread overview: 53+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-29 21:02 [PATCH v6 00/30] More SH4 clean ups (including code style series) BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 15/30] hw/intc/sh_intc: Use existing macro instead of local one BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 25/30] hw/intc/sh_intc: Simplify allocating sources array BALATON Zoltan
2021-10-29 22:06   ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 22/30] hw/intc/sh_intc: Inline and drop sh_intc_source() function BALATON Zoltan
2021-10-29 22:45   ` Philippe Mathieu-Daudé
2021-10-29 23:48     ` BALATON Zoltan
2021-10-30  9:45       ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 12/30] hw/char/sh_serial: Split off sh_serial_reset() from sh_serial_init() BALATON Zoltan
2021-10-29 22:01   ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 28/30] hw/timer/sh_timer: Do not wrap lines that are not too long BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 19/30] hw/intc/sh_intc: Move sh_intc_register() closer to its only user BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 20/30] hw/intc/sh_intc: Remove excessive parenthesis BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 27/30] hw/timer/sh_timer: Rename sh_timer_state to SHTimerState BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 10/30] hw/char/sh_serial: Rename type sh_serial_state to SHSerialState BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 24/30] hw/intc/sh_intc: Avoid using continue in loops BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 07/30] hw/sh4: Change debug printfs to traces BALATON Zoltan
2021-10-30 14:11   ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 14/30] hw/char/sh_serial: Add device id to trace output BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 30/30] hw/timer/sh_timer: Remove use of hw_error BALATON Zoltan
2021-10-29 22:06   ` Philippe Mathieu-Daudé
2021-10-29 23:39     ` BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 18/30] hw/intc/sh_intc: Drop another useless macro BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 09/30] hw/char/sh_serial: Do not abort on invalid access BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 05/30] hw/sh4: Coding style: Remove unnecessary casts BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 03/30] hw/sh4: Coding style: White space fixes BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 21/30] hw/intc/sh_intc: Use array index instead of pointer arithmetics BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 17/30] hw/intc/sh_intc: Rename iomem region BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 16/30] hw/intc/sh_intc: Turn some defines into an enum BALATON Zoltan
2021-10-30 10:42   ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 26/30] hw/intc/sh_intc: Remove unneeded local variable initialisers BALATON Zoltan
2021-10-29 22:28   ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 04/30] hw/sh4: Coding style: Add missing braces BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 08/30] hw/sh4/r2d: Use error_report instead of fprintf to stderr BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 01/30] hw/sh4: Coding style: Remove tabs BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 11/30] hw/char/sh_serial: Embed QEMUTimer in state struct BALATON Zoltan
2021-10-29 22:00   ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 29/30] hw/timer/sh_timer: Fix timer memory region size BALATON Zoltan
2021-10-29 22:26   ` Philippe Mathieu-Daudé
2021-10-29 23:36     ` BALATON Zoltan
2021-10-30  9:41       ` Philippe Mathieu-Daudé
2021-10-29 23:27   ` [PATCH v7 " BALATON Zoltan
2021-10-30  9:42     ` Philippe Mathieu-Daudé
2021-10-29 21:02 ` [PATCH v6 02/30] hw/sh4: Coding style: Fix multi-line comments BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 06/30] hw/sh4: Fix typos in a comment BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 23/30] hw/intc/sh_intc: Replace abort() with g_assert_not_reached() BALATON Zoltan
2021-10-29 21:02 ` [PATCH v6 13/30] hw/char/sh_serial: QOM-ify BALATON Zoltan
2021-10-29 22:36   ` Philippe Mathieu-Daudé
2021-10-29 23:44     ` BALATON Zoltan
2021-10-29 21:57 ` [PATCH v6 00/30] More SH4 clean ups (including code style series) Philippe Mathieu-Daudé
2021-10-29 23:52   ` BALATON Zoltan
2021-10-30 10:05 ` Philippe Mathieu-Daudé
2021-10-30 11:50   ` BALATON Zoltan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).