On 06/28/2016 06:21 PM, Cornelia Huck wrote:
On Tue, 28 Jun 2016 17:49:18 +0300
Marcel Apfelbaum <mar...@redhat.com> wrote:

On 06/24/2016 04:28 PM, Cornelia Huck wrote:
From: Yi Min Zhao <zyi...@linux.vnet.ibm.com>

+static const TypeInfo s390_pci_device_info = {
+    .name = TYPE_S390_PCI_DEVICE,
+    .parent = TYPE_DEVICE,

Hi,

Here we have the same mismatch.  TYPE_S390_PCI_DEVICE does not derive
from TYPE_PCI_DEVICE. If I understand correctly the "zpci device"
is a wrapper to the actual PCI device, maybe the name can reflect it.

TYPE_S390PCI_DEVICE?


The same as on prev discussion, maybe TYPE_ZPCI_DEVICE, but is not *that* 
important.

Thanks,
Marcel

Reply via email to