[v2,0/3] Serial device cleanup

Message

Bernhard Beschow Sept. 5, 2024, 7:38 a.m. UTC

This series extracts TYPE_SERIAL_MM into its own module and introduces a
dedicated header for TYPE_SERIAL_ISA. The result is that no board directly
depends on CONFIG_SERIAL or includes hw/char/serial.h any more.

The first two patches are only paritally reviewed and would benefit from a few
more R-b tags. Since the changes are mostly mechanical this may be "boring".

v2:
* Rebase onto master which eliminates patch "hw/ppc/Kconfig: Add missing
  SERIAL_ISA dependency to POWERNV machine" from the series (already applied)
* Drop patch "hw/char/serial: Remove unused funtion" (Zoltan)
* Add Mark's R-b and Andrew's T-b tags

Bernhard Beschow (3):
  hw: Remove unused inclusion of hw/char/serial.h
  hw/char/serial.h: Extract serial-isa.h
  hw/char: Extract serial-mm

 include/hw/arm/aspeed_soc.h         |   2 +-
 include/hw/char/mchp_pfsoc_mmuart.h |   2 +-
 include/hw/char/serial-isa.h        |  38 +++++++
 include/hw/char/serial-mm.h         |  52 +++++++++
 include/hw/char/serial.h            |  28 -----
 hw/arm/allwinner-a10.c              |   2 +-
 hw/arm/allwinner-h3.c               |   2 +-
 hw/arm/allwinner-r40.c              |   2 +-
 hw/arm/aspeed_ast2400.c             |   2 +-
 hw/arm/aspeed_soc_common.c          |   2 +-
 hw/arm/kzm.c                        |   2 +-
 hw/arm/msf2-soc.c                   |   2 +-
 hw/arm/musicpal.c                   |   2 +-
 hw/arm/npcm7xx.c                    |   2 +-
 hw/arm/pxa2xx.c                     |   2 +-
 hw/char/omap_uart.c                 |   2 +-
 hw/char/riscv_htif.c                |   1 -
 hw/char/serial-isa.c                |   1 +
 hw/char/serial-mm.c                 | 157 ++++++++++++++++++++++++++++
 hw/char/serial.c                    | 126 ----------------------
 hw/display/sm501.c                  |   2 +-
 hw/hppa/machine.c                   |   2 +-
 hw/i386/microvm-dt.c                |   2 +-
 hw/i386/microvm.c                   |   2 +-
 hw/i386/pc.c                        |   2 +-
 hw/isa/isa-superio.c                |   2 +-
 hw/isa/vt82c686.c                   |   2 +-
 hw/loongarch/virt.c                 |   2 +-
 hw/microblaze/petalogix_ml605_mmu.c |   2 +-
 hw/mips/boston.c                    |   2 +-
 hw/mips/jazz.c                      |   2 +-
 hw/mips/loongson3_virt.c            |   2 +-
 hw/mips/malta.c                     |   2 +-
 hw/mips/mipssim.c                   |   2 +-
 hw/openrisc/openrisc_sim.c          |   2 +-
 hw/openrisc/virt.c                  |   2 +-
 hw/ppc/e500.c                       |   2 +-
 hw/ppc/pnv.c                        |   2 +-
 hw/ppc/ppc405_uc.c                  |   2 +-
 hw/ppc/ppc440_bamboo.c              |   2 +-
 hw/ppc/prep.c                       |   1 -
 hw/ppc/sam460ex.c                   |   2 +-
 hw/ppc/virtex_ml507.c               |   2 +-
 hw/riscv/sifive_e.c                 |   1 -
 hw/riscv/sifive_u.c                 |   1 -
 hw/riscv/virt.c                     |   2 +-
 hw/sparc64/niagara.c                |   2 +-
 hw/sparc64/sun4u.c                  |   3 +-
 hw/xtensa/xtfpga.c                  |   2 +-
 hw/arm/Kconfig                      |  20 ++--
 hw/char/Kconfig                     |   4 +
 hw/char/meson.build                 |   1 +
 hw/display/Kconfig                  |   2 +-
 hw/hppa/Kconfig                     |   2 +-
 hw/loongarch/Kconfig                |   2 +-
 hw/microblaze/Kconfig               |   2 +-
 hw/mips/Kconfig                     |  10 +-
 hw/openrisc/Kconfig                 |   4 +-
 hw/ppc/Kconfig                      |  10 +-
 hw/riscv/Kconfig                    |   2 +-
 hw/sparc64/Kconfig                  |   1 +
 hw/xtensa/Kconfig                   |   2 +-
 62 files changed, 322 insertions(+), 225 deletions(-)
 create mode 100644 include/hw/char/serial-isa.h
 create mode 100644 include/hw/char/serial-mm.h
 create mode 100644 hw/char/serial-mm.c

Comments

Paolo Bonzini Sept. 5, 2024, 7:56 a.m. UTC | #1

Queued, thanks.

Paolo