On Tue, 17 Nov 2020 at 05:41, Markus Armbruster <arm...@redhat.com> wrote: > > Peter Maydell <peter.mayd...@linaro.org> writes: > > I'm really dubious of devices that directly inherit from > > TYPE_DEVICE, because their reset method won't be automatically > > called. In this case it looks like the TYPE_BCM2835_CPRMAN > > device's reset method manually calls reset on these devices, > > though, so it isn't actually buggy, just confusing. > > I guess this is a trap for unwary implementers, aggravated by our usual > dearth of qdev documentation. I can see ~100 .parent = TYPE_DEVICE > lines. Checking them all manually won't be fun. Any automation ideas? > > The concept "bus-less device" is sane. We used not to have it, and the > resulting need for pseudo-busses was annoying enough to make us add > bus-less devices.
Yeah, the problem really is that our reset handling remains a mess. I'm not sure (a) what the right model for reset is and (b) what would be a feasible transition plan to get from here to there... thanks -- PMM
