On Tue, Nov 12, 2024 at 07:10:30PM +0100, Philippe Mathieu-Daudé wrote:
> Rather than handling the MDIO registers as RAM, map them
> as unimplemented I/O within the device MR.
>
> The memory flat view becomes:
>
> (qemu) info mtree -f
> FlatView #0
> Root memory region: system
> 0000000081000000-00000000810007e3 (prio 0, i/o): xlnx.xps-ethernetlite
> 00000000810007e4-00000000810007f3 (prio 0, i/o): ethlite.mdio
> 00000000810007f4-0000000081001fff (prio 0, i/o): xlnx.xps-ethernetlite @00000000000007f4
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@amd.com>
> ---
> hw/net/xilinx_ethlite.c | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/hw/net/xilinx_ethlite.c b/hw/net/xilinx_ethlite.c
> index e6f6179fce..76b1e7d826 100644
> --- a/hw/net/xilinx_ethlite.c
> +++ b/hw/net/xilinx_ethlite.c
> @@ -25,13 +25,17 @@
> #include "qemu/osdep.h"
> #include "qemu/module.h"
> #include "qom/object.h"
> +#include "qapi/error.h"
> #include "exec/tswap.h"
> #include "hw/sysbus.h"
> #include "hw/irq.h"
> #include "hw/qdev-properties.h"
> +#include "hw/misc/unimp.h"
> #include "net/net.h"
> #include "trace.h"
>
> +#define A_MDIO_BASE 0x07e4
> +
> #define R_TX_BUF0 0
> #define R_TX_LEN0 (0x07f4 / 4)
> #define R_TX_GIE0 (0x07f8 / 4)
> @@ -71,6 +75,7 @@ struct XlnxXpsEthLite
> unsigned int txbuf;
> unsigned int rxbuf;
>
> + UnimplementedDeviceState mdio;
> uint32_t regs[R_MAX];
> };
>
> @@ -231,6 +236,14 @@ static void xilinx_ethlite_realize(DeviceState *dev, Error **errp)
> {
> XlnxXpsEthLite *s = XILINX_ETHLITE(dev);
>
> + object_initialize_child(OBJECT(dev), "ethlite.mdio", &s->mdio,
> + TYPE_UNIMPLEMENTED_DEVICE);
> + qdev_prop_set_string(DEVICE(&s->mdio), "name", "ethlite.mdio");
> + qdev_prop_set_uint64(DEVICE(&s->mdio), "size", 4 * 4);
> + sysbus_realize(SYS_BUS_DEVICE(&s->mdio), &error_fatal);
> + memory_region_add_subregion(&s->mmio, A_MDIO_BASE,
> + sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->mdio), 0));
> +
> qemu_macaddr_default_if_unset(&s->conf.macaddr);
> s->nic = qemu_new_nic(&net_xilinx_ethlite_info, &s->conf,
> object_get_typename(OBJECT(dev)), dev->id,
> --
> 2.45.2
>