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

Philippe Mathieu-Daudé posted 2 patches 3 years, 7 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
include/hw/qdev-core.h | 26 +++++++++++++++++++++++++-
hw/hyperv/vmbus.c      |  5 +++--
hw/nubus/nubus-bus.c   |  5 +++--
hw/pci/pci.c           | 12 +++++++++---
hw/xen/xen-bus.c       |  5 +++--
5 files changed, 43 insertions(+), 10 deletions(-)
[PATCH 0/2] qdev: Let BusRealize() return a boolean value to indicate error
Posted by Philippe Mathieu-Daudé 3 years, 7 months ago
To ease error checking in DeviceRealize handlers, let BusRealize
return a boolean value, as qbus_realize() does.

Having DeviceRealize similarly return a boolean value is left as
an exercice for volunteers :)

Philippe Mathieu-Daudé (2):
  qdev: Document qbus_realize() and qbus_unrealize()
  qdev: Let BusRealize() return a boolean value to indicate error

 include/hw/qdev-core.h | 26 +++++++++++++++++++++++++-
 hw/hyperv/vmbus.c      |  5 +++--
 hw/nubus/nubus-bus.c   |  5 +++--
 hw/pci/pci.c           | 12 +++++++++---
 hw/xen/xen-bus.c       |  5 +++--
 5 files changed, 43 insertions(+), 10 deletions(-)

-- 
2.26.2

Re: [PATCH 0/2] qdev: Let BusRealize() return a boolean value to indicate error
Posted by Richard Henderson 3 years, 7 months ago
On 9/20/20 4:44 AM, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (2):
>   qdev: Document qbus_realize() and qbus_unrealize()
>   qdev: Let BusRealize() return a boolean value to indicate error

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~