|
From: | Paolo Bonzini |
Subject: | Re: [Qemu-arm] [PATCH] hw/arm/bcm283x: Fix crash with device_add bcm2837 on unsupported machines |
Date: | Wed, 11 Jul 2018 22:16:42 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 11/07/2018 20:30, Eduardo Habkost wrote: >> The theoretical behavior should be: > It's not clear below where you expect > qdev_set_parent_bus(..., sysbus_get_default()) > to be called (if it should be called at all). > > I don't know where it should be called, but I'm absolutely sure > instance_init is not the right place. I think instance_init is fine to call qdev_set_parent_bus on contained devices. Why do you say it's not? Thanks, Paolo
[Prev in Thread] | Current Thread | [Next in Thread] |