[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 05/34] tests/qgraph: sdhci driver and interfa
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v2 05/34] tests/qgraph: sdhci driver and interface nodes |
Date: |
Thu, 9 Aug 2018 15:58:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 09/08/2018 15:27, Emanuele wrote:
>
>>> +{
>>> + QSDHCI_PCI *spci = (QSDHCI_PCI *)obj;
>>> + qpci_device_enable(&spci->dev);
>>> +}
>>> +
>>> +static void sdhci_destroy(QOSGraphObject *obj)
>> "sdhci_pci_destructor()"?
> In general, I called the constructor name *_create, so in theory to
> keep it consistent the destructor should be *_destroy, no?
Yes, but the field name in QOSGraphObject is "destructor".
You have:
obj.get_device = XXX_get_device;
obj.get_driver = XXX_get_driver;
obj.start_hw = XXX_start_hw;
So we can expect:
obj.destructor = XXX_destructor;
Thanks,
Laurent
[Qemu-devel] [PATCH v2 06/34] tests/qgraph: sdhci test node, Emanuele Giuseppe Esposito, 2018/08/06
[Qemu-devel] [PATCH v2 08/34] tests/qgraph: rename qpci_init_spapr functions, Emanuele Giuseppe Esposito, 2018/08/06
[Qemu-devel] [PATCH v2 07/34] tests/qgraph: arm/raspi2 machine node, Emanuele Giuseppe Esposito, 2018/08/06
[Qemu-devel] [PATCH v2 10/34] tests/qgraph: ppc64/pseries machine node, Emanuele Giuseppe Esposito, 2018/08/06
[Qemu-devel] [PATCH v2 09/34] tests/qgraph: pci-spapr driver and interface nodes, Emanuele Giuseppe Esposito, 2018/08/06