[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 28/74] s390x: Move typedef SCLPEventFacility to event-faci
From: |
Thomas Huth |
Subject: |
Re: [PATCH v3 28/74] s390x: Move typedef SCLPEventFacility to event-facility.h |
Date: |
Thu, 27 Aug 2020 10:07:21 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0 |
On 25/08/2020 21.20, Eduardo Habkost wrote:
> This will make future conversion to OBJECT_DECLARE* easier.
>
> In sclp.h, use "struct SCLPEventFacility" to avoid introducing
> unnecessary header dependencies.
>
> Acked-by: Cornelia Huck <cohuck@redhat.com>
> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Changes v2 -> v3: none
>
> Changes series v1 -> v2: new patch in series v2
>
> Cc: Cornelia Huck <cohuck@redhat.com>
> Cc: Halil Pasic <pasic@linux.ibm.com>
> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: qemu-s390x@nongnu.org
> Cc: qemu-devel@nongnu.org
> ---
> include/hw/s390x/event-facility.h | 1 +
> include/hw/s390x/sclp.h | 4 ++--
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/hw/s390x/event-facility.h
> b/include/hw/s390x/event-facility.h
> index 700a610f33..e61c4651d7 100644
> --- a/include/hw/s390x/event-facility.h
> +++ b/include/hw/s390x/event-facility.h
> @@ -195,6 +195,7 @@ typedef struct SCLPEventClass {
> } SCLPEventClass;
>
> #define TYPE_SCLP_EVENT_FACILITY "s390-sclp-event-facility"
> +typedef struct SCLPEventFacility SCLPEventFacility;
> #define EVENT_FACILITY(obj) \
> OBJECT_CHECK(SCLPEventFacility, (obj), TYPE_SCLP_EVENT_FACILITY)
> #define EVENT_FACILITY_CLASS(klass) \
> diff --git a/include/hw/s390x/sclp.h b/include/hw/s390x/sclp.h
> index 822eff4396..a87ed2a0ab 100644
> --- a/include/hw/s390x/sclp.h
> +++ b/include/hw/s390x/sclp.h
> @@ -185,12 +185,12 @@ typedef struct SCCB {
> #define SCLP_CLASS(oc) OBJECT_CLASS_CHECK(SCLPDeviceClass, (oc), TYPE_SCLP)
> #define SCLP_GET_CLASS(obj) OBJECT_GET_CLASS(SCLPDeviceClass, (obj),
> TYPE_SCLP)
>
> -typedef struct SCLPEventFacility SCLPEventFacility;
> +struct SCLPEventFacility;
>
> typedef struct SCLPDevice {
> /* private */
> DeviceState parent_obj;
> - SCLPEventFacility *event_facility;
> + struct SCLPEventFacility *event_facility;
> int increment_size;
>
> /* public */
>
Reviewed-by: Thomas Huth <thuth@redhat.com>
- [PATCH v3 17/74] pci: Move PCIBusClass typedef to pci.h, (continued)
- [PATCH v3 17/74] pci: Move PCIBusClass typedef to pci.h, Eduardo Habkost, 2020/08/25
- [PATCH v3 18/74] i8254: Move PITCommonState/PITCommonClass typedefs to i8254.h, Eduardo Habkost, 2020/08/25
- [PATCH v3 19/74] hvf: Move HVFState typedef to hvf.h, Eduardo Habkost, 2020/08/25
- [PATCH v3 20/74] mcf_fec: Move mcf_fec_state typedef to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 21/74] s390_flic: Move KVMS390FLICState typedef to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 22/74] can_emu: Delete macros for non-existing typedef, Eduardo Habkost, 2020/08/25
- [PATCH v3 24/74] platform-bus: Delete macros for non-existing typedef, Eduardo Habkost, 2020/08/25
- [PATCH v3 26/74] xen-legacy-backend: Add missing typedef XenLegacyDevice, Eduardo Habkost, 2020/08/25
- [PATCH v3 28/74] s390x: Move typedef SCLPEventFacility to event-facility.h, Eduardo Habkost, 2020/08/25
- Re: [PATCH v3 28/74] s390x: Move typedef SCLPEventFacility to event-facility.h,
Thomas Huth <=
- [PATCH v3 27/74] spapr: Move typedef SpaprMachineState to spapr.h, Eduardo Habkost, 2020/08/25
- [PATCH v3 25/74] armsse: Rename QOM macros to avoid conflicts, Eduardo Habkost, 2020/08/25
- [PATCH v3 23/74] nubus: Delete unused NUBUS_BRIDGE macro, Eduardo Habkost, 2020/08/25
- [PATCH v3 29/74] vhost-user-gpu: Move QOM macro to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 30/74] ahci: Move QOM macros to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 31/74] i8257: Move QOM macro to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 32/74] ahci: Move QOM macro to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 33/74] pckbd: Move QOM macro to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 34/74] vmbus: Move QOM macros to vmbus.h, Eduardo Habkost, 2020/08/25
- [PATCH v3 35/74] virtio-serial-bus: Move QOM macros to header, Eduardo Habkost, 2020/08/25