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/arm/allwinner-h3.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/arm/allwinner-h3.c')
| -rw-r--r-- | hw/arm/allwinner-h3.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/arm/allwinner-h3.c b/hw/arm/allwinner-h3.c index 8e09468e86..ff92ded82c 100644 --- a/hw/arm/allwinner-h3.c +++ b/hw/arm/allwinner-h3.c @@ -358,6 +358,7 @@ static void allwinner_h3_realize(DeviceState *dev, Error **errp) "sd-bus"); /* EMAC */ + /* FIXME use qdev NIC properties instead of nd_table[] */ if (nd_table[0].used) { qemu_check_nic_model(&nd_table[0], TYPE_AW_SUN8I_EMAC); qdev_set_nic_properties(DEVICE(&s->emac), &nd_table[0]); |