diff options
| author | Markus Armbruster <armbru@redhat.com> | 2020-07-15 16:04:40 +0200 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2020-07-21 08:41:15 +0200 |
| commit | 7ad36e2e241bd924f774a1f9fb208c102da58e50 (patch) | |
| tree | 6f21ee31b7408198bc21391806e04010b076e5f2 /hw/riscv | |
| parent | 2b0650205b71c2aa8bf6f877a8333ef25bf288b2 (diff) | |
| download | qemu-7ad36e2e241bd924f774a1f9fb208c102da58e50.zip qemu-7ad36e2e241bd924f774a1f9fb208c102da58e50.tar.gz qemu-7ad36e2e241bd924f774a1f9fb208c102da58e50.tar.bz2 | |
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/riscv')
| -rw-r--r-- | hw/riscv/sifive_u.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/riscv/sifive_u.c b/hw/riscv/sifive_u.c index 19a976c..e5682c3 100644 --- a/hw/riscv/sifive_u.c +++ b/hw/riscv/sifive_u.c @@ -714,6 +714,7 @@ static void sifive_u_soc_realize(DeviceState *dev, Error **errp) } sysbus_mmio_map(SYS_BUS_DEVICE(&s->otp), 0, memmap[SIFIVE_U_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); |