diff options
| author | Markus Armbruster | 2020-07-15 16:04:40 +0200 |
|---|---|---|
| committer | Thomas Huth | 2020-07-21 08:41:15 +0200 |
| commit | 7ad36e2e241bd924f774a1f9fb208c102da58e50 (patch) | |
| tree | 6f21ee31b7408198bc21391806e04010b076e5f2 /hw/dma/sparc32_dma.c | |
| parent | msf2: Unbreak device-list-properties for "msf-soc" (diff) | |
| download | qemu-7ad36e2e241bd924f774a1f9fb208c102da58e50.tar.gz qemu-7ad36e2e241bd924f774a1f9fb208c102da58e50.tar.xz qemu-7ad36e2e241bd924f774a1f9fb208c102da58e50.zip | |
hw: Mark nd_table[] misuse in realize methods FIXME
nd_table[] contains NIC configuration for boards to pick up. Device
code has no business looking there. Several devices do it anyway.
Two of them already have a suitable FIXME comment: "allwinner-a10" and
"msf2-soc". Copy it to the others: "allwinner-h3", "xlnx-versal",
"xlnx,zynqmp", "sparc32-ledma", "riscv.sifive.u.soc".
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20200715140440.3540942-3-armbru@redhat.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Niek Linnenbank <nieklinnenbank@gmail.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'hw/dma/sparc32_dma.c')
| -rw-r--r-- | hw/dma/sparc32_dma.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/dma/sparc32_dma.c b/hw/dma/sparc32_dma.c index 9459178866..bcd1626fbd 100644 --- a/hw/dma/sparc32_dma.c +++ b/hw/dma/sparc32_dma.c @@ -341,6 +341,7 @@ static void sparc32_ledma_device_realize(DeviceState *dev, Error **errp) DeviceState *d; NICInfo *nd = &nd_table[0]; + /* FIXME use qdev NIC properties instead of nd_table[] */ qemu_check_nic_model(nd, TYPE_LANCE); d = qdev_new(TYPE_LANCE); |