[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 19/37] mips: use sysbus_add_io()
From: |
Aleksandar Markovic |
Subject: |
Re: [PATCH v4 19/37] mips: use sysbus_add_io() |
Date: |
Wed, 27 Nov 2019 13:28:54 +0100 |
On Wednesday, November 20, 2019, Marc-André Lureau <address@hidden> wrote:
Signed-off-by: Marc-André Lureau <address@hidden>
---
hw/mips/mips_mipssim.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
I agree with the change, and with overall series in general, but please add at least a sentence in the commit message here, explaining what is achieved by the change.
With that sentence, certainly:
diff --git a/hw/mips/mips_mipssim.c b/hw/mips/mips_mipssim.c
index 3cd0e6eb33..2c2c7f25b2 100644
--- a/hw/mips/mips_mipssim.c
+++ b/hw/mips/mips_mipssim.c
@@ -227,8 +227,7 @@ mips_mipssim_init(MachineState *machine)
qdev_set_legacy_instance_id(dev, 0x3f8, 2);
qdev_init_nofail(dev);
sysbus_connect_irq(SYS_BUS_DEVICE(dev), 0, env->irq[4]);
- memory_region_add_subregion(get_system_io(), 0x3f8,
- &SERIAL_IO(dev)->serial.io);
+ sysbus_add_io(SYS_BUS_DEVICE(dev), 0x3f8, &SERIAL_IO(dev)->serial.io);
}
if (nd_table[0].used)
--
2.24.0
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, (continued)
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Aleksandar Markovic, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Marc-André Lureau, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Philippe Mathieu-Daudé, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Aleksandar Markovic, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Philippe Mathieu-Daudé, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Philippe Mathieu-Daudé, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Marc-André Lureau, 2019/11/27
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Aleksandar Markovic, 2019/11/27
[PATCH v4 19/37] mips: use sysbus_add_io(), Marc-André Lureau, 2019/11/20
[PATCH v4 20/37] mips: use sysbus_mmio_get_region() instead of internal fields, Marc-André Lureau, 2019/11/20
[PATCH v4 21/37] sm501: make SerialMM a child, export chardev property, Marc-André Lureau, 2019/11/20
[PATCH v4 22/37] vmmouse: replace PROP_PTR with PROP_LINK, Marc-André Lureau, 2019/11/20
[PATCH v4 23/37] lance: replace PROP_PTR with PROP_LINK, Marc-André Lureau, 2019/11/20
[PATCH v4 24/37] etraxfs: remove PROP_PTR usage, Marc-André Lureau, 2019/11/20
[PATCH v4 25/37] dp8393x: replace PROP_PTR with PROP_LINK, Marc-André Lureau, 2019/11/20