RE: [PATCH 2/2] qdev: Let BusRealize() return a boolean value to indicate error

[Paul Durrant <xadimgnik@xxxxxxxxx>]

> -----Original Message-----
> From: Philippe Mathieu-Daudé <philippe.mathieu.daude@xxxxxxxxx> On Behalf Of 
> Philippe Mathieu-Daudé
> Sent: 20 September 2020 12:44
> To: Markus Armbruster <armbru@xxxxxxxxxx>; qemu-devel@xxxxxxxxxx
> Cc: Laurent Vivier <laurent@xxxxxxxxx>; Paolo Bonzini <pbonzini@xxxxxxxxxx>; 
> Anthony Perard
> <anthony.perard@xxxxxxxxxx>; Stefano Stabellini <sstabellini@xxxxxxxxxx>; 
> Daniel P. Berrangé
> <berrange@xxxxxxxxxx>; Eduardo Habkost <ehabkost@xxxxxxxxxx>; Paul Durrant 
> <paul@xxxxxxx>; Marcel
> Apfelbaum <marcel.apfelbaum@xxxxxxxxx>; Michael S. Tsirkin <mst@xxxxxxxxxx>; 
> xen-
> devel@xxxxxxxxxxxxxxxxxxxx; Philippe Mathieu-Daudé <f4bug@xxxxxxxxx>
> Subject: [PATCH 2/2] qdev: Let BusRealize() return a boolean value to 
> indicate error
> 
> Commit 9940b2cfbc0 introduced qdev_realize() and qbus_realize()
> with the ability to return a boolean value if an error occured,
> thus the caller does not need to check if the Error* pointer is
> set.
> Provide the same ability to the BusRealize type.
> 
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@xxxxxxxxx>
> ---
>  include/hw/qdev-core.h | 14 +++++++++++++-
>  hw/hyperv/vmbus.c      |  5 +++--
>  hw/nubus/nubus-bus.c   |  5 +++--
>  hw/pci/pci.c           | 12 +++++++++---
>  hw/xen/xen-bus.c       |  5 +++--

Acked-by: Paul Durrant <paul@xxxxxxx>