Re: [Qemu-devel] [PATCH qom-next for-next 0/5] QOM PCMCIA, MicroDrive and IDE cleanups

[Markus Armbruster]

Andreas Färber <address@hidden> writes:

> Hello,
>
> Based on an initial draft from Othmar Pasteka, this series QOM'ifies 
> MicroDrive.
> As a prerequisite, PCMCIA must be QOM'ified as pointed out by Peter. We can 
> then
> finally drop ide_init2_with_non_qdev_drives(), as desired by Markus for his
> IDE refactorings. As a final touch, pxa2xx_pcmcia is made a SysBusDevice.

Full IDE qdevification achieved in just under four years (assuming we
get this committed before September 15).  I'm tickled pink.

Thanks a lot for making this happen!  To Othmar, too, of course.

> A for-1.6 patch to fix an assertion while inspecting the QOM composition tree
> was sent out separately.
>
> QOM'ifying the PXA2xx SoCs to replace pxa2xx_pcmcia_init() and other helpers
> is left as follow-up.
>
> We probably also want to move "info pcmcia" HMP out of vl.c but I'm not sure
> in which form we want to keep PCMCIASocket? A SysBusDevice subtype would seem
> easiest for our current use case, but there are also PCI-CardBus bridges IIRC.
> PCMCIASocket::card_string should probably be a property of PCMCIACardState,
> accessed through a link<pcmcia-card> property.
>
> Markus, unfortunately git am --3way didn't work for your old IDE patches.
> Please rebase them on this series yourself, thanks.

Sure!