diff options
author | Markus Armbruster <armbru@redhat.com> | 2020-05-05 17:29:21 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2020-05-15 07:07:58 +0200 |
commit | 9f742c28f52d55ff83dc441a0cea365239a4906d (patch) | |
tree | c0e2cbef83276a24fba9c43df94b0940064514a1 /hw/scsi | |
parent | 5462cc8110845f88ae80b82799121d15c9c3a8fc (diff) | |
download | qemu-9f742c28f52d55ff83dc441a0cea365239a4906d.zip qemu-9f742c28f52d55ff83dc441a0cea365239a4906d.tar.gz qemu-9f742c28f52d55ff83dc441a0cea365239a4906d.tar.bz2 |
qdev: Clean up qdev_connect_gpio_out_named()
Both qdev_connect_gpio_out_named() and device_set_realized() put
objects without a parent into the "/machine/unattached/" orphanage.
qdev_connect_gpio_out_named() needs a lengthy comment to explain how
it works. It exploits that object_property_add_child() can fail only
when we got a parent already, and ignoring that error does what we
want. True. If it failed due to "duplicate property", we'd be in
trouble, but that would be a programming error.
device_set_realized() is cleaner: it checks whether we need a parent,
then calls object_property_add_child(), aborting on failure. No need
for a comment, and programming errors get caught.
Change qdev_connect_gpio_out_named() to match.
Cc: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20200505152926.18877-14-armbru@redhat.com>
Diffstat (limited to 'hw/scsi')
0 files changed, 0 insertions, 0 deletions