On 08/01/2024 21.27, David Woodhouse wrote:
> From: David Woodhouse <dwmw@amazon.co.uk>
>
> Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
> ---
> hw/riscv/microchip_pfsoc.c | 14 ++------------
> hw/riscv/sifive_u.c | 7 +------
> 2 files changed, 3 insertions(+), 18 deletions(-)
>
> diff --git a/hw/riscv/microchip_pfsoc.c b/hw/riscv/microchip_pfsoc.c
> index b775aa8946..7725dfbde5 100644
> --- a/hw/riscv/microchip_pfsoc.c
> +++ b/hw/riscv/microchip_pfsoc.c
> @@ -202,7 +202,6 @@ static void microchip_pfsoc_soc_realize(DeviceState *dev, Error **errp)
> MemoryRegion *envm_data = g_new(MemoryRegion, 1);
> MemoryRegion *qspi_xip_mem = g_new(MemoryRegion, 1);
> char *plic_hart_config;
> - NICInfo *nd;
> int i;
>
> sysbus_realize(SYS_BUS_DEVICE(&s->e_cpus), &error_abort);
> @@ -411,17 +410,8 @@ static void microchip_pfsoc_soc_realize(DeviceState *dev, Error **errp)
> memmap[MICROCHIP_PFSOC_USB].size);
>
> /* GEMs */
> -
> - nd = &nd_table[0];
> - if (nd->used) {
> - qemu_check_nic_model(nd, TYPE_CADENCE_GEM);
> - qdev_set_nic_properties(DEVICE(&s->gem0), nd);
> - }
> - nd = &nd_table[1];
> - if (nd->used) {
> - qemu_check_nic_model(nd, TYPE_CADENCE_GEM);
> - qdev_set_nic_properties(DEVICE(&s->gem1), nd);
> - }
> + qemu_configure_nic_device(DEVICE(&s->gem0), true, NULL);
> + qemu_configure_nic_device(DEVICE(&s->gem1), true, NULL);
>
> object_property_set_int(OBJECT(&s->gem0), "revision", GEM_REVISION, errp);
> object_property_set_int(OBJECT(&s->gem0), "phy-addr", 8, errp);
> diff --git a/hw/riscv/sifive_u.c b/hw/riscv/sifive_u.c
> index ec76dce6c9..5207ec1fa5 100644
> --- a/hw/riscv/sifive_u.c
> +++ b/hw/riscv/sifive_u.c
> @@ -789,7 +789,6 @@ static void sifive_u_soc_realize(DeviceState *dev, Error **errp)
> MemoryRegion *l2lim_mem = g_new(MemoryRegion, 1);
> char *plic_hart_config;
> int i, j;
> - NICInfo *nd = &nd_table[0];
>
> qdev_prop_set_uint32(DEVICE(&s->u_cpus), "num-harts", ms->smp.cpus - 1);
> qdev_prop_set_uint32(DEVICE(&s->u_cpus), "hartid-base", 1);
> @@ -893,11 +892,7 @@ static void sifive_u_soc_realize(DeviceState *dev, Error **errp)
> }
> sysbus_mmio_map(SYS_BUS_DEVICE(&s->otp), 0, memmap[SIFIVE_U_DEV_OTP].base);
>
> - /* FIXME use qdev NIC properties instead of nd_table[] */
> - if (nd->used) {
> - qemu_check_nic_model(nd, TYPE_CADENCE_GEM);
> - qdev_set_nic_properties(DEVICE(&s->gem), nd);
> - }
> + qemu_configure_nic_device(DEVICE(&s->gem), true, NULL);
> object_property_set_int(OBJECT(&s->gem), "revision", GEM_REVISION,
> &error_abort);
> if (!sysbus_realize(SYS_BUS_DEVICE(&s->gem), errp)) {
Reviewed-by: Thomas Huth <thuth@redhat.com>