* [PATCH 0/5] Serial device cleanup
@ 2024-08-14 18:15 Bernhard Beschow
2024-08-14 18:15 ` [PATCH 1/5] hw: Remove unused inclusion of hw/char/serial.h Bernhard Beschow
` (4 more replies)
0 siblings, 5 replies; 16+ messages in thread
From: Bernhard Beschow @ 2024-08-14 18:15 UTC (permalink / raw)
To: qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Alistair Francis,
Artyom Tarasenko, Jia Liu, Helge Deller, Max Filippov,
Subbaraya Sundeep, Weiwei Li, Marc-André Lureau,
Nicholas Piggin, Mark Cave-Ayland, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao,
Bernhard Beschow
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.
Bernhard Beschow (5):
hw: Remove unused inclusion of hw/char/serial.h
hw/char/serial: Remove unused funtion
hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine
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 | 30 ------
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 | 133 -----------------------
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 | 11 +-
hw/riscv/Kconfig | 2 +-
hw/sparc64/Kconfig | 1 +
hw/xtensa/Kconfig | 2 +-
62 files changed, 323 insertions(+), 234 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
--
2.46.0
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/5] hw: Remove unused inclusion of hw/char/serial.h
2024-08-14 18:15 [PATCH 0/5] Serial device cleanup Bernhard Beschow
@ 2024-08-14 18:15 ` Bernhard Beschow
2024-08-21 14:39 ` Mark Cave-Ayland
2024-08-14 18:15 ` [PATCH 2/5] hw/char/serial: Remove unused funtion Bernhard Beschow
` (3 subsequent siblings)
4 siblings, 1 reply; 16+ messages in thread
From: Bernhard Beschow @ 2024-08-14 18:15 UTC (permalink / raw)
To: qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Alistair Francis,
Artyom Tarasenko, Jia Liu, Helge Deller, Max Filippov,
Subbaraya Sundeep, Weiwei Li, Marc-André Lureau,
Nicholas Piggin, Mark Cave-Ayland, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao,
Bernhard Beschow
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
hw/char/riscv_htif.c | 1 -
hw/ppc/prep.c | 1 -
hw/riscv/sifive_e.c | 1 -
hw/riscv/sifive_u.c | 1 -
4 files changed, 4 deletions(-)
diff --git a/hw/char/riscv_htif.c b/hw/char/riscv_htif.c
index 9bef60def1..54fd55c3e6 100644
--- a/hw/char/riscv_htif.c
+++ b/hw/char/riscv_htif.c
@@ -24,7 +24,6 @@
#include "qapi/error.h"
#include "qemu/log.h"
#include "hw/char/riscv_htif.h"
-#include "hw/char/serial.h"
#include "chardev/char.h"
#include "chardev/char-fe.h"
#include "qemu/timer.h"
diff --git a/hw/ppc/prep.c b/hw/ppc/prep.c
index 4eb5477069..fb58c312ac 100644
--- a/hw/ppc/prep.c
+++ b/hw/ppc/prep.c
@@ -25,7 +25,6 @@
#include "qemu/osdep.h"
#include "hw/rtc/m48t59.h"
-#include "hw/char/serial.h"
#include "hw/block/fdc.h"
#include "net/net.h"
#include "hw/isa/isa.h"
diff --git a/hw/riscv/sifive_e.c b/hw/riscv/sifive_e.c
index 87d9602383..5a1959f2a9 100644
--- a/hw/riscv/sifive_e.c
+++ b/hw/riscv/sifive_e.c
@@ -35,7 +35,6 @@
#include "hw/boards.h"
#include "hw/loader.h"
#include "hw/sysbus.h"
-#include "hw/char/serial.h"
#include "hw/misc/unimp.h"
#include "target/riscv/cpu.h"
#include "hw/riscv/riscv_hart.h"
diff --git a/hw/riscv/sifive_u.c b/hw/riscv/sifive_u.c
index af5f923f54..efc8443c84 100644
--- a/hw/riscv/sifive_u.c
+++ b/hw/riscv/sifive_u.c
@@ -43,7 +43,6 @@
#include "hw/irq.h"
#include "hw/loader.h"
#include "hw/sysbus.h"
-#include "hw/char/serial.h"
#include "hw/cpu/cluster.h"
#include "hw/misc/unimp.h"
#include "hw/sd/sd.h"
--
2.46.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 2/5] hw/char/serial: Remove unused funtion
2024-08-14 18:15 [PATCH 0/5] Serial device cleanup Bernhard Beschow
2024-08-14 18:15 ` [PATCH 1/5] hw: Remove unused inclusion of hw/char/serial.h Bernhard Beschow
@ 2024-08-14 18:15 ` Bernhard Beschow
2024-08-14 21:38 ` BALATON Zoltan
2024-08-14 18:15 ` [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine Bernhard Beschow
` (2 subsequent siblings)
4 siblings, 1 reply; 16+ messages in thread
From: Bernhard Beschow @ 2024-08-14 18:15 UTC (permalink / raw)
To: qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Alistair Francis,
Artyom Tarasenko, Jia Liu, Helge Deller, Max Filippov,
Subbaraya Sundeep, Weiwei Li, Marc-André Lureau,
Nicholas Piggin, Mark Cave-Ayland, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao,
Bernhard Beschow
The serial port's frequency is set via the "baudbase" property nowadays.
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
include/hw/char/serial.h | 2 --
hw/char/serial.c | 7 -------
2 files changed, 9 deletions(-)
diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h
index 6e14099ee7..40aad21df3 100644
--- a/include/hw/char/serial.h
+++ b/include/hw/char/serial.h
@@ -93,8 +93,6 @@ struct SerialMM {
extern const VMStateDescription vmstate_serial;
extern const MemoryRegionOps serial_io_ops;
-void serial_set_frequency(SerialState *s, uint32_t frequency);
-
#define TYPE_SERIAL "serial"
OBJECT_DECLARE_SIMPLE_TYPE(SerialState, SERIAL)
diff --git a/hw/char/serial.c b/hw/char/serial.c
index d8b2db5082..6c5c4a23c7 100644
--- a/hw/char/serial.c
+++ b/hw/char/serial.c
@@ -951,13 +951,6 @@ static void serial_unrealize(DeviceState *dev)
qemu_unregister_reset(serial_reset, s);
}
-/* Change the main reference oscillator frequency. */
-void serial_set_frequency(SerialState *s, uint32_t frequency)
-{
- s->baudbase = frequency;
- serial_update_parameters(s);
-}
-
const MemoryRegionOps serial_io_ops = {
.read = serial_ioport_read,
.write = serial_ioport_write,
--
2.46.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine
2024-08-14 18:15 [PATCH 0/5] Serial device cleanup Bernhard Beschow
2024-08-14 18:15 ` [PATCH 1/5] hw: Remove unused inclusion of hw/char/serial.h Bernhard Beschow
2024-08-14 18:15 ` [PATCH 2/5] hw/char/serial: Remove unused funtion Bernhard Beschow
@ 2024-08-14 18:15 ` Bernhard Beschow
2024-08-16 9:18 ` Philippe Mathieu-Daudé
2024-08-21 14:42 ` Mark Cave-Ayland
2024-08-14 18:15 ` [PATCH 4/5] hw/char/serial.h: Extract serial-isa.h Bernhard Beschow
2024-08-14 18:15 ` [PATCH 5/5] hw/char: Extract serial-mm Bernhard Beschow
4 siblings, 2 replies; 16+ messages in thread
From: Bernhard Beschow @ 2024-08-14 18:15 UTC (permalink / raw)
To: qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Alistair Francis,
Artyom Tarasenko, Jia Liu, Helge Deller, Max Filippov,
Subbaraya Sundeep, Weiwei Li, Marc-André Lureau,
Nicholas Piggin, Mark Cave-Ayland, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao,
Bernhard Beschow
The machine calls serial_hds_isa_init() which is provided by serial-isa.c,
guarded by SERIAL_ISA.
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
hw/ppc/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/ppc/Kconfig b/hw/ppc/Kconfig
index c235519881..5addad1124 100644
--- a/hw/ppc/Kconfig
+++ b/hw/ppc/Kconfig
@@ -39,6 +39,7 @@ config POWERNV
select PCI_POWERNV
select PCA9552
select PCA9554
+ select SERIAL_ISA
select SSI
select SSI_M25P80
select PNV_SPI
--
2.46.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 4/5] hw/char/serial.h: Extract serial-isa.h
2024-08-14 18:15 [PATCH 0/5] Serial device cleanup Bernhard Beschow
` (2 preceding siblings ...)
2024-08-14 18:15 ` [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine Bernhard Beschow
@ 2024-08-14 18:15 ` Bernhard Beschow
2024-08-21 14:44 ` Mark Cave-Ayland
2024-08-14 18:15 ` [PATCH 5/5] hw/char: Extract serial-mm Bernhard Beschow
4 siblings, 1 reply; 16+ messages in thread
From: Bernhard Beschow @ 2024-08-14 18:15 UTC (permalink / raw)
To: qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Alistair Francis,
Artyom Tarasenko, Jia Liu, Helge Deller, Max Filippov,
Subbaraya Sundeep, Weiwei Li, Marc-André Lureau,
Nicholas Piggin, Mark Cave-Ayland, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao,
Bernhard Beschow
The includes where updated based on compile errors. Now, the inclusion of the
header roughly matches Kconfig dependencies:
# grep -r -e "select SERIAL_ISA"
hw/ppc/Kconfig: select SERIAL_ISA
hw/isa/Kconfig: select SERIAL_ISA
hw/sparc64/Kconfig: select SERIAL_ISA
hw/i386/Kconfig: select SERIAL_ISA
hw/i386/Kconfig: select SERIAL_ISA # for serial_hds_isa_init()
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
include/hw/char/serial-isa.h | 38 ++++++++++++++++++++++++++++++++++++
include/hw/char/serial.h | 9 ---------
hw/char/serial-isa.c | 1 +
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/ppc/pnv.c | 2 +-
hw/sparc64/sun4u.c | 1 +
10 files changed, 46 insertions(+), 15 deletions(-)
create mode 100644 include/hw/char/serial-isa.h
diff --git a/include/hw/char/serial-isa.h b/include/hw/char/serial-isa.h
new file mode 100644
index 0000000000..8517afa128
--- /dev/null
+++ b/include/hw/char/serial-isa.h
@@ -0,0 +1,38 @@
+/*
+ * QEMU ISA 16550A UART emulation
+ *
+ * Copyright (c) 2003-2004 Fabrice Bellard
+ * Copyright (c) 2008 Citrix Systems, Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+
+#ifndef HW_SERIAL_ISA_H
+#define HW_SERIAL_ISA_H
+
+#include "hw/isa/isa.h"
+
+#define MAX_ISA_SERIAL_PORTS 4
+
+#define TYPE_ISA_SERIAL "isa-serial"
+void serial_hds_isa_init(ISABus *bus, int from, int to);
+void isa_serial_set_iobase(ISADevice *serial, hwaddr iobase);
+void isa_serial_set_enabled(ISADevice *serial, bool enabled);
+
+#endif
diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h
index 40aad21df3..ccf322a031 100644
--- a/include/hw/char/serial.h
+++ b/include/hw/char/serial.h
@@ -104,13 +104,4 @@ SerialMM *serial_mm_init(MemoryRegion *address_space,
qemu_irq irq, int baudbase,
Chardev *chr, enum device_endian end);
-/* serial-isa.c */
-
-#define MAX_ISA_SERIAL_PORTS 4
-
-#define TYPE_ISA_SERIAL "isa-serial"
-void serial_hds_isa_init(ISABus *bus, int from, int to);
-void isa_serial_set_iobase(ISADevice *serial, hwaddr iobase);
-void isa_serial_set_enabled(ISADevice *serial, bool enabled);
-
#endif
diff --git a/hw/char/serial-isa.c b/hw/char/serial-isa.c
index 329b352b9a..b562ec9d37 100644
--- a/hw/char/serial-isa.c
+++ b/hw/char/serial-isa.c
@@ -29,6 +29,7 @@
#include "sysemu/sysemu.h"
#include "hw/acpi/acpi_aml_interface.h"
#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "hw/isa/isa.h"
#include "hw/qdev-properties.h"
#include "migration/vmstate.h"
diff --git a/hw/i386/microvm-dt.c b/hw/i386/microvm-dt.c
index b3049e4f9f..fc5db6ed7f 100644
--- a/hw/i386/microvm-dt.c
+++ b/hw/i386/microvm-dt.c
@@ -34,7 +34,7 @@
#include "qemu/cutils.h"
#include "qapi/error.h"
#include "sysemu/device_tree.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "hw/i386/fw_cfg.h"
#include "hw/rtc/mc146818rtc.h"
#include "hw/sysbus.h"
diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c
index 40edcee7af..05c882669c 100644
--- a/hw/i386/microvm.c
+++ b/hw/i386/microvm.c
@@ -39,7 +39,7 @@
#include "hw/intc/i8259.h"
#include "hw/timer/i8254.h"
#include "hw/rtc/mc146818rtc.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "hw/display/ramfb.h"
#include "hw/i386/topology.h"
#include "hw/i386/e820_memory_layout.h"
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index c74931d577..b273d4a2e1 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -25,7 +25,7 @@
#include "qemu/osdep.h"
#include "qemu/units.h"
#include "hw/i386/pc.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "hw/char/parallel.h"
#include "hw/hyperv/hv-balloon.h"
#include "hw/i386/fw_cfg.h"
diff --git a/hw/isa/isa-superio.c b/hw/isa/isa-superio.c
index a8c8c58ef7..cff756e791 100644
--- a/hw/isa/isa-superio.c
+++ b/hw/isa/isa-superio.c
@@ -22,7 +22,7 @@
#include "hw/qdev-properties.h"
#include "hw/input/i8042.h"
#include "hw/char/parallel-isa.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "trace.h"
static void isa_superio_realize(DeviceState *dev, Error **errp)
diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c
index 505b44c4e6..1aa5c9613f 100644
--- a/hw/isa/vt82c686.c
+++ b/hw/isa/vt82c686.c
@@ -17,7 +17,7 @@
#include "hw/isa/vt82c686.h"
#include "hw/block/fdc.h"
#include "hw/char/parallel-isa.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "hw/pci/pci.h"
#include "hw/qdev-properties.h"
#include "hw/ide/pci.h"
diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
index 3526852685..ffc3f2bca0 100644
--- a/hw/ppc/pnv.c
+++ b/hw/ppc/pnv.c
@@ -53,7 +53,7 @@
#include "hw/ppc/pnv_pnor.h"
#include "hw/isa/isa.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "hw/rtc/mc146818rtc.h"
#include <libfdt.h>
diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
index 4ece1ac1ff..a839abbf5d 100644
--- a/hw/sparc64/sun4u.c
+++ b/hw/sparc64/sun4u.c
@@ -35,6 +35,7 @@
#include "hw/qdev-properties.h"
#include "hw/pci-host/sabre.h"
#include "hw/char/serial.h"
+#include "hw/char/serial-isa.h"
#include "hw/char/parallel-isa.h"
#include "hw/rtc/m48t59.h"
#include "migration/vmstate.h"
--
2.46.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 5/5] hw/char: Extract serial-mm
2024-08-14 18:15 [PATCH 0/5] Serial device cleanup Bernhard Beschow
` (3 preceding siblings ...)
2024-08-14 18:15 ` [PATCH 4/5] hw/char/serial.h: Extract serial-isa.h Bernhard Beschow
@ 2024-08-14 18:15 ` Bernhard Beschow
2024-08-15 0:35 ` Andrew Jeffery
2024-08-21 14:48 ` Mark Cave-Ayland
4 siblings, 2 replies; 16+ messages in thread
From: Bernhard Beschow @ 2024-08-14 18:15 UTC (permalink / raw)
To: qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Alistair Francis,
Artyom Tarasenko, Jia Liu, Helge Deller, Max Filippov,
Subbaraya Sundeep, Weiwei Li, Marc-André Lureau,
Nicholas Piggin, Mark Cave-Ayland, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao,
Bernhard Beschow
hw/char/serial currently contains the implementation of both TYPE_SERIAL and
TYPE_SERIAL_MM. According to serial_class_init(), TYPE_SERIAL is an internal
class while TYPE_SERIAL_MM is used by numerous machine types directly. Let's
move the latter into its own module which makes the dependencies more obvious
and the code more tidy.
The includes and the dependencies have been converted mechanically except in the
hw/char directories which were updated manually. The result was compile-tested.
Now, only hw/char makes direct use of TYPE_SERIAL:
# grep -r -e "select SERIAL" | grep -v SERIAL_
hw/char/Kconfig: select SERIAL
hw/char/Kconfig: select SERIAL
hw/char/Kconfig: select SERIAL
hw/char/Kconfig: select SERIAL
hw/char/Kconfig: select SERIAL
# grep -r -e "/serial\\.h"
include/hw/char/serial-mm.h:#include "hw/char/serial.h"
hw/char/serial-pci-multi.c:#include "hw/char/serial.h"
hw/char/serial.c:#include "hw/char/serial.h"
hw/char/serial-isa.c:#include "hw/char/serial.h"
hw/char/serial-pci.c:#include "hw/char/serial.h"
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
include/hw/arm/aspeed_soc.h | 2 +-
include/hw/char/mchp_pfsoc_mmuart.h | 2 +-
include/hw/char/serial-mm.h | 52 +++++++++
include/hw/char/serial.h | 19 ----
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/serial-mm.c | 157 ++++++++++++++++++++++++++++
hw/char/serial.c | 126 ----------------------
hw/display/sm501.c | 2 +-
hw/hppa/machine.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/ppc405_uc.c | 2 +-
hw/ppc/ppc440_bamboo.c | 2 +-
hw/ppc/sam460ex.c | 2 +-
hw/ppc/virtex_ml507.c | 2 +-
hw/riscv/virt.c | 2 +-
hw/sparc64/niagara.c | 2 +-
hw/sparc64/sun4u.c | 2 +-
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 +-
50 files changed, 276 insertions(+), 206 deletions(-)
create mode 100644 include/hw/char/serial-mm.h
create mode 100644 hw/char/serial-mm.c
diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
index 624d489e0d..689f52dae8 100644
--- a/include/hw/arm/aspeed_soc.h
+++ b/include/hw/arm/aspeed_soc.h
@@ -39,7 +39,7 @@
#include "hw/misc/unimp.h"
#include "hw/misc/aspeed_peci.h"
#include "hw/fsi/aspeed_apb2opb.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/intc/arm_gicv3.h"
#define ASPEED_SPIS_NUM 2
diff --git a/include/hw/char/mchp_pfsoc_mmuart.h b/include/hw/char/mchp_pfsoc_mmuart.h
index b0e14ca355..a7b8b1b08b 100644
--- a/include/hw/char/mchp_pfsoc_mmuart.h
+++ b/include/hw/char/mchp_pfsoc_mmuart.h
@@ -29,7 +29,7 @@
#define HW_MCHP_PFSOC_MMUART_H
#include "hw/sysbus.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#define MCHP_PFSOC_MMUART_REG_COUNT 13
diff --git a/include/hw/char/serial-mm.h b/include/hw/char/serial-mm.h
new file mode 100644
index 0000000000..62a8489d69
--- /dev/null
+++ b/include/hw/char/serial-mm.h
@@ -0,0 +1,52 @@
+/*
+ * QEMU 16550A UART emulation
+ *
+ * Copyright (c) 2003-2004 Fabrice Bellard
+ * Copyright (c) 2008 Citrix Systems, Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+
+#ifndef HW_SERIAL_MM_H
+#define HW_SERIAL_MM_H
+
+#include "hw/char/serial.h"
+#include "exec/memory.h"
+#include "chardev/char.h"
+#include "hw/sysbus.h"
+#include "qom/object.h"
+
+#define TYPE_SERIAL_MM "serial-mm"
+OBJECT_DECLARE_SIMPLE_TYPE(SerialMM, SERIAL_MM)
+
+struct SerialMM {
+ SysBusDevice parent;
+
+ SerialState serial;
+
+ uint8_t regshift;
+ uint8_t endianness;
+};
+
+SerialMM *serial_mm_init(MemoryRegion *address_space,
+ hwaddr base, int regshift,
+ qemu_irq irq, int baudbase,
+ Chardev *chr, enum device_endian end);
+
+#endif
diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h
index ccf322a031..942b372df6 100644
--- a/include/hw/char/serial.h
+++ b/include/hw/char/serial.h
@@ -29,8 +29,6 @@
#include "chardev/char-fe.h"
#include "exec/memory.h"
#include "qemu/fifo8.h"
-#include "chardev/char.h"
-#include "hw/sysbus.h"
#include "qom/object.h"
#define UART_FIFO_LENGTH 16 /* 16550A Fifo Length */
@@ -81,27 +79,10 @@ struct SerialState {
};
typedef struct SerialState SerialState;
-struct SerialMM {
- SysBusDevice parent;
-
- SerialState serial;
-
- uint8_t regshift;
- uint8_t endianness;
-};
-
extern const VMStateDescription vmstate_serial;
extern const MemoryRegionOps serial_io_ops;
#define TYPE_SERIAL "serial"
OBJECT_DECLARE_SIMPLE_TYPE(SerialState, SERIAL)
-#define TYPE_SERIAL_MM "serial-mm"
-OBJECT_DECLARE_SIMPLE_TYPE(SerialMM, SERIAL_MM)
-
-SerialMM *serial_mm_init(MemoryRegion *address_space,
- hwaddr base, int regshift,
- qemu_irq irq, int baudbase,
- Chardev *chr, enum device_endian end);
-
#endif
diff --git a/hw/arm/allwinner-a10.c b/hw/arm/allwinner-a10.c
index 57d5d80159..08cdff61e4 100644
--- a/hw/arm/allwinner-a10.c
+++ b/hw/arm/allwinner-a10.c
@@ -18,7 +18,7 @@
#include "qemu/osdep.h"
#include "qapi/error.h"
#include "qemu/module.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/sysbus.h"
#include "hw/arm/allwinner-a10.h"
#include "hw/misc/unimp.h"
diff --git a/hw/arm/allwinner-h3.c b/hw/arm/allwinner-h3.c
index 6870c3fe96..9bc57cd266 100644
--- a/hw/arm/allwinner-h3.c
+++ b/hw/arm/allwinner-h3.c
@@ -24,7 +24,7 @@
#include "qemu/units.h"
#include "hw/qdev-core.h"
#include "hw/sysbus.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/misc/unimp.h"
#include "hw/usb/hcd-ehci.h"
#include "hw/loader.h"
diff --git a/hw/arm/allwinner-r40.c b/hw/arm/allwinner-r40.c
index b8c7202133..ced73009d6 100644
--- a/hw/arm/allwinner-r40.c
+++ b/hw/arm/allwinner-r40.c
@@ -26,7 +26,7 @@
#include "hw/boards.h"
#include "hw/qdev-core.h"
#include "hw/sysbus.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/misc/unimp.h"
#include "hw/usb/hcd-ehci.h"
#include "hw/loader.h"
diff --git a/hw/arm/aspeed_ast2400.c b/hw/arm/aspeed_ast2400.c
index d125886207..ecc81ecc79 100644
--- a/hw/arm/aspeed_ast2400.c
+++ b/hw/arm/aspeed_ast2400.c
@@ -15,7 +15,7 @@
#include "qapi/error.h"
#include "hw/misc/unimp.h"
#include "hw/arm/aspeed_soc.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "qemu/module.h"
#include "qemu/error-report.h"
#include "hw/i2c/aspeed_i2c.h"
diff --git a/hw/arm/aspeed_soc_common.c b/hw/arm/aspeed_soc_common.c
index 05551461ae..a5ff33c46d 100644
--- a/hw/arm/aspeed_soc_common.c
+++ b/hw/arm/aspeed_soc_common.c
@@ -15,7 +15,7 @@
#include "hw/qdev-properties.h"
#include "hw/misc/unimp.h"
#include "hw/arm/aspeed_soc.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
const char *aspeed_soc_cpu_type(AspeedSoCClass *sc)
diff --git a/hw/arm/kzm.c b/hw/arm/kzm.c
index 2ccd6f8a76..fbd140e383 100644
--- a/hw/arm/kzm.c
+++ b/hw/arm/kzm.c
@@ -22,7 +22,7 @@
#include "exec/address-spaces.h"
#include "net/net.h"
#include "hw/net/lan9118.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "sysemu/qtest.h"
#include "sysemu/sysemu.h"
#include "qemu/cutils.h"
diff --git a/hw/arm/msf2-soc.c b/hw/arm/msf2-soc.c
index a94a10adcc..c4999ebce3 100644
--- a/hw/arm/msf2-soc.c
+++ b/hw/arm/msf2-soc.c
@@ -26,7 +26,7 @@
#include "qemu/units.h"
#include "qapi/error.h"
#include "exec/address-spaces.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/arm/msf2-soc.h"
#include "hw/misc/unimp.h"
#include "hw/qdev-clock.h"
diff --git a/hw/arm/musicpal.c b/hw/arm/musicpal.c
index 2020f73a57..940e4437d9 100644
--- a/hw/arm/musicpal.c
+++ b/hw/arm/musicpal.c
@@ -18,7 +18,7 @@
#include "net/net.h"
#include "sysemu/sysemu.h"
#include "hw/boards.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "qemu/timer.h"
#include "hw/ptimer.h"
#include "hw/qdev-properties.h"
diff --git a/hw/arm/npcm7xx.c b/hw/arm/npcm7xx.c
index cb7791301b..af04c4b7ec 100644
--- a/hw/arm/npcm7xx.c
+++ b/hw/arm/npcm7xx.c
@@ -18,7 +18,7 @@
#include "hw/arm/boot.h"
#include "hw/arm/npcm7xx.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/loader.h"
#include "hw/misc/unimp.h"
#include "hw/qdev-clock.h"
diff --git a/hw/arm/pxa2xx.c b/hw/arm/pxa2xx.c
index 6b2e54473b..c1448e670e 100644
--- a/hw/arm/pxa2xx.c
+++ b/hw/arm/pxa2xx.c
@@ -17,7 +17,7 @@
#include "migration/vmstate.h"
#include "hw/arm/pxa.h"
#include "sysemu/sysemu.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/i2c/i2c.h"
#include "hw/irq.h"
#include "hw/qdev-properties.h"
diff --git a/hw/char/omap_uart.c b/hw/char/omap_uart.c
index c2ef4c137e..6ec0429996 100644
--- a/hw/char/omap_uart.c
+++ b/hw/char/omap_uart.c
@@ -20,7 +20,7 @@
#include "qemu/osdep.h"
#include "chardev/char.h"
#include "hw/arm/omap.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "exec/address-spaces.h"
/* UARTs */
diff --git a/hw/char/serial-mm.c b/hw/char/serial-mm.c
new file mode 100644
index 0000000000..2f67776b19
--- /dev/null
+++ b/hw/char/serial-mm.c
@@ -0,0 +1,157 @@
+/*
+ * QEMU 16550A UART emulation
+ *
+ * Copyright (c) 2003-2004 Fabrice Bellard
+ * Copyright (c) 2008 Citrix Systems, Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+
+#include "qemu/osdep.h"
+#include "hw/char/serial-mm.h"
+#include "exec/cpu-common.h"
+#include "migration/vmstate.h"
+#include "qapi/error.h"
+#include "hw/qdev-properties.h"
+
+static uint64_t serial_mm_read(void *opaque, hwaddr addr, unsigned size)
+{
+ SerialMM *s = SERIAL_MM(opaque);
+ return serial_io_ops.read(&s->serial, addr >> s->regshift, 1);
+}
+
+static void serial_mm_write(void *opaque, hwaddr addr,
+ uint64_t value, unsigned size)
+{
+ SerialMM *s = SERIAL_MM(opaque);
+ value &= 255;
+ serial_io_ops.write(&s->serial, addr >> s->regshift, value, 1);
+}
+
+static const MemoryRegionOps serial_mm_ops[3] = {
+ [DEVICE_NATIVE_ENDIAN] = {
+ .read = serial_mm_read,
+ .write = serial_mm_write,
+ .endianness = DEVICE_NATIVE_ENDIAN,
+ .valid.max_access_size = 8,
+ .impl.max_access_size = 8,
+ },
+ [DEVICE_LITTLE_ENDIAN] = {
+ .read = serial_mm_read,
+ .write = serial_mm_write,
+ .endianness = DEVICE_LITTLE_ENDIAN,
+ .valid.max_access_size = 8,
+ .impl.max_access_size = 8,
+ },
+ [DEVICE_BIG_ENDIAN] = {
+ .read = serial_mm_read,
+ .write = serial_mm_write,
+ .endianness = DEVICE_BIG_ENDIAN,
+ .valid.max_access_size = 8,
+ .impl.max_access_size = 8,
+ },
+};
+
+static void serial_mm_realize(DeviceState *dev, Error **errp)
+{
+ SerialMM *smm = SERIAL_MM(dev);
+ SerialState *s = &smm->serial;
+
+ if (!qdev_realize(DEVICE(s), NULL, errp)) {
+ return;
+ }
+
+ memory_region_init_io(&s->io, OBJECT(dev),
+ &serial_mm_ops[smm->endianness], smm, "serial",
+ 8 << smm->regshift);
+ sysbus_init_mmio(SYS_BUS_DEVICE(smm), &s->io);
+ sysbus_init_irq(SYS_BUS_DEVICE(smm), &smm->serial.irq);
+}
+
+static const VMStateDescription vmstate_serial_mm = {
+ .name = "serial",
+ .version_id = 3,
+ .minimum_version_id = 2,
+ .fields = (const VMStateField[]) {
+ VMSTATE_STRUCT(serial, SerialMM, 0, vmstate_serial, SerialState),
+ VMSTATE_END_OF_LIST()
+ }
+};
+
+SerialMM *serial_mm_init(MemoryRegion *address_space,
+ hwaddr base, int regshift,
+ qemu_irq irq, int baudbase,
+ Chardev *chr, enum device_endian end)
+{
+ SerialMM *smm = SERIAL_MM(qdev_new(TYPE_SERIAL_MM));
+ MemoryRegion *mr;
+
+ qdev_prop_set_uint8(DEVICE(smm), "regshift", regshift);
+ qdev_prop_set_uint32(DEVICE(smm), "baudbase", baudbase);
+ qdev_prop_set_chr(DEVICE(smm), "chardev", chr);
+ qdev_set_legacy_instance_id(DEVICE(smm), base, 2);
+ qdev_prop_set_uint8(DEVICE(smm), "endianness", end);
+ sysbus_realize_and_unref(SYS_BUS_DEVICE(smm), &error_fatal);
+
+ sysbus_connect_irq(SYS_BUS_DEVICE(smm), 0, irq);
+ mr = sysbus_mmio_get_region(SYS_BUS_DEVICE(smm), 0);
+ memory_region_add_subregion(address_space, base, mr);
+
+ return smm;
+}
+
+static void serial_mm_instance_init(Object *o)
+{
+ SerialMM *smm = SERIAL_MM(o);
+
+ object_initialize_child(o, "serial", &smm->serial, TYPE_SERIAL);
+
+ qdev_alias_all_properties(DEVICE(&smm->serial), o);
+}
+
+static Property serial_mm_properties[] = {
+ /*
+ * Set the spacing between adjacent memory-mapped UART registers.
+ * Each register will be at (1 << regshift) bytes after the previous one.
+ */
+ DEFINE_PROP_UINT8("regshift", SerialMM, regshift, 0),
+ DEFINE_PROP_UINT8("endianness", SerialMM, endianness, DEVICE_NATIVE_ENDIAN),
+ DEFINE_PROP_END_OF_LIST(),
+};
+
+static void serial_mm_class_init(ObjectClass *oc, void *data)
+{
+ DeviceClass *dc = DEVICE_CLASS(oc);
+
+ device_class_set_props(dc, serial_mm_properties);
+ dc->realize = serial_mm_realize;
+ dc->vmsd = &vmstate_serial_mm;
+}
+
+static const TypeInfo types[] = {
+ {
+ .name = TYPE_SERIAL_MM,
+ .parent = TYPE_SYS_BUS_DEVICE,
+ .class_init = serial_mm_class_init,
+ .instance_init = serial_mm_instance_init,
+ .instance_size = sizeof(SerialMM),
+ },
+};
+
+DEFINE_TYPES(types)
diff --git a/hw/char/serial.c b/hw/char/serial.c
index 6c5c4a23c7..b50a8a1313 100644
--- a/hw/char/serial.c
+++ b/hw/char/serial.c
@@ -989,135 +989,9 @@ static const TypeInfo serial_info = {
.class_init = serial_class_init,
};
-/* Memory mapped interface */
-static uint64_t serial_mm_read(void *opaque, hwaddr addr,
- unsigned size)
-{
- SerialMM *s = SERIAL_MM(opaque);
- return serial_ioport_read(&s->serial, addr >> s->regshift, 1);
-}
-
-static void serial_mm_write(void *opaque, hwaddr addr,
- uint64_t value, unsigned size)
-{
- SerialMM *s = SERIAL_MM(opaque);
- value &= 255;
- serial_ioport_write(&s->serial, addr >> s->regshift, value, 1);
-}
-
-static const MemoryRegionOps serial_mm_ops[3] = {
- [DEVICE_NATIVE_ENDIAN] = {
- .read = serial_mm_read,
- .write = serial_mm_write,
- .endianness = DEVICE_NATIVE_ENDIAN,
- .valid.max_access_size = 8,
- .impl.max_access_size = 8,
- },
- [DEVICE_LITTLE_ENDIAN] = {
- .read = serial_mm_read,
- .write = serial_mm_write,
- .endianness = DEVICE_LITTLE_ENDIAN,
- .valid.max_access_size = 8,
- .impl.max_access_size = 8,
- },
- [DEVICE_BIG_ENDIAN] = {
- .read = serial_mm_read,
- .write = serial_mm_write,
- .endianness = DEVICE_BIG_ENDIAN,
- .valid.max_access_size = 8,
- .impl.max_access_size = 8,
- },
-};
-
-static void serial_mm_realize(DeviceState *dev, Error **errp)
-{
- SerialMM *smm = SERIAL_MM(dev);
- SerialState *s = &smm->serial;
-
- if (!qdev_realize(DEVICE(s), NULL, errp)) {
- return;
- }
-
- memory_region_init_io(&s->io, OBJECT(dev),
- &serial_mm_ops[smm->endianness], smm, "serial",
- 8 << smm->regshift);
- sysbus_init_mmio(SYS_BUS_DEVICE(smm), &s->io);
- sysbus_init_irq(SYS_BUS_DEVICE(smm), &smm->serial.irq);
-}
-
-static const VMStateDescription vmstate_serial_mm = {
- .name = "serial",
- .version_id = 3,
- .minimum_version_id = 2,
- .fields = (const VMStateField[]) {
- VMSTATE_STRUCT(serial, SerialMM, 0, vmstate_serial, SerialState),
- VMSTATE_END_OF_LIST()
- }
-};
-
-SerialMM *serial_mm_init(MemoryRegion *address_space,
- hwaddr base, int regshift,
- qemu_irq irq, int baudbase,
- Chardev *chr, enum device_endian end)
-{
- SerialMM *smm = SERIAL_MM(qdev_new(TYPE_SERIAL_MM));
- MemoryRegion *mr;
-
- qdev_prop_set_uint8(DEVICE(smm), "regshift", regshift);
- qdev_prop_set_uint32(DEVICE(smm), "baudbase", baudbase);
- qdev_prop_set_chr(DEVICE(smm), "chardev", chr);
- qdev_set_legacy_instance_id(DEVICE(smm), base, 2);
- qdev_prop_set_uint8(DEVICE(smm), "endianness", end);
- sysbus_realize_and_unref(SYS_BUS_DEVICE(smm), &error_fatal);
-
- sysbus_connect_irq(SYS_BUS_DEVICE(smm), 0, irq);
- mr = sysbus_mmio_get_region(SYS_BUS_DEVICE(smm), 0);
- memory_region_add_subregion(address_space, base, mr);
-
- return smm;
-}
-
-static void serial_mm_instance_init(Object *o)
-{
- SerialMM *smm = SERIAL_MM(o);
-
- object_initialize_child(o, "serial", &smm->serial, TYPE_SERIAL);
-
- qdev_alias_all_properties(DEVICE(&smm->serial), o);
-}
-
-static Property serial_mm_properties[] = {
- /*
- * Set the spacing between adjacent memory-mapped UART registers.
- * Each register will be at (1 << regshift) bytes after the
- * previous one.
- */
- DEFINE_PROP_UINT8("regshift", SerialMM, regshift, 0),
- DEFINE_PROP_UINT8("endianness", SerialMM, endianness, DEVICE_NATIVE_ENDIAN),
- DEFINE_PROP_END_OF_LIST(),
-};
-
-static void serial_mm_class_init(ObjectClass *oc, void *data)
-{
- DeviceClass *dc = DEVICE_CLASS(oc);
-
- device_class_set_props(dc, serial_mm_properties);
- dc->realize = serial_mm_realize;
- dc->vmsd = &vmstate_serial_mm;
-}
-
-static const TypeInfo serial_mm_info = {
- .name = TYPE_SERIAL_MM,
- .parent = TYPE_SYS_BUS_DEVICE,
- .class_init = serial_mm_class_init,
- .instance_init = serial_mm_instance_init,
- .instance_size = sizeof(SerialMM),
-};
-
static void serial_register_types(void)
{
type_register_static(&serial_info);
- type_register_static(&serial_mm_info);
}
type_init(serial_register_types)
diff --git a/hw/display/sm501.c b/hw/display/sm501.c
index 26dc8170d8..ad47fb6010 100644
--- a/hw/display/sm501.c
+++ b/hw/display/sm501.c
@@ -29,7 +29,7 @@
#include "qemu/log.h"
#include "qemu/module.h"
#include "hw/usb/hcd-ohci.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "ui/console.h"
#include "hw/sysbus.h"
#include "migration/vmstate.h"
diff --git a/hw/hppa/machine.c b/hw/hppa/machine.c
index 5d0a8739de..0ab54b0676 100644
--- a/hw/hppa/machine.c
+++ b/hw/hppa/machine.c
@@ -17,7 +17,7 @@
#include "sysemu/runstate.h"
#include "hw/rtc/mc146818rtc.h"
#include "hw/timer/i8254.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/char/parallel.h"
#include "hw/intc/i8259.h"
#include "hw/input/lasips2.h"
diff --git a/hw/loongarch/virt.c b/hw/loongarch/virt.c
index 29040422aa..e3a6a252be 100644
--- a/hw/loongarch/virt.c
+++ b/hw/loongarch/virt.c
@@ -9,7 +9,7 @@
#include "qemu/datadir.h"
#include "qapi/error.h"
#include "hw/boards.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "sysemu/kvm.h"
#include "sysemu/tcg.h"
#include "sysemu/sysemu.h"
diff --git a/hw/microblaze/petalogix_ml605_mmu.c b/hw/microblaze/petalogix_ml605_mmu.c
index 0f5fabc32e..b4183c5267 100644
--- a/hw/microblaze/petalogix_ml605_mmu.c
+++ b/hw/microblaze/petalogix_ml605_mmu.c
@@ -34,7 +34,7 @@
#include "hw/block/flash.h"
#include "sysemu/sysemu.h"
#include "hw/boards.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/qdev-properties.h"
#include "exec/address-spaces.h"
#include "hw/ssi/ssi.h"
diff --git a/hw/mips/boston.c b/hw/mips/boston.c
index 1b44fb354c..1ced1e337a 100644
--- a/hw/mips/boston.c
+++ b/hw/mips/boston.c
@@ -22,7 +22,7 @@
#include "elf.h"
#include "hw/boards.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/ide/pci.h"
#include "hw/ide/ahci-pci.h"
#include "hw/loader.h"
diff --git a/hw/mips/jazz.c b/hw/mips/jazz.c
index 1bc17e69d3..a3b39c8663 100644
--- a/hw/mips/jazz.c
+++ b/hw/mips/jazz.c
@@ -28,7 +28,7 @@
#include "hw/mips/mips.h"
#include "hw/intc/i8259.h"
#include "hw/dma/i8257.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/char/parallel.h"
#include "hw/isa/isa.h"
#include "hw/block/fdc.h"
diff --git a/hw/mips/loongson3_virt.c b/hw/mips/loongson3_virt.c
index 408e3d7054..4a27d7b9da 100644
--- a/hw/mips/loongson3_virt.c
+++ b/hw/mips/loongson3_virt.c
@@ -29,7 +29,7 @@
#include "qemu/datadir.h"
#include "qapi/error.h"
#include "elf.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/intc/loongson_liointc.h"
#include "hw/mips/mips.h"
#include "hw/mips/fw_cfg.h"
diff --git a/hw/mips/malta.c b/hw/mips/malta.c
index 664a2ae0a9..1df00c4bf9 100644
--- a/hw/mips/malta.c
+++ b/hw/mips/malta.c
@@ -31,7 +31,7 @@
#include "hw/clock.h"
#include "hw/southbridge/piix.h"
#include "hw/isa/superio.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "net/net.h"
#include "hw/boards.h"
#include "hw/i2c/smbus_eeprom.h"
diff --git a/hw/mips/mipssim.c b/hw/mips/mipssim.c
index 9170d6c474..a07732d3dc 100644
--- a/hw/mips/mipssim.c
+++ b/hw/mips/mipssim.c
@@ -31,7 +31,7 @@
#include "exec/address-spaces.h"
#include "hw/clock.h"
#include "hw/mips/mips.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "net/net.h"
#include "sysemu/sysemu.h"
#include "hw/boards.h"
diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
index bffd6f721f..9fb63515ef 100644
--- a/hw/openrisc/openrisc_sim.c
+++ b/hw/openrisc/openrisc_sim.c
@@ -24,7 +24,7 @@
#include "cpu.h"
#include "hw/irq.h"
#include "hw/boards.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "net/net.h"
#include "hw/openrisc/boot.h"
#include "hw/qdev-properties.h"
diff --git a/hw/openrisc/virt.c b/hw/openrisc/virt.c
index f8a68a6a6b..47d2c9bd3c 100644
--- a/hw/openrisc/virt.c
+++ b/hw/openrisc/virt.c
@@ -14,7 +14,7 @@
#include "exec/address-spaces.h"
#include "hw/irq.h"
#include "hw/boards.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/core/split-irq.h"
#include "hw/openrisc/boot.h"
#include "hw/misc/sifive_test.h"
diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c
index 3bd12b54ab..b760c6d6a2 100644
--- a/hw/ppc/e500.c
+++ b/hw/ppc/e500.c
@@ -24,7 +24,7 @@
#include "net/net.h"
#include "qemu/config-file.h"
#include "hw/block/flash.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/pci/pci.h"
#include "sysemu/block-backend-io.h"
#include "sysemu/sysemu.h"
diff --git a/hw/ppc/ppc405_uc.c b/hw/ppc/ppc405_uc.c
index 0cc68178ad..36b18d6694 100644
--- a/hw/ppc/ppc405_uc.c
+++ b/hw/ppc/ppc405_uc.c
@@ -32,7 +32,7 @@
#include "hw/irq.h"
#include "hw/qdev-properties.h"
#include "ppc405.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "qemu/timer.h"
#include "sysemu/reset.h"
#include "sysemu/sysemu.h"
diff --git a/hw/ppc/ppc440_bamboo.c b/hw/ppc/ppc440_bamboo.c
index 73f80cf706..96d9ce65c2 100644
--- a/hw/ppc/ppc440_bamboo.c
+++ b/hw/ppc/ppc440_bamboo.c
@@ -23,7 +23,7 @@
#include "sysemu/device_tree.h"
#include "hw/loader.h"
#include "elf.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/ppc/ppc.h"
#include "hw/pci-host/ppc4xx.h"
#include "sysemu/sysemu.h"
diff --git a/hw/ppc/sam460ex.c b/hw/ppc/sam460ex.c
index 8dc75fb9f0..1fce093ac8 100644
--- a/hw/ppc/sam460ex.c
+++ b/hw/ppc/sam460ex.c
@@ -31,7 +31,7 @@
#include "sysemu/sysemu.h"
#include "sysemu/reset.h"
#include "hw/sysbus.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/i2c/ppc4xx_i2c.h"
#include "hw/i2c/smbus_eeprom.h"
#include "hw/ide/pci.h"
diff --git a/hw/ppc/virtex_ml507.c b/hw/ppc/virtex_ml507.c
index c49da1f46f..235281e939 100644
--- a/hw/ppc/virtex_ml507.c
+++ b/hw/ppc/virtex_ml507.c
@@ -28,7 +28,7 @@
#include "exec/page-protection.h"
#include "cpu.h"
#include "hw/sysbus.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/block/flash.h"
#include "sysemu/sysemu.h"
#include "sysemu/reset.h"
diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
index 9981e0f6c9..b1a0811071 100644
--- a/hw/riscv/virt.c
+++ b/hw/riscv/virt.c
@@ -27,7 +27,7 @@
#include "hw/loader.h"
#include "hw/sysbus.h"
#include "hw/qdev-properties.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "target/riscv/cpu.h"
#include "hw/core/sysbus-fdt.h"
#include "target/riscv/pmu.h"
diff --git a/hw/sparc64/niagara.c b/hw/sparc64/niagara.c
index ab3c4ec346..67ec403e1d 100644
--- a/hw/sparc64/niagara.c
+++ b/hw/sparc64/niagara.c
@@ -27,7 +27,7 @@
#include "qemu/units.h"
#include "cpu.h"
#include "hw/boards.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "hw/misc/unimp.h"
#include "hw/loader.h"
#include "hw/sparc/sparc64.h"
diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
index a839abbf5d..541c7f74fa 100644
--- a/hw/sparc64/sun4u.c
+++ b/hw/sparc64/sun4u.c
@@ -34,8 +34,8 @@
#include "hw/pci/pci_host.h"
#include "hw/qdev-properties.h"
#include "hw/pci-host/sabre.h"
-#include "hw/char/serial.h"
#include "hw/char/serial-isa.h"
+#include "hw/char/serial-mm.h"
#include "hw/char/parallel-isa.h"
#include "hw/rtc/m48t59.h"
#include "migration/vmstate.h"
diff --git a/hw/xtensa/xtfpga.c b/hw/xtensa/xtfpga.c
index 955e8867a3..45b29d3b4e 100644
--- a/hw/xtensa/xtfpga.c
+++ b/hw/xtensa/xtfpga.c
@@ -35,7 +35,7 @@
#include "hw/qdev-properties.h"
#include "elf.h"
#include "exec/memory.h"
-#include "hw/char/serial.h"
+#include "hw/char/serial-mm.h"
#include "net/net.h"
#include "hw/sysbus.h"
#include "hw/block/flash.h"
diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
index 1ad60da7aa..ecde52f4a4 100644
--- a/hw/arm/Kconfig
+++ b/hw/arm/Kconfig
@@ -136,7 +136,7 @@ config MUSICPAL
select MARVELL_88W8618
select PTIMER
select PFLASH_CFI02
- select SERIAL
+ select SERIAL_MM
select WM8750
config NETDUINO2
@@ -179,13 +179,13 @@ config OMAP
select NAND
select PFLASH_CFI01
select SD
- select SERIAL
+ select SERIAL_MM
config PXA2XX
bool
select FRAMEBUFFER
select I2C
- select SERIAL
+ select SERIAL_MM
select SD
select SSI
select USB_OHCI_SYSBUS
@@ -399,7 +399,7 @@ config ALLWINNER_A10
select ALLWINNER_EMAC
select ALLWINNER_I2C
select AXP2XX_PMU
- select SERIAL
+ select SERIAL_MM
select UNIMP
select USB_OHCI_SYSBUS
@@ -411,7 +411,7 @@ config ALLWINNER_H3
select ALLWINNER_SUN8I_EMAC
select ALLWINNER_I2C
select ALLWINNER_WDT
- select SERIAL
+ select SERIAL_MM
select ARM_TIMER
select ARM_GIC
select UNIMP
@@ -427,7 +427,7 @@ config ALLWINNER_R40
select ALLWINNER_A10_PIT
select ALLWINNER_WDT
select AXP2XX_PMU
- select SERIAL
+ select SERIAL_MM
select ARM_TIMER
select ARM_GIC
select UNIMP
@@ -542,7 +542,7 @@ config NPCM7XX
select ISL_PMBUS_VR
select PL310 # cache controller
select PMBUS
- select SERIAL
+ select SERIAL_MM
select SSI
select UNIMP
select PCA954X
@@ -564,7 +564,7 @@ config FSL_IMX31
default y
depends on TCG && ARM
imply I2C_DEVICES
- select SERIAL
+ select SERIAL_MM
select IMX
select IMX_I2C
select WDT_IMX2
@@ -593,7 +593,7 @@ config ASPEED_SOC
select I2C
select DPS310
select PCA9552
- select SERIAL
+ select SERIAL_MM
select SMBUS_EEPROM
select PCA954X
select SSI
@@ -681,7 +681,7 @@ config MSF2
bool
select ARM_V7M
select PTIMER
- select SERIAL
+ select SERIAL_MM
select SSI
select UNIMP
diff --git a/hw/char/Kconfig b/hw/char/Kconfig
index 4fd74ea878..4b73a803bf 100644
--- a/hw/char/Kconfig
+++ b/hw/char/Kconfig
@@ -21,6 +21,10 @@ config SERIAL_ISA
depends on ISA_BUS
select SERIAL
+config SERIAL_MM
+ bool
+ select SERIAL
+
config SERIAL_PCI
bool
default y if PCI_DEVICES
diff --git a/hw/char/meson.build b/hw/char/meson.build
index e5b13b6958..b99938f756 100644
--- a/hw/char/meson.build
+++ b/hw/char/meson.build
@@ -14,6 +14,7 @@ system_ss.add(when: 'CONFIG_PL011', if_true: files('pl011.c'))
system_ss.add(when: 'CONFIG_SCLPCONSOLE', if_true: files('sclpconsole.c', 'sclpconsole-lm.c'))
system_ss.add(when: 'CONFIG_SERIAL', if_true: files('serial.c'))
system_ss.add(when: 'CONFIG_SERIAL_ISA', if_true: files('serial-isa.c'))
+system_ss.add(when: 'CONFIG_SERIAL_MM', if_true: files('serial-mm.c'))
system_ss.add(when: 'CONFIG_SERIAL_PCI', if_true: files('serial-pci.c'))
system_ss.add(when: 'CONFIG_SERIAL_PCI_MULTI', if_true: files('serial-pci-multi.c'))
system_ss.add(when: 'CONFIG_SHAKTI_UART', if_true: files('shakti_uart.c'))
diff --git a/hw/display/Kconfig b/hw/display/Kconfig
index a4552c8ed7..5fde4855e3 100644
--- a/hw/display/Kconfig
+++ b/hw/display/Kconfig
@@ -76,7 +76,7 @@ config SM501
bool
select I2C
select DDC
- select SERIAL
+ select SERIAL_MM
select USB_OHCI_SYSBUS
config TCX
diff --git a/hw/hppa/Kconfig b/hw/hppa/Kconfig
index d4d457f4ab..9312c4294a 100644
--- a/hw/hppa/Kconfig
+++ b/hw/hppa/Kconfig
@@ -9,7 +9,7 @@ config HPPA_B160L
select ASTRO
select DINO
select LASI
- select SERIAL
+ select SERIAL_MM
select SERIAL_PCI
select ISA_BUS
select I8259
diff --git a/hw/loongarch/Kconfig b/hw/loongarch/Kconfig
index 0de713a439..6b67b34b7a 100644
--- a/hw/loongarch/Kconfig
+++ b/hw/loongarch/Kconfig
@@ -9,7 +9,7 @@ config LOONGARCH_VIRT
imply PCI_DEVICES
imply NVDIMM
imply TPM_TIS_SYSBUS
- select SERIAL
+ select SERIAL_MM
select VIRTIO_PCI
select PLATFORM_BUS
select LOONGARCH_IPI
diff --git a/hw/microblaze/Kconfig b/hw/microblaze/Kconfig
index d78ba843fa..b0214b2c8b 100644
--- a/hw/microblaze/Kconfig
+++ b/hw/microblaze/Kconfig
@@ -13,7 +13,7 @@ config PETALOGIX_ML605
default y
depends on MICROBLAZE
select PFLASH_CFI01
- select SERIAL
+ select SERIAL_MM
select SSI_M25P80
select XILINX
select XILINX_AXI
diff --git a/hw/mips/Kconfig b/hw/mips/Kconfig
index 692bede538..b09c89a017 100644
--- a/hw/mips/Kconfig
+++ b/hw/mips/Kconfig
@@ -10,14 +10,14 @@ config MALTA
select MIPS_CPS
select PIIX
select PFLASH_CFI01
- select SERIAL
+ select SERIAL_MM
select SMBUS_EEPROM
config MIPSSIM
bool
default y
depends on MIPS
- select SERIAL
+ select SERIAL_MM
select MIPSNET
config JAZZ
@@ -37,7 +37,7 @@ config JAZZ
select FDC_SYSBUS
select MC146818RTC
select PCKBD
- select SERIAL
+ select SERIAL_MM
select PARALLEL
select DS1225Y
select JAZZ_LED
@@ -65,7 +65,7 @@ config LOONGSON3V
imply VIRTIO_VGA
imply QXL if SPICE
imply USB_OHCI_PCI
- select SERIAL
+ select SERIAL_MM
select GOLDFISH_RTC
select LOONGSON_IPI
select LOONGSON_LIOINTC
@@ -89,7 +89,7 @@ config MIPS_BOSTON
select MIPS_CPS
select PCI_EXPRESS_XILINX
select AHCI_ICH9
- select SERIAL
+ select SERIAL_MM
config FW_CFG_MIPS
bool
diff --git a/hw/openrisc/Kconfig b/hw/openrisc/Kconfig
index 76b953c62c..0702f622a5 100644
--- a/hw/openrisc/Kconfig
+++ b/hw/openrisc/Kconfig
@@ -3,7 +3,7 @@ config OR1K_SIM
default y
depends on OPENRISC
select DEVICE_TREE
- select SERIAL
+ select SERIAL_MM
select OPENCORES_ETH
select OMPIC
select SPLIT_IRQ
@@ -19,6 +19,6 @@ config OR1K_VIRT
select PCI
select PCI_EXPRESS_GENERIC_BRIDGE
select GOLDFISH_RTC
- select SERIAL
+ select SERIAL_MM
select SIFIVE_TEST
select VIRTIO_MMIO
diff --git a/hw/ppc/Kconfig b/hw/ppc/Kconfig
index 5addad1124..b44d91bebb 100644
--- a/hw/ppc/Kconfig
+++ b/hw/ppc/Kconfig
@@ -51,7 +51,7 @@ config PPC405
select M48T59
select PFLASH_CFI02
select PPC4XX
- select SERIAL
+ select SERIAL_MM
config PPC440
bool
@@ -63,7 +63,7 @@ config PPC440
select PCI_EXPRESS
select PPC440_PCIX
select PPC4XX
- select SERIAL
+ select SERIAL_MM
select FDT_PPC
config PPC4XX
@@ -80,7 +80,7 @@ config SAM460EX
select IDE_SII3112
select M41T80
select PPC440
- select SERIAL
+ select SERIAL_MM
select SM501
select SMBUS_EEPROM
select USB_EHCI_SYSBUS
@@ -163,7 +163,7 @@ config E500
select PLATFORM_BUS
select PPCE500_PCI
select SDHCI
- select SERIAL
+ select SERIAL_MM
select MPC_I2C
select FDT_PPC
select DS1338
@@ -187,7 +187,7 @@ config VIRTEX
depends on PPC && FDT
select PPC4XX
select PFLASH_CFI01
- select SERIAL
+ select SERIAL_MM
select XILINX
select XILINX_ETHLITE
select FDT_PPC
diff --git a/hw/riscv/Kconfig b/hw/riscv/Kconfig
index a2030e3a6f..44695ff9f2 100644
--- a/hw/riscv/Kconfig
+++ b/hw/riscv/Kconfig
@@ -44,7 +44,7 @@ config RISCV_VIRT
select PCI
select PCI_EXPRESS_GENERIC_BRIDGE
select PFLASH_CFI01
- select SERIAL
+ select SERIAL_MM
select RISCV_ACLINT
select RISCV_APLIC
select RISCV_IMSIC
diff --git a/hw/sparc64/Kconfig b/hw/sparc64/Kconfig
index 3b948a2290..f764c8a219 100644
--- a/hw/sparc64/Kconfig
+++ b/hw/sparc64/Kconfig
@@ -10,6 +10,7 @@ config SUN4U
select ISA_BUS
select FDC_ISA
select SERIAL_ISA
+ select SERIAL_MM
select PCI_SABRE
select IDE_CMD646
select PCKBD
diff --git a/hw/xtensa/Kconfig b/hw/xtensa/Kconfig
index fc5c785cfa..1f0492d89c 100644
--- a/hw/xtensa/Kconfig
+++ b/hw/xtensa/Kconfig
@@ -18,4 +18,4 @@ config XTENSA_XTFPGA
select DEVICE_TREE
select OPENCORES_ETH
select PFLASH_CFI01
- select SERIAL
+ select SERIAL_MM
--
2.46.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 2/5] hw/char/serial: Remove unused funtion
2024-08-14 18:15 ` [PATCH 2/5] hw/char/serial: Remove unused funtion Bernhard Beschow
@ 2024-08-14 21:38 ` BALATON Zoltan
2024-08-15 8:24 ` Mark Cave-Ayland
0 siblings, 1 reply; 16+ messages in thread
From: BALATON Zoltan @ 2024-08-14 21:38 UTC (permalink / raw)
To: Bernhard Beschow
Cc: qemu-devel, Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, Jan Kiszka,
Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Alistair Francis,
Artyom Tarasenko, Jia Liu, Helge Deller, Max Filippov,
Subbaraya Sundeep, Weiwei Li, Marc-André Lureau,
Nicholas Piggin, Mark Cave-Ayland, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On Wed, 14 Aug 2024, Bernhard Beschow wrote:
> The serial port's frequency is set via the "baudbase" property nowadays.
Please keep it as some devices might have registers that set this freq and
this function will be needed for emulating that even if it's not emulated
currently.
Regards,
BALATON Zoltan
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
> include/hw/char/serial.h | 2 --
> hw/char/serial.c | 7 -------
> 2 files changed, 9 deletions(-)
>
> diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h
> index 6e14099ee7..40aad21df3 100644
> --- a/include/hw/char/serial.h
> +++ b/include/hw/char/serial.h
> @@ -93,8 +93,6 @@ struct SerialMM {
> extern const VMStateDescription vmstate_serial;
> extern const MemoryRegionOps serial_io_ops;
>
> -void serial_set_frequency(SerialState *s, uint32_t frequency);
> -
> #define TYPE_SERIAL "serial"
> OBJECT_DECLARE_SIMPLE_TYPE(SerialState, SERIAL)
>
> diff --git a/hw/char/serial.c b/hw/char/serial.c
> index d8b2db5082..6c5c4a23c7 100644
> --- a/hw/char/serial.c
> +++ b/hw/char/serial.c
> @@ -951,13 +951,6 @@ static void serial_unrealize(DeviceState *dev)
> qemu_unregister_reset(serial_reset, s);
> }
>
> -/* Change the main reference oscillator frequency. */
> -void serial_set_frequency(SerialState *s, uint32_t frequency)
> -{
> - s->baudbase = frequency;
> - serial_update_parameters(s);
> -}
> -
> const MemoryRegionOps serial_io_ops = {
> .read = serial_ioport_read,
> .write = serial_ioport_write,
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 5/5] hw/char: Extract serial-mm
2024-08-14 18:15 ` [PATCH 5/5] hw/char: Extract serial-mm Bernhard Beschow
@ 2024-08-15 0:35 ` Andrew Jeffery
2024-08-21 14:48 ` Mark Cave-Ayland
1 sibling, 0 replies; 16+ messages in thread
From: Andrew Jeffery @ 2024-08-15 0:35 UTC (permalink / raw)
To: Bernhard Beschow, qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Edgar E. Iglesias, Paolo Bonzini,
Hao Wu, Alistair Francis, BALATON Zoltan, Jan Kiszka,
Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Mark Cave-Ayland,
Daniel Henrique Barboza, Steven Lee, Philippe Mathieu-Daudé,
Huacai Chen, Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On Wed, 2024-08-14 at 20:15 +0200, Bernhard Beschow wrote:
> hw/char/serial currently contains the implementation of both TYPE_SERIAL and
> TYPE_SERIAL_MM. According to serial_class_init(), TYPE_SERIAL is an internal
> class while TYPE_SERIAL_MM is used by numerous machine types directly. Let's
> move the latter into its own module which makes the dependencies more obvious
> and the code more tidy.
>
> The includes and the dependencies have been converted mechanically except in the
> hw/char directories which were updated manually. The result was compile-tested.
> Now, only hw/char makes direct use of TYPE_SERIAL:
>
> # grep -r -e "select SERIAL" | grep -v SERIAL_
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
>
> # grep -r -e "/serial\\.h"
> include/hw/char/serial-mm.h:#include "hw/char/serial.h"
> hw/char/serial-pci-multi.c:#include "hw/char/serial.h"
> hw/char/serial.c:#include "hw/char/serial.h"
> hw/char/serial-isa.c:#include "hw/char/serial.h"
> hw/char/serial-pci.c:#include "hw/char/serial.h"
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
> include/hw/arm/aspeed_soc.h | 2 +-
...
> hw/arm/aspeed_ast2400.c | 2 +-
> hw/arm/aspeed_soc_common.c | 2 +-
For the Aspeed bits:
Tested-by: Andrew Jeffery <andrew@codeconstruct.com.au>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/5] hw/char/serial: Remove unused funtion
2024-08-14 21:38 ` BALATON Zoltan
@ 2024-08-15 8:24 ` Mark Cave-Ayland
2024-08-15 9:26 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 16+ messages in thread
From: Mark Cave-Ayland @ 2024-08-15 8:24 UTC (permalink / raw)
To: BALATON Zoltan, Bernhard Beschow
Cc: qemu-devel, Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, Jan Kiszka,
Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On 14/08/2024 22:38, BALATON Zoltan wrote:
> On Wed, 14 Aug 2024, Bernhard Beschow wrote:
>> The serial port's frequency is set via the "baudbase" property nowadays.
>
> Please keep it as some devices might have registers that set this freq and this
> function will be needed for emulating that even if it's not emulated currently.
Why not remove the function if it isn't being used? In general having unused code
lying around is bad as it tends to bitrot.
If someone did want to re-add this functionality then you would likely want to do it
with clock API rather than using a separate function.
ATB,
Mark.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/5] hw/char/serial: Remove unused funtion
2024-08-15 8:24 ` Mark Cave-Ayland
@ 2024-08-15 9:26 ` Philippe Mathieu-Daudé
2024-08-15 18:00 ` Bernhard Beschow
0 siblings, 1 reply; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-08-15 9:26 UTC (permalink / raw)
To: Mark Cave-Ayland, BALATON Zoltan, Bernhard Beschow
Cc: qemu-devel, Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, Jan Kiszka,
Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Daniel Henrique Barboza,
Steven Lee, Huacai Chen, Richard Henderson, Strahinja Jankovic,
qemu-arm, Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On 15/8/24 10:24, Mark Cave-Ayland wrote:
> On 14/08/2024 22:38, BALATON Zoltan wrote:
>
>> On Wed, 14 Aug 2024, Bernhard Beschow wrote:
>>> The serial port's frequency is set via the "baudbase" property nowadays.
>>
>> Please keep it as some devices might have registers that set this freq
>> and this function will be needed for emulating that even if it's not
>> emulated currently.
>
> Why not remove the function if it isn't being used? In general having
> unused code lying around is bad as it tends to bitrot.
>
> If someone did want to re-add this functionality then you would likely
> want to do it with clock API rather than using a separate function.
I also ended using it in a branch including this old series:
https://lore.kernel.org/qemu-devel/20200806130340.17316-1-f4bug@amsat.org/
Not my priority anymore, so I don't mind re-adding it.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/5] hw/char/serial: Remove unused funtion
2024-08-15 9:26 ` Philippe Mathieu-Daudé
@ 2024-08-15 18:00 ` Bernhard Beschow
0 siblings, 0 replies; 16+ messages in thread
From: Bernhard Beschow @ 2024-08-15 18:00 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Mark Cave-Ayland, BALATON Zoltan
Cc: qemu-devel, Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, Jan Kiszka,
Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Daniel Henrique Barboza,
Steven Lee, Huacai Chen, Richard Henderson, Strahinja Jankovic,
qemu-arm, Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
Am 15. August 2024 09:26:38 UTC schrieb "Philippe Mathieu-Daudé" <philmd@linaro.org>:
>On 15/8/24 10:24, Mark Cave-Ayland wrote:
>> On 14/08/2024 22:38, BALATON Zoltan wrote:
>>
>>> On Wed, 14 Aug 2024, Bernhard Beschow wrote:
>>>> The serial port's frequency is set via the "baudbase" property nowadays.
>>>
>>> Please keep it as some devices might have registers that set this freq and this function will be needed for emulating that even if it's not emulated currently.
>>
>> Why not remove the function if it isn't being used? In general having unused code lying around is bad as it tends to bitrot.
>>
>> If someone did want to re-add this functionality then you would likely want to do it with clock API rather than using a separate function.
>
>I also ended using it in a branch including this old series:
>https://lore.kernel.org/qemu-devel/20200806130340.17316-1-f4bug@amsat.org/
>
>Not my priority anymore, so I don't mind re-adding it.
>
This removal seems debatable. I'd err on the safe side and drop the patch in v2.
Any further comments?
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine
2024-08-14 18:15 ` [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine Bernhard Beschow
@ 2024-08-16 9:18 ` Philippe Mathieu-Daudé
2024-08-21 14:42 ` Mark Cave-Ayland
1 sibling, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-08-16 9:18 UTC (permalink / raw)
To: Bernhard Beschow, qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Mark Cave-Ayland,
Daniel Henrique Barboza, Steven Lee, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On 14/8/24 20:15, Bernhard Beschow wrote:
> The machine calls serial_hds_isa_init() which is provided by serial-isa.c,
> guarded by SERIAL_ISA.
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
> hw/ppc/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 1/5] hw: Remove unused inclusion of hw/char/serial.h
2024-08-14 18:15 ` [PATCH 1/5] hw: Remove unused inclusion of hw/char/serial.h Bernhard Beschow
@ 2024-08-21 14:39 ` Mark Cave-Ayland
0 siblings, 0 replies; 16+ messages in thread
From: Mark Cave-Ayland @ 2024-08-21 14:39 UTC (permalink / raw)
To: Bernhard Beschow, qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On 14/08/2024 19:15, Bernhard Beschow wrote:
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
> hw/char/riscv_htif.c | 1 -
> hw/ppc/prep.c | 1 -
> hw/riscv/sifive_e.c | 1 -
> hw/riscv/sifive_u.c | 1 -
> 4 files changed, 4 deletions(-)
>
> diff --git a/hw/char/riscv_htif.c b/hw/char/riscv_htif.c
> index 9bef60def1..54fd55c3e6 100644
> --- a/hw/char/riscv_htif.c
> +++ b/hw/char/riscv_htif.c
> @@ -24,7 +24,6 @@
> #include "qapi/error.h"
> #include "qemu/log.h"
> #include "hw/char/riscv_htif.h"
> -#include "hw/char/serial.h"
> #include "chardev/char.h"
> #include "chardev/char-fe.h"
> #include "qemu/timer.h"
> diff --git a/hw/ppc/prep.c b/hw/ppc/prep.c
> index 4eb5477069..fb58c312ac 100644
> --- a/hw/ppc/prep.c
> +++ b/hw/ppc/prep.c
> @@ -25,7 +25,6 @@
>
> #include "qemu/osdep.h"
> #include "hw/rtc/m48t59.h"
> -#include "hw/char/serial.h"
> #include "hw/block/fdc.h"
> #include "net/net.h"
> #include "hw/isa/isa.h"
> diff --git a/hw/riscv/sifive_e.c b/hw/riscv/sifive_e.c
> index 87d9602383..5a1959f2a9 100644
> --- a/hw/riscv/sifive_e.c
> +++ b/hw/riscv/sifive_e.c
> @@ -35,7 +35,6 @@
> #include "hw/boards.h"
> #include "hw/loader.h"
> #include "hw/sysbus.h"
> -#include "hw/char/serial.h"
> #include "hw/misc/unimp.h"
> #include "target/riscv/cpu.h"
> #include "hw/riscv/riscv_hart.h"
> diff --git a/hw/riscv/sifive_u.c b/hw/riscv/sifive_u.c
> index af5f923f54..efc8443c84 100644
> --- a/hw/riscv/sifive_u.c
> +++ b/hw/riscv/sifive_u.c
> @@ -43,7 +43,6 @@
> #include "hw/irq.h"
> #include "hw/loader.h"
> #include "hw/sysbus.h"
> -#include "hw/char/serial.h"
> #include "hw/cpu/cluster.h"
> #include "hw/misc/unimp.h"
> #include "hw/sd/sd.h"
For the PReP machine:
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
ATB,
Mark.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine
2024-08-14 18:15 ` [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine Bernhard Beschow
2024-08-16 9:18 ` Philippe Mathieu-Daudé
@ 2024-08-21 14:42 ` Mark Cave-Ayland
1 sibling, 0 replies; 16+ messages in thread
From: Mark Cave-Ayland @ 2024-08-21 14:42 UTC (permalink / raw)
To: Bernhard Beschow, qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On 14/08/2024 19:15, Bernhard Beschow wrote:
> The machine calls serial_hds_isa_init() which is provided by serial-isa.c,
> guarded by SERIAL_ISA.
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
> hw/ppc/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/ppc/Kconfig b/hw/ppc/Kconfig
> index c235519881..5addad1124 100644
> --- a/hw/ppc/Kconfig
> +++ b/hw/ppc/Kconfig
> @@ -39,6 +39,7 @@ config POWERNV
> select PCI_POWERNV
> select PCA9552
> select PCA9554
> + select SERIAL_ISA
> select SSI
> select SSI_M25P80
> select PNV_SPI
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
ATB,
Mark.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 4/5] hw/char/serial.h: Extract serial-isa.h
2024-08-14 18:15 ` [PATCH 4/5] hw/char/serial.h: Extract serial-isa.h Bernhard Beschow
@ 2024-08-21 14:44 ` Mark Cave-Ayland
0 siblings, 0 replies; 16+ messages in thread
From: Mark Cave-Ayland @ 2024-08-21 14:44 UTC (permalink / raw)
To: Bernhard Beschow, qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On 14/08/2024 19:15, Bernhard Beschow wrote:
> The includes where updated based on compile errors. Now, the inclusion of the
> header roughly matches Kconfig dependencies:
>
> # grep -r -e "select SERIAL_ISA"
> hw/ppc/Kconfig: select SERIAL_ISA
> hw/isa/Kconfig: select SERIAL_ISA
> hw/sparc64/Kconfig: select SERIAL_ISA
> hw/i386/Kconfig: select SERIAL_ISA
> hw/i386/Kconfig: select SERIAL_ISA # for serial_hds_isa_init()
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
> include/hw/char/serial-isa.h | 38 ++++++++++++++++++++++++++++++++++++
> include/hw/char/serial.h | 9 ---------
> hw/char/serial-isa.c | 1 +
> 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/ppc/pnv.c | 2 +-
> hw/sparc64/sun4u.c | 1 +
> 10 files changed, 46 insertions(+), 15 deletions(-)
> create mode 100644 include/hw/char/serial-isa.h
>
> diff --git a/include/hw/char/serial-isa.h b/include/hw/char/serial-isa.h
> new file mode 100644
> index 0000000000..8517afa128
> --- /dev/null
> +++ b/include/hw/char/serial-isa.h
> @@ -0,0 +1,38 @@
> +/*
> + * QEMU ISA 16550A UART emulation
> + *
> + * Copyright (c) 2003-2004 Fabrice Bellard
> + * Copyright (c) 2008 Citrix Systems, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a copy
> + * of this software and associated documentation files (the "Software"), to deal
> + * in the Software without restriction, including without limitation the rights
> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
> + * copies of the Software, and to permit persons to whom the Software is
> + * furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
> + * THE SOFTWARE.
> + */
> +
> +#ifndef HW_SERIAL_ISA_H
> +#define HW_SERIAL_ISA_H
> +
> +#include "hw/isa/isa.h"
> +
> +#define MAX_ISA_SERIAL_PORTS 4
> +
> +#define TYPE_ISA_SERIAL "isa-serial"
> +void serial_hds_isa_init(ISABus *bus, int from, int to);
> +void isa_serial_set_iobase(ISADevice *serial, hwaddr iobase);
> +void isa_serial_set_enabled(ISADevice *serial, bool enabled);
> +
> +#endif
> diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h
> index 40aad21df3..ccf322a031 100644
> --- a/include/hw/char/serial.h
> +++ b/include/hw/char/serial.h
> @@ -104,13 +104,4 @@ SerialMM *serial_mm_init(MemoryRegion *address_space,
> qemu_irq irq, int baudbase,
> Chardev *chr, enum device_endian end);
>
> -/* serial-isa.c */
> -
> -#define MAX_ISA_SERIAL_PORTS 4
> -
> -#define TYPE_ISA_SERIAL "isa-serial"
> -void serial_hds_isa_init(ISABus *bus, int from, int to);
> -void isa_serial_set_iobase(ISADevice *serial, hwaddr iobase);
> -void isa_serial_set_enabled(ISADevice *serial, bool enabled);
> -
> #endif
> diff --git a/hw/char/serial-isa.c b/hw/char/serial-isa.c
> index 329b352b9a..b562ec9d37 100644
> --- a/hw/char/serial-isa.c
> +++ b/hw/char/serial-isa.c
> @@ -29,6 +29,7 @@
> #include "sysemu/sysemu.h"
> #include "hw/acpi/acpi_aml_interface.h"
> #include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "hw/isa/isa.h"
> #include "hw/qdev-properties.h"
> #include "migration/vmstate.h"
> diff --git a/hw/i386/microvm-dt.c b/hw/i386/microvm-dt.c
> index b3049e4f9f..fc5db6ed7f 100644
> --- a/hw/i386/microvm-dt.c
> +++ b/hw/i386/microvm-dt.c
> @@ -34,7 +34,7 @@
> #include "qemu/cutils.h"
> #include "qapi/error.h"
> #include "sysemu/device_tree.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "hw/i386/fw_cfg.h"
> #include "hw/rtc/mc146818rtc.h"
> #include "hw/sysbus.h"
> diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c
> index 40edcee7af..05c882669c 100644
> --- a/hw/i386/microvm.c
> +++ b/hw/i386/microvm.c
> @@ -39,7 +39,7 @@
> #include "hw/intc/i8259.h"
> #include "hw/timer/i8254.h"
> #include "hw/rtc/mc146818rtc.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "hw/display/ramfb.h"
> #include "hw/i386/topology.h"
> #include "hw/i386/e820_memory_layout.h"
> diff --git a/hw/i386/pc.c b/hw/i386/pc.c
> index c74931d577..b273d4a2e1 100644
> --- a/hw/i386/pc.c
> +++ b/hw/i386/pc.c
> @@ -25,7 +25,7 @@
> #include "qemu/osdep.h"
> #include "qemu/units.h"
> #include "hw/i386/pc.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "hw/char/parallel.h"
> #include "hw/hyperv/hv-balloon.h"
> #include "hw/i386/fw_cfg.h"
> diff --git a/hw/isa/isa-superio.c b/hw/isa/isa-superio.c
> index a8c8c58ef7..cff756e791 100644
> --- a/hw/isa/isa-superio.c
> +++ b/hw/isa/isa-superio.c
> @@ -22,7 +22,7 @@
> #include "hw/qdev-properties.h"
> #include "hw/input/i8042.h"
> #include "hw/char/parallel-isa.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "trace.h"
>
> static void isa_superio_realize(DeviceState *dev, Error **errp)
> diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c
> index 505b44c4e6..1aa5c9613f 100644
> --- a/hw/isa/vt82c686.c
> +++ b/hw/isa/vt82c686.c
> @@ -17,7 +17,7 @@
> #include "hw/isa/vt82c686.h"
> #include "hw/block/fdc.h"
> #include "hw/char/parallel-isa.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "hw/pci/pci.h"
> #include "hw/qdev-properties.h"
> #include "hw/ide/pci.h"
> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
> index 3526852685..ffc3f2bca0 100644
> --- a/hw/ppc/pnv.c
> +++ b/hw/ppc/pnv.c
> @@ -53,7 +53,7 @@
> #include "hw/ppc/pnv_pnor.h"
>
> #include "hw/isa/isa.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "hw/rtc/mc146818rtc.h"
>
> #include <libfdt.h>
> diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
> index 4ece1ac1ff..a839abbf5d 100644
> --- a/hw/sparc64/sun4u.c
> +++ b/hw/sparc64/sun4u.c
> @@ -35,6 +35,7 @@
> #include "hw/qdev-properties.h"
> #include "hw/pci-host/sabre.h"
> #include "hw/char/serial.h"
> +#include "hw/char/serial-isa.h"
> #include "hw/char/parallel-isa.h"
> #include "hw/rtc/m48t59.h"
> #include "migration/vmstate.h"
This appears to be a straightforward substitution and LGTM:
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
ATB,
Mark.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 5/5] hw/char: Extract serial-mm
2024-08-14 18:15 ` [PATCH 5/5] hw/char: Extract serial-mm Bernhard Beschow
2024-08-15 0:35 ` Andrew Jeffery
@ 2024-08-21 14:48 ` Mark Cave-Ayland
1 sibling, 0 replies; 16+ messages in thread
From: Mark Cave-Ayland @ 2024-08-21 14:48 UTC (permalink / raw)
To: Bernhard Beschow, qemu-devel
Cc: Jiaxun Yang, Niek Linnenbank, Bin Meng, Paul Burton,
Stafford Horne, qemu-ppc, Tyrone Ting, Joel Stanley,
Eduardo Habkost, Sergio Lopez, Andrew Jeffery, Edgar E. Iglesias,
Paolo Bonzini, Hao Wu, Alistair Francis, BALATON Zoltan,
Jan Kiszka, Beniamino Galvani, Peter Maydell, Michael S. Tsirkin,
Aurelien Jarno, Palmer Dabbelt, Artyom Tarasenko, Jia Liu,
Helge Deller, Max Filippov, Subbaraya Sundeep, Weiwei Li,
Marc-André Lureau, Nicholas Piggin, Daniel Henrique Barboza,
Steven Lee, Philippe Mathieu-Daudé, Huacai Chen,
Richard Henderson, Strahinja Jankovic, qemu-arm,
Aleksandar Rikalo, Troy Lee, Jamin Lin, Liu Zhiwei,
Frédéric Barrat, Cédric Le Goater,
Hervé Poussineau, qemu-riscv, Marcel Apfelbaum, Song Gao
On 14/08/2024 19:15, Bernhard Beschow wrote:
> hw/char/serial currently contains the implementation of both TYPE_SERIAL and
> TYPE_SERIAL_MM. According to serial_class_init(), TYPE_SERIAL is an internal
> class while TYPE_SERIAL_MM is used by numerous machine types directly. Let's
> move the latter into its own module which makes the dependencies more obvious
> and the code more tidy.
>
> The includes and the dependencies have been converted mechanically except in the
> hw/char directories which were updated manually. The result was compile-tested.
> Now, only hw/char makes direct use of TYPE_SERIAL:
>
> # grep -r -e "select SERIAL" | grep -v SERIAL_
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
> hw/char/Kconfig: select SERIAL
>
> # grep -r -e "/serial\\.h"
> include/hw/char/serial-mm.h:#include "hw/char/serial.h"
> hw/char/serial-pci-multi.c:#include "hw/char/serial.h"
> hw/char/serial.c:#include "hw/char/serial.h"
> hw/char/serial-isa.c:#include "hw/char/serial.h"
> hw/char/serial-pci.c:#include "hw/char/serial.h"
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
> include/hw/arm/aspeed_soc.h | 2 +-
> include/hw/char/mchp_pfsoc_mmuart.h | 2 +-
> include/hw/char/serial-mm.h | 52 +++++++++
> include/hw/char/serial.h | 19 ----
> 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/serial-mm.c | 157 ++++++++++++++++++++++++++++
> hw/char/serial.c | 126 ----------------------
> hw/display/sm501.c | 2 +-
> hw/hppa/machine.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/ppc405_uc.c | 2 +-
> hw/ppc/ppc440_bamboo.c | 2 +-
> hw/ppc/sam460ex.c | 2 +-
> hw/ppc/virtex_ml507.c | 2 +-
> hw/riscv/virt.c | 2 +-
> hw/sparc64/niagara.c | 2 +-
> hw/sparc64/sun4u.c | 2 +-
> 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 +-
> 50 files changed, 276 insertions(+), 206 deletions(-)
> create mode 100644 include/hw/char/serial-mm.h
> create mode 100644 hw/char/serial-mm.c
>
> diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
> index 624d489e0d..689f52dae8 100644
> --- a/include/hw/arm/aspeed_soc.h
> +++ b/include/hw/arm/aspeed_soc.h
> @@ -39,7 +39,7 @@
> #include "hw/misc/unimp.h"
> #include "hw/misc/aspeed_peci.h"
> #include "hw/fsi/aspeed_apb2opb.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/intc/arm_gicv3.h"
>
> #define ASPEED_SPIS_NUM 2
> diff --git a/include/hw/char/mchp_pfsoc_mmuart.h b/include/hw/char/mchp_pfsoc_mmuart.h
> index b0e14ca355..a7b8b1b08b 100644
> --- a/include/hw/char/mchp_pfsoc_mmuart.h
> +++ b/include/hw/char/mchp_pfsoc_mmuart.h
> @@ -29,7 +29,7 @@
> #define HW_MCHP_PFSOC_MMUART_H
>
> #include "hw/sysbus.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
>
> #define MCHP_PFSOC_MMUART_REG_COUNT 13
>
> diff --git a/include/hw/char/serial-mm.h b/include/hw/char/serial-mm.h
> new file mode 100644
> index 0000000000..62a8489d69
> --- /dev/null
> +++ b/include/hw/char/serial-mm.h
> @@ -0,0 +1,52 @@
> +/*
> + * QEMU 16550A UART emulation
> + *
> + * Copyright (c) 2003-2004 Fabrice Bellard
> + * Copyright (c) 2008 Citrix Systems, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a copy
> + * of this software and associated documentation files (the "Software"), to deal
> + * in the Software without restriction, including without limitation the rights
> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
> + * copies of the Software, and to permit persons to whom the Software is
> + * furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
> + * THE SOFTWARE.
> + */
> +
> +#ifndef HW_SERIAL_MM_H
> +#define HW_SERIAL_MM_H
> +
> +#include "hw/char/serial.h"
> +#include "exec/memory.h"
> +#include "chardev/char.h"
> +#include "hw/sysbus.h"
> +#include "qom/object.h"
> +
> +#define TYPE_SERIAL_MM "serial-mm"
> +OBJECT_DECLARE_SIMPLE_TYPE(SerialMM, SERIAL_MM)
> +
> +struct SerialMM {
> + SysBusDevice parent;
> +
> + SerialState serial;
> +
> + uint8_t regshift;
> + uint8_t endianness;
> +};
> +
> +SerialMM *serial_mm_init(MemoryRegion *address_space,
> + hwaddr base, int regshift,
> + qemu_irq irq, int baudbase,
> + Chardev *chr, enum device_endian end);
> +
> +#endif
> diff --git a/include/hw/char/serial.h b/include/hw/char/serial.h
> index ccf322a031..942b372df6 100644
> --- a/include/hw/char/serial.h
> +++ b/include/hw/char/serial.h
> @@ -29,8 +29,6 @@
> #include "chardev/char-fe.h"
> #include "exec/memory.h"
> #include "qemu/fifo8.h"
> -#include "chardev/char.h"
> -#include "hw/sysbus.h"
> #include "qom/object.h"
>
> #define UART_FIFO_LENGTH 16 /* 16550A Fifo Length */
> @@ -81,27 +79,10 @@ struct SerialState {
> };
> typedef struct SerialState SerialState;
>
> -struct SerialMM {
> - SysBusDevice parent;
> -
> - SerialState serial;
> -
> - uint8_t regshift;
> - uint8_t endianness;
> -};
> -
> extern const VMStateDescription vmstate_serial;
> extern const MemoryRegionOps serial_io_ops;
>
> #define TYPE_SERIAL "serial"
> OBJECT_DECLARE_SIMPLE_TYPE(SerialState, SERIAL)
>
> -#define TYPE_SERIAL_MM "serial-mm"
> -OBJECT_DECLARE_SIMPLE_TYPE(SerialMM, SERIAL_MM)
> -
> -SerialMM *serial_mm_init(MemoryRegion *address_space,
> - hwaddr base, int regshift,
> - qemu_irq irq, int baudbase,
> - Chardev *chr, enum device_endian end);
> -
> #endif
> diff --git a/hw/arm/allwinner-a10.c b/hw/arm/allwinner-a10.c
> index 57d5d80159..08cdff61e4 100644
> --- a/hw/arm/allwinner-a10.c
> +++ b/hw/arm/allwinner-a10.c
> @@ -18,7 +18,7 @@
> #include "qemu/osdep.h"
> #include "qapi/error.h"
> #include "qemu/module.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/sysbus.h"
> #include "hw/arm/allwinner-a10.h"
> #include "hw/misc/unimp.h"
> diff --git a/hw/arm/allwinner-h3.c b/hw/arm/allwinner-h3.c
> index 6870c3fe96..9bc57cd266 100644
> --- a/hw/arm/allwinner-h3.c
> +++ b/hw/arm/allwinner-h3.c
> @@ -24,7 +24,7 @@
> #include "qemu/units.h"
> #include "hw/qdev-core.h"
> #include "hw/sysbus.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/misc/unimp.h"
> #include "hw/usb/hcd-ehci.h"
> #include "hw/loader.h"
> diff --git a/hw/arm/allwinner-r40.c b/hw/arm/allwinner-r40.c
> index b8c7202133..ced73009d6 100644
> --- a/hw/arm/allwinner-r40.c
> +++ b/hw/arm/allwinner-r40.c
> @@ -26,7 +26,7 @@
> #include "hw/boards.h"
> #include "hw/qdev-core.h"
> #include "hw/sysbus.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/misc/unimp.h"
> #include "hw/usb/hcd-ehci.h"
> #include "hw/loader.h"
> diff --git a/hw/arm/aspeed_ast2400.c b/hw/arm/aspeed_ast2400.c
> index d125886207..ecc81ecc79 100644
> --- a/hw/arm/aspeed_ast2400.c
> +++ b/hw/arm/aspeed_ast2400.c
> @@ -15,7 +15,7 @@
> #include "qapi/error.h"
> #include "hw/misc/unimp.h"
> #include "hw/arm/aspeed_soc.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "qemu/module.h"
> #include "qemu/error-report.h"
> #include "hw/i2c/aspeed_i2c.h"
> diff --git a/hw/arm/aspeed_soc_common.c b/hw/arm/aspeed_soc_common.c
> index 05551461ae..a5ff33c46d 100644
> --- a/hw/arm/aspeed_soc_common.c
> +++ b/hw/arm/aspeed_soc_common.c
> @@ -15,7 +15,7 @@
> #include "hw/qdev-properties.h"
> #include "hw/misc/unimp.h"
> #include "hw/arm/aspeed_soc.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
>
>
> const char *aspeed_soc_cpu_type(AspeedSoCClass *sc)
> diff --git a/hw/arm/kzm.c b/hw/arm/kzm.c
> index 2ccd6f8a76..fbd140e383 100644
> --- a/hw/arm/kzm.c
> +++ b/hw/arm/kzm.c
> @@ -22,7 +22,7 @@
> #include "exec/address-spaces.h"
> #include "net/net.h"
> #include "hw/net/lan9118.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "sysemu/qtest.h"
> #include "sysemu/sysemu.h"
> #include "qemu/cutils.h"
> diff --git a/hw/arm/msf2-soc.c b/hw/arm/msf2-soc.c
> index a94a10adcc..c4999ebce3 100644
> --- a/hw/arm/msf2-soc.c
> +++ b/hw/arm/msf2-soc.c
> @@ -26,7 +26,7 @@
> #include "qemu/units.h"
> #include "qapi/error.h"
> #include "exec/address-spaces.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/arm/msf2-soc.h"
> #include "hw/misc/unimp.h"
> #include "hw/qdev-clock.h"
> diff --git a/hw/arm/musicpal.c b/hw/arm/musicpal.c
> index 2020f73a57..940e4437d9 100644
> --- a/hw/arm/musicpal.c
> +++ b/hw/arm/musicpal.c
> @@ -18,7 +18,7 @@
> #include "net/net.h"
> #include "sysemu/sysemu.h"
> #include "hw/boards.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "qemu/timer.h"
> #include "hw/ptimer.h"
> #include "hw/qdev-properties.h"
> diff --git a/hw/arm/npcm7xx.c b/hw/arm/npcm7xx.c
> index cb7791301b..af04c4b7ec 100644
> --- a/hw/arm/npcm7xx.c
> +++ b/hw/arm/npcm7xx.c
> @@ -18,7 +18,7 @@
>
> #include "hw/arm/boot.h"
> #include "hw/arm/npcm7xx.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/loader.h"
> #include "hw/misc/unimp.h"
> #include "hw/qdev-clock.h"
> diff --git a/hw/arm/pxa2xx.c b/hw/arm/pxa2xx.c
> index 6b2e54473b..c1448e670e 100644
> --- a/hw/arm/pxa2xx.c
> +++ b/hw/arm/pxa2xx.c
> @@ -17,7 +17,7 @@
> #include "migration/vmstate.h"
> #include "hw/arm/pxa.h"
> #include "sysemu/sysemu.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/i2c/i2c.h"
> #include "hw/irq.h"
> #include "hw/qdev-properties.h"
> diff --git a/hw/char/omap_uart.c b/hw/char/omap_uart.c
> index c2ef4c137e..6ec0429996 100644
> --- a/hw/char/omap_uart.c
> +++ b/hw/char/omap_uart.c
> @@ -20,7 +20,7 @@
> #include "qemu/osdep.h"
> #include "chardev/char.h"
> #include "hw/arm/omap.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "exec/address-spaces.h"
>
> /* UARTs */
> diff --git a/hw/char/serial-mm.c b/hw/char/serial-mm.c
> new file mode 100644
> index 0000000000..2f67776b19
> --- /dev/null
> +++ b/hw/char/serial-mm.c
> @@ -0,0 +1,157 @@
> +/*
> + * QEMU 16550A UART emulation
> + *
> + * Copyright (c) 2003-2004 Fabrice Bellard
> + * Copyright (c) 2008 Citrix Systems, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a copy
> + * of this software and associated documentation files (the "Software"), to deal
> + * in the Software without restriction, including without limitation the rights
> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
> + * copies of the Software, and to permit persons to whom the Software is
> + * furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
> + * THE SOFTWARE.
> + */
> +
> +#include "qemu/osdep.h"
> +#include "hw/char/serial-mm.h"
> +#include "exec/cpu-common.h"
> +#include "migration/vmstate.h"
> +#include "qapi/error.h"
> +#include "hw/qdev-properties.h"
> +
> +static uint64_t serial_mm_read(void *opaque, hwaddr addr, unsigned size)
> +{
> + SerialMM *s = SERIAL_MM(opaque);
> + return serial_io_ops.read(&s->serial, addr >> s->regshift, 1);
> +}
> +
> +static void serial_mm_write(void *opaque, hwaddr addr,
> + uint64_t value, unsigned size)
> +{
> + SerialMM *s = SERIAL_MM(opaque);
> + value &= 255;
> + serial_io_ops.write(&s->serial, addr >> s->regshift, value, 1);
> +}
> +
> +static const MemoryRegionOps serial_mm_ops[3] = {
> + [DEVICE_NATIVE_ENDIAN] = {
> + .read = serial_mm_read,
> + .write = serial_mm_write,
> + .endianness = DEVICE_NATIVE_ENDIAN,
> + .valid.max_access_size = 8,
> + .impl.max_access_size = 8,
> + },
> + [DEVICE_LITTLE_ENDIAN] = {
> + .read = serial_mm_read,
> + .write = serial_mm_write,
> + .endianness = DEVICE_LITTLE_ENDIAN,
> + .valid.max_access_size = 8,
> + .impl.max_access_size = 8,
> + },
> + [DEVICE_BIG_ENDIAN] = {
> + .read = serial_mm_read,
> + .write = serial_mm_write,
> + .endianness = DEVICE_BIG_ENDIAN,
> + .valid.max_access_size = 8,
> + .impl.max_access_size = 8,
> + },
> +};
> +
> +static void serial_mm_realize(DeviceState *dev, Error **errp)
> +{
> + SerialMM *smm = SERIAL_MM(dev);
> + SerialState *s = &smm->serial;
> +
> + if (!qdev_realize(DEVICE(s), NULL, errp)) {
> + return;
> + }
> +
> + memory_region_init_io(&s->io, OBJECT(dev),
> + &serial_mm_ops[smm->endianness], smm, "serial",
> + 8 << smm->regshift);
> + sysbus_init_mmio(SYS_BUS_DEVICE(smm), &s->io);
> + sysbus_init_irq(SYS_BUS_DEVICE(smm), &smm->serial.irq);
> +}
> +
> +static const VMStateDescription vmstate_serial_mm = {
> + .name = "serial",
> + .version_id = 3,
> + .minimum_version_id = 2,
> + .fields = (const VMStateField[]) {
> + VMSTATE_STRUCT(serial, SerialMM, 0, vmstate_serial, SerialState),
> + VMSTATE_END_OF_LIST()
> + }
> +};
> +
> +SerialMM *serial_mm_init(MemoryRegion *address_space,
> + hwaddr base, int regshift,
> + qemu_irq irq, int baudbase,
> + Chardev *chr, enum device_endian end)
> +{
> + SerialMM *smm = SERIAL_MM(qdev_new(TYPE_SERIAL_MM));
> + MemoryRegion *mr;
> +
> + qdev_prop_set_uint8(DEVICE(smm), "regshift", regshift);
> + qdev_prop_set_uint32(DEVICE(smm), "baudbase", baudbase);
> + qdev_prop_set_chr(DEVICE(smm), "chardev", chr);
> + qdev_set_legacy_instance_id(DEVICE(smm), base, 2);
> + qdev_prop_set_uint8(DEVICE(smm), "endianness", end);
> + sysbus_realize_and_unref(SYS_BUS_DEVICE(smm), &error_fatal);
> +
> + sysbus_connect_irq(SYS_BUS_DEVICE(smm), 0, irq);
> + mr = sysbus_mmio_get_region(SYS_BUS_DEVICE(smm), 0);
> + memory_region_add_subregion(address_space, base, mr);
> +
> + return smm;
> +}
> +
> +static void serial_mm_instance_init(Object *o)
> +{
> + SerialMM *smm = SERIAL_MM(o);
> +
> + object_initialize_child(o, "serial", &smm->serial, TYPE_SERIAL);
> +
> + qdev_alias_all_properties(DEVICE(&smm->serial), o);
> +}
> +
> +static Property serial_mm_properties[] = {
> + /*
> + * Set the spacing between adjacent memory-mapped UART registers.
> + * Each register will be at (1 << regshift) bytes after the previous one.
> + */
> + DEFINE_PROP_UINT8("regshift", SerialMM, regshift, 0),
> + DEFINE_PROP_UINT8("endianness", SerialMM, endianness, DEVICE_NATIVE_ENDIAN),
> + DEFINE_PROP_END_OF_LIST(),
> +};
> +
> +static void serial_mm_class_init(ObjectClass *oc, void *data)
> +{
> + DeviceClass *dc = DEVICE_CLASS(oc);
> +
> + device_class_set_props(dc, serial_mm_properties);
> + dc->realize = serial_mm_realize;
> + dc->vmsd = &vmstate_serial_mm;
> +}
> +
> +static const TypeInfo types[] = {
> + {
> + .name = TYPE_SERIAL_MM,
> + .parent = TYPE_SYS_BUS_DEVICE,
> + .class_init = serial_mm_class_init,
> + .instance_init = serial_mm_instance_init,
> + .instance_size = sizeof(SerialMM),
> + },
> +};
> +
> +DEFINE_TYPES(types)
> diff --git a/hw/char/serial.c b/hw/char/serial.c
> index 6c5c4a23c7..b50a8a1313 100644
> --- a/hw/char/serial.c
> +++ b/hw/char/serial.c
> @@ -989,135 +989,9 @@ static const TypeInfo serial_info = {
> .class_init = serial_class_init,
> };
>
> -/* Memory mapped interface */
> -static uint64_t serial_mm_read(void *opaque, hwaddr addr,
> - unsigned size)
> -{
> - SerialMM *s = SERIAL_MM(opaque);
> - return serial_ioport_read(&s->serial, addr >> s->regshift, 1);
> -}
> -
> -static void serial_mm_write(void *opaque, hwaddr addr,
> - uint64_t value, unsigned size)
> -{
> - SerialMM *s = SERIAL_MM(opaque);
> - value &= 255;
> - serial_ioport_write(&s->serial, addr >> s->regshift, value, 1);
> -}
> -
> -static const MemoryRegionOps serial_mm_ops[3] = {
> - [DEVICE_NATIVE_ENDIAN] = {
> - .read = serial_mm_read,
> - .write = serial_mm_write,
> - .endianness = DEVICE_NATIVE_ENDIAN,
> - .valid.max_access_size = 8,
> - .impl.max_access_size = 8,
> - },
> - [DEVICE_LITTLE_ENDIAN] = {
> - .read = serial_mm_read,
> - .write = serial_mm_write,
> - .endianness = DEVICE_LITTLE_ENDIAN,
> - .valid.max_access_size = 8,
> - .impl.max_access_size = 8,
> - },
> - [DEVICE_BIG_ENDIAN] = {
> - .read = serial_mm_read,
> - .write = serial_mm_write,
> - .endianness = DEVICE_BIG_ENDIAN,
> - .valid.max_access_size = 8,
> - .impl.max_access_size = 8,
> - },
> -};
> -
> -static void serial_mm_realize(DeviceState *dev, Error **errp)
> -{
> - SerialMM *smm = SERIAL_MM(dev);
> - SerialState *s = &smm->serial;
> -
> - if (!qdev_realize(DEVICE(s), NULL, errp)) {
> - return;
> - }
> -
> - memory_region_init_io(&s->io, OBJECT(dev),
> - &serial_mm_ops[smm->endianness], smm, "serial",
> - 8 << smm->regshift);
> - sysbus_init_mmio(SYS_BUS_DEVICE(smm), &s->io);
> - sysbus_init_irq(SYS_BUS_DEVICE(smm), &smm->serial.irq);
> -}
> -
> -static const VMStateDescription vmstate_serial_mm = {
> - .name = "serial",
> - .version_id = 3,
> - .minimum_version_id = 2,
> - .fields = (const VMStateField[]) {
> - VMSTATE_STRUCT(serial, SerialMM, 0, vmstate_serial, SerialState),
> - VMSTATE_END_OF_LIST()
> - }
> -};
> -
> -SerialMM *serial_mm_init(MemoryRegion *address_space,
> - hwaddr base, int regshift,
> - qemu_irq irq, int baudbase,
> - Chardev *chr, enum device_endian end)
> -{
> - SerialMM *smm = SERIAL_MM(qdev_new(TYPE_SERIAL_MM));
> - MemoryRegion *mr;
> -
> - qdev_prop_set_uint8(DEVICE(smm), "regshift", regshift);
> - qdev_prop_set_uint32(DEVICE(smm), "baudbase", baudbase);
> - qdev_prop_set_chr(DEVICE(smm), "chardev", chr);
> - qdev_set_legacy_instance_id(DEVICE(smm), base, 2);
> - qdev_prop_set_uint8(DEVICE(smm), "endianness", end);
> - sysbus_realize_and_unref(SYS_BUS_DEVICE(smm), &error_fatal);
> -
> - sysbus_connect_irq(SYS_BUS_DEVICE(smm), 0, irq);
> - mr = sysbus_mmio_get_region(SYS_BUS_DEVICE(smm), 0);
> - memory_region_add_subregion(address_space, base, mr);
> -
> - return smm;
> -}
> -
> -static void serial_mm_instance_init(Object *o)
> -{
> - SerialMM *smm = SERIAL_MM(o);
> -
> - object_initialize_child(o, "serial", &smm->serial, TYPE_SERIAL);
> -
> - qdev_alias_all_properties(DEVICE(&smm->serial), o);
> -}
> -
> -static Property serial_mm_properties[] = {
> - /*
> - * Set the spacing between adjacent memory-mapped UART registers.
> - * Each register will be at (1 << regshift) bytes after the
> - * previous one.
> - */
> - DEFINE_PROP_UINT8("regshift", SerialMM, regshift, 0),
> - DEFINE_PROP_UINT8("endianness", SerialMM, endianness, DEVICE_NATIVE_ENDIAN),
> - DEFINE_PROP_END_OF_LIST(),
> -};
> -
> -static void serial_mm_class_init(ObjectClass *oc, void *data)
> -{
> - DeviceClass *dc = DEVICE_CLASS(oc);
> -
> - device_class_set_props(dc, serial_mm_properties);
> - dc->realize = serial_mm_realize;
> - dc->vmsd = &vmstate_serial_mm;
> -}
> -
> -static const TypeInfo serial_mm_info = {
> - .name = TYPE_SERIAL_MM,
> - .parent = TYPE_SYS_BUS_DEVICE,
> - .class_init = serial_mm_class_init,
> - .instance_init = serial_mm_instance_init,
> - .instance_size = sizeof(SerialMM),
> -};
> -
> static void serial_register_types(void)
> {
> type_register_static(&serial_info);
> - type_register_static(&serial_mm_info);
> }
>
> type_init(serial_register_types)
> diff --git a/hw/display/sm501.c b/hw/display/sm501.c
> index 26dc8170d8..ad47fb6010 100644
> --- a/hw/display/sm501.c
> +++ b/hw/display/sm501.c
> @@ -29,7 +29,7 @@
> #include "qemu/log.h"
> #include "qemu/module.h"
> #include "hw/usb/hcd-ohci.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "ui/console.h"
> #include "hw/sysbus.h"
> #include "migration/vmstate.h"
> diff --git a/hw/hppa/machine.c b/hw/hppa/machine.c
> index 5d0a8739de..0ab54b0676 100644
> --- a/hw/hppa/machine.c
> +++ b/hw/hppa/machine.c
> @@ -17,7 +17,7 @@
> #include "sysemu/runstate.h"
> #include "hw/rtc/mc146818rtc.h"
> #include "hw/timer/i8254.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/char/parallel.h"
> #include "hw/intc/i8259.h"
> #include "hw/input/lasips2.h"
> diff --git a/hw/loongarch/virt.c b/hw/loongarch/virt.c
> index 29040422aa..e3a6a252be 100644
> --- a/hw/loongarch/virt.c
> +++ b/hw/loongarch/virt.c
> @@ -9,7 +9,7 @@
> #include "qemu/datadir.h"
> #include "qapi/error.h"
> #include "hw/boards.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "sysemu/kvm.h"
> #include "sysemu/tcg.h"
> #include "sysemu/sysemu.h"
> diff --git a/hw/microblaze/petalogix_ml605_mmu.c b/hw/microblaze/petalogix_ml605_mmu.c
> index 0f5fabc32e..b4183c5267 100644
> --- a/hw/microblaze/petalogix_ml605_mmu.c
> +++ b/hw/microblaze/petalogix_ml605_mmu.c
> @@ -34,7 +34,7 @@
> #include "hw/block/flash.h"
> #include "sysemu/sysemu.h"
> #include "hw/boards.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/qdev-properties.h"
> #include "exec/address-spaces.h"
> #include "hw/ssi/ssi.h"
> diff --git a/hw/mips/boston.c b/hw/mips/boston.c
> index 1b44fb354c..1ced1e337a 100644
> --- a/hw/mips/boston.c
> +++ b/hw/mips/boston.c
> @@ -22,7 +22,7 @@
>
> #include "elf.h"
> #include "hw/boards.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/ide/pci.h"
> #include "hw/ide/ahci-pci.h"
> #include "hw/loader.h"
> diff --git a/hw/mips/jazz.c b/hw/mips/jazz.c
> index 1bc17e69d3..a3b39c8663 100644
> --- a/hw/mips/jazz.c
> +++ b/hw/mips/jazz.c
> @@ -28,7 +28,7 @@
> #include "hw/mips/mips.h"
> #include "hw/intc/i8259.h"
> #include "hw/dma/i8257.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/char/parallel.h"
> #include "hw/isa/isa.h"
> #include "hw/block/fdc.h"
> diff --git a/hw/mips/loongson3_virt.c b/hw/mips/loongson3_virt.c
> index 408e3d7054..4a27d7b9da 100644
> --- a/hw/mips/loongson3_virt.c
> +++ b/hw/mips/loongson3_virt.c
> @@ -29,7 +29,7 @@
> #include "qemu/datadir.h"
> #include "qapi/error.h"
> #include "elf.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/intc/loongson_liointc.h"
> #include "hw/mips/mips.h"
> #include "hw/mips/fw_cfg.h"
> diff --git a/hw/mips/malta.c b/hw/mips/malta.c
> index 664a2ae0a9..1df00c4bf9 100644
> --- a/hw/mips/malta.c
> +++ b/hw/mips/malta.c
> @@ -31,7 +31,7 @@
> #include "hw/clock.h"
> #include "hw/southbridge/piix.h"
> #include "hw/isa/superio.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "net/net.h"
> #include "hw/boards.h"
> #include "hw/i2c/smbus_eeprom.h"
> diff --git a/hw/mips/mipssim.c b/hw/mips/mipssim.c
> index 9170d6c474..a07732d3dc 100644
> --- a/hw/mips/mipssim.c
> +++ b/hw/mips/mipssim.c
> @@ -31,7 +31,7 @@
> #include "exec/address-spaces.h"
> #include "hw/clock.h"
> #include "hw/mips/mips.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "net/net.h"
> #include "sysemu/sysemu.h"
> #include "hw/boards.h"
> diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
> index bffd6f721f..9fb63515ef 100644
> --- a/hw/openrisc/openrisc_sim.c
> +++ b/hw/openrisc/openrisc_sim.c
> @@ -24,7 +24,7 @@
> #include "cpu.h"
> #include "hw/irq.h"
> #include "hw/boards.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "net/net.h"
> #include "hw/openrisc/boot.h"
> #include "hw/qdev-properties.h"
> diff --git a/hw/openrisc/virt.c b/hw/openrisc/virt.c
> index f8a68a6a6b..47d2c9bd3c 100644
> --- a/hw/openrisc/virt.c
> +++ b/hw/openrisc/virt.c
> @@ -14,7 +14,7 @@
> #include "exec/address-spaces.h"
> #include "hw/irq.h"
> #include "hw/boards.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/core/split-irq.h"
> #include "hw/openrisc/boot.h"
> #include "hw/misc/sifive_test.h"
> diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c
> index 3bd12b54ab..b760c6d6a2 100644
> --- a/hw/ppc/e500.c
> +++ b/hw/ppc/e500.c
> @@ -24,7 +24,7 @@
> #include "net/net.h"
> #include "qemu/config-file.h"
> #include "hw/block/flash.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/pci/pci.h"
> #include "sysemu/block-backend-io.h"
> #include "sysemu/sysemu.h"
> diff --git a/hw/ppc/ppc405_uc.c b/hw/ppc/ppc405_uc.c
> index 0cc68178ad..36b18d6694 100644
> --- a/hw/ppc/ppc405_uc.c
> +++ b/hw/ppc/ppc405_uc.c
> @@ -32,7 +32,7 @@
> #include "hw/irq.h"
> #include "hw/qdev-properties.h"
> #include "ppc405.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "qemu/timer.h"
> #include "sysemu/reset.h"
> #include "sysemu/sysemu.h"
> diff --git a/hw/ppc/ppc440_bamboo.c b/hw/ppc/ppc440_bamboo.c
> index 73f80cf706..96d9ce65c2 100644
> --- a/hw/ppc/ppc440_bamboo.c
> +++ b/hw/ppc/ppc440_bamboo.c
> @@ -23,7 +23,7 @@
> #include "sysemu/device_tree.h"
> #include "hw/loader.h"
> #include "elf.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/ppc/ppc.h"
> #include "hw/pci-host/ppc4xx.h"
> #include "sysemu/sysemu.h"
> diff --git a/hw/ppc/sam460ex.c b/hw/ppc/sam460ex.c
> index 8dc75fb9f0..1fce093ac8 100644
> --- a/hw/ppc/sam460ex.c
> +++ b/hw/ppc/sam460ex.c
> @@ -31,7 +31,7 @@
> #include "sysemu/sysemu.h"
> #include "sysemu/reset.h"
> #include "hw/sysbus.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/i2c/ppc4xx_i2c.h"
> #include "hw/i2c/smbus_eeprom.h"
> #include "hw/ide/pci.h"
> diff --git a/hw/ppc/virtex_ml507.c b/hw/ppc/virtex_ml507.c
> index c49da1f46f..235281e939 100644
> --- a/hw/ppc/virtex_ml507.c
> +++ b/hw/ppc/virtex_ml507.c
> @@ -28,7 +28,7 @@
> #include "exec/page-protection.h"
> #include "cpu.h"
> #include "hw/sysbus.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/block/flash.h"
> #include "sysemu/sysemu.h"
> #include "sysemu/reset.h"
> diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
> index 9981e0f6c9..b1a0811071 100644
> --- a/hw/riscv/virt.c
> +++ b/hw/riscv/virt.c
> @@ -27,7 +27,7 @@
> #include "hw/loader.h"
> #include "hw/sysbus.h"
> #include "hw/qdev-properties.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "target/riscv/cpu.h"
> #include "hw/core/sysbus-fdt.h"
> #include "target/riscv/pmu.h"
> diff --git a/hw/sparc64/niagara.c b/hw/sparc64/niagara.c
> index ab3c4ec346..67ec403e1d 100644
> --- a/hw/sparc64/niagara.c
> +++ b/hw/sparc64/niagara.c
> @@ -27,7 +27,7 @@
> #include "qemu/units.h"
> #include "cpu.h"
> #include "hw/boards.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/misc/unimp.h"
> #include "hw/loader.h"
> #include "hw/sparc/sparc64.h"
> diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
> index a839abbf5d..541c7f74fa 100644
> --- a/hw/sparc64/sun4u.c
> +++ b/hw/sparc64/sun4u.c
> @@ -34,8 +34,8 @@
> #include "hw/pci/pci_host.h"
> #include "hw/qdev-properties.h"
> #include "hw/pci-host/sabre.h"
> -#include "hw/char/serial.h"
> #include "hw/char/serial-isa.h"
> +#include "hw/char/serial-mm.h"
> #include "hw/char/parallel-isa.h"
> #include "hw/rtc/m48t59.h"
> #include "migration/vmstate.h"
> diff --git a/hw/xtensa/xtfpga.c b/hw/xtensa/xtfpga.c
> index 955e8867a3..45b29d3b4e 100644
> --- a/hw/xtensa/xtfpga.c
> +++ b/hw/xtensa/xtfpga.c
> @@ -35,7 +35,7 @@
> #include "hw/qdev-properties.h"
> #include "elf.h"
> #include "exec/memory.h"
> -#include "hw/char/serial.h"
> +#include "hw/char/serial-mm.h"
> #include "net/net.h"
> #include "hw/sysbus.h"
> #include "hw/block/flash.h"
> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
> index 1ad60da7aa..ecde52f4a4 100644
> --- a/hw/arm/Kconfig
> +++ b/hw/arm/Kconfig
> @@ -136,7 +136,7 @@ config MUSICPAL
> select MARVELL_88W8618
> select PTIMER
> select PFLASH_CFI02
> - select SERIAL
> + select SERIAL_MM
> select WM8750
>
> config NETDUINO2
> @@ -179,13 +179,13 @@ config OMAP
> select NAND
> select PFLASH_CFI01
> select SD
> - select SERIAL
> + select SERIAL_MM
>
> config PXA2XX
> bool
> select FRAMEBUFFER
> select I2C
> - select SERIAL
> + select SERIAL_MM
> select SD
> select SSI
> select USB_OHCI_SYSBUS
> @@ -399,7 +399,7 @@ config ALLWINNER_A10
> select ALLWINNER_EMAC
> select ALLWINNER_I2C
> select AXP2XX_PMU
> - select SERIAL
> + select SERIAL_MM
> select UNIMP
> select USB_OHCI_SYSBUS
>
> @@ -411,7 +411,7 @@ config ALLWINNER_H3
> select ALLWINNER_SUN8I_EMAC
> select ALLWINNER_I2C
> select ALLWINNER_WDT
> - select SERIAL
> + select SERIAL_MM
> select ARM_TIMER
> select ARM_GIC
> select UNIMP
> @@ -427,7 +427,7 @@ config ALLWINNER_R40
> select ALLWINNER_A10_PIT
> select ALLWINNER_WDT
> select AXP2XX_PMU
> - select SERIAL
> + select SERIAL_MM
> select ARM_TIMER
> select ARM_GIC
> select UNIMP
> @@ -542,7 +542,7 @@ config NPCM7XX
> select ISL_PMBUS_VR
> select PL310 # cache controller
> select PMBUS
> - select SERIAL
> + select SERIAL_MM
> select SSI
> select UNIMP
> select PCA954X
> @@ -564,7 +564,7 @@ config FSL_IMX31
> default y
> depends on TCG && ARM
> imply I2C_DEVICES
> - select SERIAL
> + select SERIAL_MM
> select IMX
> select IMX_I2C
> select WDT_IMX2
> @@ -593,7 +593,7 @@ config ASPEED_SOC
> select I2C
> select DPS310
> select PCA9552
> - select SERIAL
> + select SERIAL_MM
> select SMBUS_EEPROM
> select PCA954X
> select SSI
> @@ -681,7 +681,7 @@ config MSF2
> bool
> select ARM_V7M
> select PTIMER
> - select SERIAL
> + select SERIAL_MM
> select SSI
> select UNIMP
>
> diff --git a/hw/char/Kconfig b/hw/char/Kconfig
> index 4fd74ea878..4b73a803bf 100644
> --- a/hw/char/Kconfig
> +++ b/hw/char/Kconfig
> @@ -21,6 +21,10 @@ config SERIAL_ISA
> depends on ISA_BUS
> select SERIAL
>
> +config SERIAL_MM
> + bool
> + select SERIAL
> +
> config SERIAL_PCI
> bool
> default y if PCI_DEVICES
> diff --git a/hw/char/meson.build b/hw/char/meson.build
> index e5b13b6958..b99938f756 100644
> --- a/hw/char/meson.build
> +++ b/hw/char/meson.build
> @@ -14,6 +14,7 @@ system_ss.add(when: 'CONFIG_PL011', if_true: files('pl011.c'))
> system_ss.add(when: 'CONFIG_SCLPCONSOLE', if_true: files('sclpconsole.c', 'sclpconsole-lm.c'))
> system_ss.add(when: 'CONFIG_SERIAL', if_true: files('serial.c'))
> system_ss.add(when: 'CONFIG_SERIAL_ISA', if_true: files('serial-isa.c'))
> +system_ss.add(when: 'CONFIG_SERIAL_MM', if_true: files('serial-mm.c'))
> system_ss.add(when: 'CONFIG_SERIAL_PCI', if_true: files('serial-pci.c'))
> system_ss.add(when: 'CONFIG_SERIAL_PCI_MULTI', if_true: files('serial-pci-multi.c'))
> system_ss.add(when: 'CONFIG_SHAKTI_UART', if_true: files('shakti_uart.c'))
> diff --git a/hw/display/Kconfig b/hw/display/Kconfig
> index a4552c8ed7..5fde4855e3 100644
> --- a/hw/display/Kconfig
> +++ b/hw/display/Kconfig
> @@ -76,7 +76,7 @@ config SM501
> bool
> select I2C
> select DDC
> - select SERIAL
> + select SERIAL_MM
> select USB_OHCI_SYSBUS
>
> config TCX
> diff --git a/hw/hppa/Kconfig b/hw/hppa/Kconfig
> index d4d457f4ab..9312c4294a 100644
> --- a/hw/hppa/Kconfig
> +++ b/hw/hppa/Kconfig
> @@ -9,7 +9,7 @@ config HPPA_B160L
> select ASTRO
> select DINO
> select LASI
> - select SERIAL
> + select SERIAL_MM
> select SERIAL_PCI
> select ISA_BUS
> select I8259
> diff --git a/hw/loongarch/Kconfig b/hw/loongarch/Kconfig
> index 0de713a439..6b67b34b7a 100644
> --- a/hw/loongarch/Kconfig
> +++ b/hw/loongarch/Kconfig
> @@ -9,7 +9,7 @@ config LOONGARCH_VIRT
> imply PCI_DEVICES
> imply NVDIMM
> imply TPM_TIS_SYSBUS
> - select SERIAL
> + select SERIAL_MM
> select VIRTIO_PCI
> select PLATFORM_BUS
> select LOONGARCH_IPI
> diff --git a/hw/microblaze/Kconfig b/hw/microblaze/Kconfig
> index d78ba843fa..b0214b2c8b 100644
> --- a/hw/microblaze/Kconfig
> +++ b/hw/microblaze/Kconfig
> @@ -13,7 +13,7 @@ config PETALOGIX_ML605
> default y
> depends on MICROBLAZE
> select PFLASH_CFI01
> - select SERIAL
> + select SERIAL_MM
> select SSI_M25P80
> select XILINX
> select XILINX_AXI
> diff --git a/hw/mips/Kconfig b/hw/mips/Kconfig
> index 692bede538..b09c89a017 100644
> --- a/hw/mips/Kconfig
> +++ b/hw/mips/Kconfig
> @@ -10,14 +10,14 @@ config MALTA
> select MIPS_CPS
> select PIIX
> select PFLASH_CFI01
> - select SERIAL
> + select SERIAL_MM
> select SMBUS_EEPROM
>
> config MIPSSIM
> bool
> default y
> depends on MIPS
> - select SERIAL
> + select SERIAL_MM
> select MIPSNET
>
> config JAZZ
> @@ -37,7 +37,7 @@ config JAZZ
> select FDC_SYSBUS
> select MC146818RTC
> select PCKBD
> - select SERIAL
> + select SERIAL_MM
> select PARALLEL
> select DS1225Y
> select JAZZ_LED
> @@ -65,7 +65,7 @@ config LOONGSON3V
> imply VIRTIO_VGA
> imply QXL if SPICE
> imply USB_OHCI_PCI
> - select SERIAL
> + select SERIAL_MM
> select GOLDFISH_RTC
> select LOONGSON_IPI
> select LOONGSON_LIOINTC
> @@ -89,7 +89,7 @@ config MIPS_BOSTON
> select MIPS_CPS
> select PCI_EXPRESS_XILINX
> select AHCI_ICH9
> - select SERIAL
> + select SERIAL_MM
>
> config FW_CFG_MIPS
> bool
> diff --git a/hw/openrisc/Kconfig b/hw/openrisc/Kconfig
> index 76b953c62c..0702f622a5 100644
> --- a/hw/openrisc/Kconfig
> +++ b/hw/openrisc/Kconfig
> @@ -3,7 +3,7 @@ config OR1K_SIM
> default y
> depends on OPENRISC
> select DEVICE_TREE
> - select SERIAL
> + select SERIAL_MM
> select OPENCORES_ETH
> select OMPIC
> select SPLIT_IRQ
> @@ -19,6 +19,6 @@ config OR1K_VIRT
> select PCI
> select PCI_EXPRESS_GENERIC_BRIDGE
> select GOLDFISH_RTC
> - select SERIAL
> + select SERIAL_MM
> select SIFIVE_TEST
> select VIRTIO_MMIO
> diff --git a/hw/ppc/Kconfig b/hw/ppc/Kconfig
> index 5addad1124..b44d91bebb 100644
> --- a/hw/ppc/Kconfig
> +++ b/hw/ppc/Kconfig
> @@ -51,7 +51,7 @@ config PPC405
> select M48T59
> select PFLASH_CFI02
> select PPC4XX
> - select SERIAL
> + select SERIAL_MM
>
> config PPC440
> bool
> @@ -63,7 +63,7 @@ config PPC440
> select PCI_EXPRESS
> select PPC440_PCIX
> select PPC4XX
> - select SERIAL
> + select SERIAL_MM
> select FDT_PPC
>
> config PPC4XX
> @@ -80,7 +80,7 @@ config SAM460EX
> select IDE_SII3112
> select M41T80
> select PPC440
> - select SERIAL
> + select SERIAL_MM
> select SM501
> select SMBUS_EEPROM
> select USB_EHCI_SYSBUS
> @@ -163,7 +163,7 @@ config E500
> select PLATFORM_BUS
> select PPCE500_PCI
> select SDHCI
> - select SERIAL
> + select SERIAL_MM
> select MPC_I2C
> select FDT_PPC
> select DS1338
> @@ -187,7 +187,7 @@ config VIRTEX
> depends on PPC && FDT
> select PPC4XX
> select PFLASH_CFI01
> - select SERIAL
> + select SERIAL_MM
> select XILINX
> select XILINX_ETHLITE
> select FDT_PPC
> diff --git a/hw/riscv/Kconfig b/hw/riscv/Kconfig
> index a2030e3a6f..44695ff9f2 100644
> --- a/hw/riscv/Kconfig
> +++ b/hw/riscv/Kconfig
> @@ -44,7 +44,7 @@ config RISCV_VIRT
> select PCI
> select PCI_EXPRESS_GENERIC_BRIDGE
> select PFLASH_CFI01
> - select SERIAL
> + select SERIAL_MM
> select RISCV_ACLINT
> select RISCV_APLIC
> select RISCV_IMSIC
> diff --git a/hw/sparc64/Kconfig b/hw/sparc64/Kconfig
> index 3b948a2290..f764c8a219 100644
> --- a/hw/sparc64/Kconfig
> +++ b/hw/sparc64/Kconfig
> @@ -10,6 +10,7 @@ config SUN4U
> select ISA_BUS
> select FDC_ISA
> select SERIAL_ISA
> + select SERIAL_MM
> select PCI_SABRE
> select IDE_CMD646
> select PCKBD
> diff --git a/hw/xtensa/Kconfig b/hw/xtensa/Kconfig
> index fc5c785cfa..1f0492d89c 100644
> --- a/hw/xtensa/Kconfig
> +++ b/hw/xtensa/Kconfig
> @@ -18,4 +18,4 @@ config XTENSA_XTFPGA
> select DEVICE_TREE
> select OPENCORES_ETH
> select PFLASH_CFI01
> - select SERIAL
> + select SERIAL_MM
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
ATB,
Mark.
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2024-08-21 14:49 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-14 18:15 [PATCH 0/5] Serial device cleanup Bernhard Beschow
2024-08-14 18:15 ` [PATCH 1/5] hw: Remove unused inclusion of hw/char/serial.h Bernhard Beschow
2024-08-21 14:39 ` Mark Cave-Ayland
2024-08-14 18:15 ` [PATCH 2/5] hw/char/serial: Remove unused funtion Bernhard Beschow
2024-08-14 21:38 ` BALATON Zoltan
2024-08-15 8:24 ` Mark Cave-Ayland
2024-08-15 9:26 ` Philippe Mathieu-Daudé
2024-08-15 18:00 ` Bernhard Beschow
2024-08-14 18:15 ` [PATCH 3/5] hw/ppc/Kconfig: Add missing SERIAL_ISA dependency to POWERNV machine Bernhard Beschow
2024-08-16 9:18 ` Philippe Mathieu-Daudé
2024-08-21 14:42 ` Mark Cave-Ayland
2024-08-14 18:15 ` [PATCH 4/5] hw/char/serial.h: Extract serial-isa.h Bernhard Beschow
2024-08-21 14:44 ` Mark Cave-Ayland
2024-08-14 18:15 ` [PATCH 5/5] hw/char: Extract serial-mm Bernhard Beschow
2024-08-15 0:35 ` Andrew Jeffery
2024-08-21 14:48 ` Mark Cave-Ayland
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).