qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	Jason Baron <jbaron@redhat.com>,
	qemu-devel@nongnu.org, Alexander Graf <agraf@suse.de>,
	anthony@codemonkey.ws, Wanpeng Li <liwanp@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH for-1.2 v5 00/14] pci_host: Convert to QOM
Date: Thu, 09 Aug 2012 17:09:50 +0200	[thread overview]
Message-ID: <5023D2BE.2030002@suse.de> (raw)
In-Reply-To: <20120802083053.GB24929@redhat.com>

Am 02.08.2012 10:30, schrieb Michael S. Tsirkin:
> 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.

Ping! I don't see these on the pci branch. Should they be picked up by
Anthony directly?

Thanks,
Andreas

> 
>>
>> 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
> 


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  reply	other threads:[~2012-08-09 15:10 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-02  1:46 [Qemu-devel] [PATCH for-1.2 v5 00/14] pci_host: Convert to QOM Andreas Färber
2012-08-02  1:46 ` [Qemu-devel] [PATCH for-1.2 v5 01/14] pci: Make host bridge TypeInfos const Andreas Färber
2012-08-02  1:46 ` [Qemu-devel] [PATCH for-1.2 v5 02/14] alpha_typhoon: QOM'ify Typhoon PCI host bridge Andreas Färber
2012-08-02  1:46 ` [Qemu-devel] [PATCH for-1.2 v5 03/14] bonito: QOM'ify Bonito " Andreas Färber
2012-08-02  1:46 ` [Qemu-devel] [PATCH for-1.2 v5 04/14] dec_pci: QOM'ify DEC 21154 PCI-PCI bridge Andreas Färber
2012-08-02  1:46 ` [Qemu-devel] [PATCH for-1.2 v5 05/14] grackle_pci: QOM'ify Grackle PCI host bridge Andreas Färber
2012-08-02  1:46 ` [Qemu-devel] [PATCH for-1.2 v5 06/14] gt64xxx: QOM'ify GT64120 " Andreas Färber
2012-08-02  1:46 ` [Qemu-devel] [PATCH for-1.2 v5 07/14] ppc4xx_pci: QOM'ify ppc4xx " Andreas Färber
2012-08-02  1:47 ` [Qemu-devel] [PATCH for-1.2 v5 08/14] ppce500_pci: QOM'ify e500 " Andreas Färber
2012-08-02  1:47 ` [Qemu-devel] [PATCH for-1.2 v5 09/14] prep_pci: QOM'ify Raven " Andreas Färber
2012-08-02  1:47 ` [Qemu-devel] [PATCH for-1.2 v5 10/14] spapr_pci: QOM'ify sPAPR " Andreas Färber
2012-08-02  1:47 ` [Qemu-devel] [PATCH for-1.2 v5 11/14] unin_pci: QOM'ify UniNorth PCI host bridges Andreas Färber
2012-08-02  1:47 ` [Qemu-devel] [PATCH for-1.2 v5 12/14] pci_host: Turn into SysBus-derived QOM type Andreas Färber
2012-08-02  1:47 ` [Qemu-devel] [PATCH for-1.2 v5 13/14] pci: Derive PCI host bridges from TYPE_PCI_HOST_BRIDGE Andreas Färber
2012-08-02  1:47 ` [Qemu-devel] [PATCH for-1.2 v5 14/14] pci: Tidy up PCI host bridges Andreas Färber
2012-08-02  8:30   ` Michael S. Tsirkin
2012-08-13 13:14   ` Anthony Liguori
2012-08-13 13:32     ` Andreas Färber
2012-08-13 13:16   ` Michael S. Tsirkin
2012-08-13 14:16     ` Anthony Liguori
2012-08-13 18:57       ` Mark Cave-Ayland
2012-08-02  8:30 ` [Qemu-devel] [PATCH for-1.2 v5 00/14] pci_host: Convert to QOM Michael S. Tsirkin
2012-08-09 15:09   ` Andreas Färber [this message]
2012-08-13 12:27     ` Andreas Färber

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5023D2BE.2030002@suse.de \
    --to=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=anthony@codemonkey.ws \
    --cc=jan.kiszka@siemens.com \
    --cc=jbaron@redhat.com \
    --cc=liwanp@linux.vnet.ibm.com \
    --cc=mst@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).