On Mon, 18 May 2020 at 06:04, Markus Armbruster <arm...@redhat.com> wrote: > > These devices are optional, and enabled by property "enable-bitband". > armv7m_instance_init() creates them unconditionally, because the > property has not been set then. armv7m_realize() realizes them only > when the property is true. Works, although it leaves unrealized > devices hanging around in the QOM composition tree. Affects machines > microbit, mps2-an505, mps2-an521, musca-a, and musca-b1. > > Bury the unwanted devices by making armv7m_realize() unparent them. > Visible in "info qom-tree"; here's the change for microbit: > > /machine (microbit-machine) > /microbit.twi (microbit.i2c) > /microbit.twi[0] (qemu:memory-region) > /nrf51 (nrf51-soc) > /armv6m (armv7m) > /armv7m-container[0] (qemu:memory-region) > - /bitband[0] (ARM,bitband-memory) > - /bitband[0] (qemu:memory-region) > - /bitband[1] (ARM,bitband-memory) > - /bitband[0] (qemu:memory-region) > /cpu (cortex-m0-arm-cpu)
What does "bury" mean here? To me it implies "they still exist but we've stuck them in a hole somewhere and covered them up", but the qom-tree delta suggests we've actually really deleted them? thanks -- PMM