[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 21/74] s390_flic: Move KVMS390FLICState typedef to header
From: |
Thomas Huth |
Subject: |
Re: [PATCH v3 21/74] s390_flic: Move KVMS390FLICState typedef to header |
Date: |
Thu, 27 Aug 2020 10:08:09 +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:
> Move typedef closer to the type check macros, to make it easier
> to convert the code to OBJECT_DEFINE_TYPE() in the future.
>
> 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 v1 -> v2: none
>
> ---
> Cc: Cornelia Huck <cohuck@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Halil Pasic <pasic@linux.ibm.com>
> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
> Cc: qemu-s390x@nongnu.org
> Cc: qemu-devel@nongnu.org
> ---
> include/hw/s390x/s390_flic.h | 1 +
> hw/intc/s390_flic_kvm.c | 4 ++--
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/hw/s390x/s390_flic.h b/include/hw/s390x/s390_flic.h
> index 4687ecfe83..df11de9b20 100644
> --- a/include/hw/s390x/s390_flic.h
> +++ b/include/hw/s390x/s390_flic.h
> @@ -75,6 +75,7 @@ typedef struct S390FLICStateClass {
> } S390FLICStateClass;
>
> #define TYPE_KVM_S390_FLIC "s390-flic-kvm"
> +typedef struct KVMS390FLICState KVMS390FLICState;
> #define KVM_S390_FLIC(obj) \
> OBJECT_CHECK(KVMS390FLICState, (obj), TYPE_KVM_S390_FLIC)
>
> diff --git a/hw/intc/s390_flic_kvm.c b/hw/intc/s390_flic_kvm.c
> index a306b26faa..dbd4e682ce 100644
> --- a/hw/intc/s390_flic_kvm.c
> +++ b/hw/intc/s390_flic_kvm.c
> @@ -29,12 +29,12 @@
> #define FLIC_FAILED (-1UL)
> #define FLIC_SAVEVM_VERSION 1
>
> -typedef struct KVMS390FLICState {
> +struct KVMS390FLICState{
> S390FLICState parent_obj;
>
> uint32_t fd;
> bool clear_io_supported;
> -} KVMS390FLICState;
> +};
>
> static KVMS390FLICState *s390_get_kvm_flic(S390FLICState *fs)
> {
>
Reviewed-by: Thomas Huth <thuth@redhat.com>
- [PATCH v3 12/74] virtio-ccw: Fix definition of VIRTIO_CCW_BUS_GET_CLASS, (continued)
- [PATCH v3 12/74] virtio-ccw: Fix definition of VIRTIO_CCW_BUS_GET_CLASS, Eduardo Habkost, 2020/08/25
- [PATCH v3 14/74] hcd-dwc2: Rename USB_*CLASS macros for consistency, Eduardo Habkost, 2020/08/25
- [PATCH v3 16/74] throttle-groups: Move ThrottleGroup typedef to header, Eduardo Habkost, 2020/08/25
- [PATCH v3 15/74] tulip: Move TulipState typedef to header, Eduardo Habkost, 2020/08/25
- [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
- Re: [PATCH v3 21/74] s390_flic: Move KVMS390FLICState typedef to header,
Thomas Huth <=
- [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
- [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