summary refs log tree commit diff stats
path: root/scripts/switch-timer-api
diff options
context:
space:
mode:
authorEric Auger <eric.auger@redhat.com>2018-11-13 10:47:58 +0000
committerPeter Maydell <peter.maydell@linaro.org>2018-11-13 10:47:58 +0000
commite9ac8e84f0603f13220aa856b0ee24df64d9686c (patch)
treeec4b1b1ab7202c8db0a69ce23d2798f123cf5e9e /scripts/switch-timer-api
parent671f11b2043803bbb795518d411b010a785a47db (diff)
downloadfocaccia-qemu-e9ac8e84f0603f13220aa856b0ee24df64d9686c.tar.gz
focaccia-qemu-e9ac8e84f0603f13220aa856b0ee24df64d9686c.zip
hw/arm/sysbus-fdt: Only call match_fn callback if the type matches
Commit af7d64ede0b9 (hw/arm/sysbus-fdt: Allow device matching with DT
compatible value) introduced a match_fn callback which gets called
for each registered combo to check whether a sysbus device can be
dynamically instantiated. However the callback gets called even if
the device type does not match the binding combo typename field.
This causes an assert when passing "-device ramfb" to the qemu
command line as vfio_platform_match() gets called on a non
vfio-platform device.

To fix this regression, let's change the add_fdt_node() logic so
that we first check the type and if the match_fn callback is defined,
then we also call it.

Binding combos only requesting a type check do not define the
match_fn callback.

Fixes: af7d64ede0b9 (hw/arm/sysbus-fdt: Allow device matching with
DT compatible value)

Signed-off-by: Eric Auger <eric.auger@redhat.com>
Reported-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Alex Williamson <alex.williamson@redhat.com>
Tested-by: Geert Uytterhoeven <geert+renesas@glider.be>
Message-id: 20181106184212.29377-1-eric.auger@redhat.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'scripts/switch-timer-api')
0 files changed, 0 insertions, 0 deletions