On 20/06/20 17:38, Philippe Mathieu-Daudé wrote: > - } else { > - assert(!DEVICE_GET_CLASS(dev)->bus_type); > + } else if (DEVICE_GET_CLASS(dev)->bus_type) { > + error_report("%s: Unexpected bus '%s' for device '%s'", > + __func__, DEVICE_GET_CLASS(dev)->bus_type, > + object_get_typename(OBJECT(dev))); > + abort(); > } >
Since there is an errp, should we use it and be even kinder? Paolo