Re: [Qemu-devel] [PATCH for-1.2 v5 00/14] pci_host: Convert to QOM

["Michael S. Tsirkin" <mst@redhat.com>]

On Thu, Aug 02, 2012 at 03:46:52AM +0200, Andreas Färber wrote:
> Hello Anthony and Michael,
> 
> Here's a fixed version of the series making pci_host a first-class QOM type.
> 
> MAINTAINERS entries for the ppc devices touched herein are stripped from the
> series but being used for sending. They can be applied later through ppc-next.
> 
> This series is a prerequisite for the i440fx refactoring and q35 introduction.
> 
> I have verified this to apply cleanly to both master and pci branch now.
> 
> mst: Followups for i440fx by Wanpeng were on the list already, but feel free
> not to apply the slimmed-down final patch if we can then finally move on!
> 
> Regards,
> Andreas
> 
> Cc: Anthony Liguori <anthony@codemonkey.ws>
> Cc: Michael S. Tsirkin <mst@redhat.com>


ACK patches 1-13.

> 
> Cc: Wanpeng Li <liwanp@linux.vnet.ibm.com>
> Cc: Jason Baron <jbaron@redhat.com>
> Cc: Jan Kiszka <jan.kiszka@siemens.com>
> Cc: Alexander Graf <agraf@suse.de>
> Cc: Peter Maydell <peter.maydell@linaro.org>
> 
> v4 -> v5:
> * Resolved PCI_HOST_BRIDGE(s)->bus usage in alpha_typhoon (forgotten in v4).
> * Update Wanpeng's email address in SoB to avoid bounces.
> * Leave out MAINTAINERS additions, still valid but no ack from Alex yet and
>   open question from Peter on ordering.
> 
> v3 -> v4:
> * Moved spapr_pci.c:register_types() rename to later dedicated spapr_pci patch.
> * Added proposed patches for related MAINTAINERS machine entries w/ PHB files.
> Anthony requested the QOM cast discussion to be held elsewhere, so no change.
> 
> v2 -> v3:
> * Renamed PCI_HOST to PCI_HOST_BRIDGE, suggested by mst.
> * Split off const changes to clarify the name/parent changes.
> * Split off cleanups per host bridge.
> * Prepared and enforced QOM-style field access through explicit types.
> 
> v1 -> v2:
> * Converted remaining PCI host bridges to new type.
> 
> Andreas Färber (14):
>   pci: Make host bridge TypeInfos const
>   alpha_typhoon: QOM'ify Typhoon PCI host bridge
>   bonito: QOM'ify Bonito PCI host bridge
>   dec_pci: QOM'ify DEC 21154 PCI-PCI bridge
>   grackle_pci: QOM'ify Grackle PCI host bridge
>   gt64xxx: QOM'ify GT64120 PCI host bridge
>   ppc4xx_pci: QOM'ify ppc4xx PCI host bridge
>   ppce500_pci: QOM'ify e500 PCI host bridge
>   prep_pci: QOM'ify Raven PCI host bridge
>   spapr_pci: QOM'ify sPAPR PCI host bridge
>   unin_pci: QOM'ify UniNorth PCI host bridges
>   pci_host: Turn into SysBus-derived QOM type
>   pci: Derive PCI host bridges from TYPE_PCI_HOST_BRIDGE
>   pci: Tidy up PCI host bridges
> 
>  hw/alpha_typhoon.c |   26 +++++---
>  hw/bonito.c        |  152 ++++++++++++++++++++++++++-----------------
>  hw/dec_pci.c       |   31 +++++----
>  hw/dec_pci.h       |    2 +
>  hw/grackle_pci.c   |   65 +++++++++---------
>  hw/gt64xxx.c       |   67 +++++++++++--------
>  hw/pci_host.c      |   12 ++++
>  hw/pci_host.h      |    5 ++
>  hw/piix_pci.c      |   22 ++++---
>  hw/ppc440_bamboo.c |    3 +-
>  hw/ppc4xx.h        |    2 +
>  hw/ppc4xx_pci.c    |   27 +++++---
>  hw/ppc_mac.h       |    1 +
>  hw/ppc_prep.c      |    4 +-
>  hw/ppce500_pci.c   |   24 ++++---
>  hw/prep_pci.c      |   33 ++++++----
>  hw/spapr_pci.c     |   29 +++++----
>  hw/spapr_pci.h     |    8 ++-
>  hw/unin_pci.c      |  183 +++++++++++++++++++++++++++-------------------------
>  19 files changed, 402 insertions(+), 294 deletions(-)
> 
> -- 
> 1.7.7