Peter Maydell <peter.mayd...@linaro.org> writes: > On Thu, 23 May 2019 at 13:09, Markus Armbruster <arm...@redhat.com> wrote: >> Registering qbus_reset_all_fn() in main() is kind of ugly, but it works. >> There's a comment pointing out it's ugly. Right now it's a TODO >> comment, which maybe expresses more hope for cleanup than there really >> is. I'd leave it alone anyway. > > ...so after this long thread, are we at a conclusion? > I think my view is that the patch-as-sent needs to be > revised to also fix bus_unparent() but is otherwise OK.
Yup. You can add my Reviewed-by: Markus Armbruster <arm...@redhat.com> to it then
