mbox series

[00/63] qom: Rename macros for consistency

Message ID 20200902224311.1321159-1-ehabkost@redhat.com
Headers show
Series qom: Rename macros for consistency | expand

Message

Eduardo Habkost Sept. 2, 2020, 10:42 p.m. UTC
Reducing boilerplate QOM code using the new OBJECT_DEFINE_* and
OBJECT_DECLARE_* macros is quite difficult when there are
multiple ways a TYPE_* constant name is written.

This series renames many type checking macros and/or TYPE_*
constants to make sure they are consistent.

This series is based on machine-next and can be fetched from:

  https://github.com/ehabkost/qemu-hacks work/qom-rename-macros

Eduardo Habkost (63):
  gpex: Fix type checking function name
  chardev: Rename TYPE_CHARDEV_* to TYPE_*_CHARDEV
  chardev: Rename TYPE_BRAILLE_CHARDEV to TYPE_BAUM_CHARDEV
  chardev: Rename TYPE_MSMOUSE_CHARDEV to TYPE_MOUSE_CHARDEV
  ap-device: Rename AP_DEVICE_TYPE to TYPE_AP_DEVICE
  dev-smartcard-reader: Rename CCID_DEV_NAME to TYPE_USB_CCID_DEV
  rdma: Rename PVRDMA_HW_NAME to TYPE_PVRDMA_DEV
  vfio: Rename VFIO_AP_DEVICE_TYPE to TYPE_VFIO_AP_DEVICE
  vmcoreinfo: Rename VMCOREINFO_DEVICE to TYPE_VMCOREINFO
  vmgenid: Rename VMGENID_DEVICE to TYPE_VMGENID
  edu: Rename TYPE_PCI_EDU_DEVICE to TYPE_EDU
  etrax: Rename TYPE_ETRAX_FS_SERIAL to TYPE_ETRAX_SERIAL
  etrax: Rename TYPE_ETRAX_FS_TIMER to TYPE_ETRAX_TIMER
  i8254: Rename TYPE_I8254 to TYPE_PIT
  i8259: Rename TYPE_I8259 to TYPE_PIC
  i8259: Rename TYPE_KVM_I8259 to TYPE_KVM_PIC
  intel-hda: Rename TYPE_INTEL_HDA_GENERIC to TYPE_INTEL_HDA
  mips: Rename TYPE_MIPS_BOSTON to TYPE_BOSTON
  mptsas: Rename TYPE_MPTSAS1068 to TYPE_MPT_SAS
  pc87312: Rename TYPE_PC87312_SUPERIO to TYPE_PC87312
  pxb: Rename TYPE_PXB_DEVICE to TYPE_PXB_DEV
  pxb: Rename TYPE_PXB_PCIE_DEVICE to TYPE_PXB_PCIE_DEV
  ramfb: Rename TYPE_RAMFB_DEVICE to TYPE_RAMFB
  scsi: Rename TYPE_AM53C974_DEVICE to TYPE_PCI_ESP
  scsi: Rename TYPE_DC390_DEVICE to TYPE_DC390
  sun4m: Rename TYPE_SUN4M_MEMORY to TYPE_SUN4M_RAM
  sun4u: Rename TYPE_SUN4U_MEMORY to TYPE_SUN4U_RAM
  tusb6010: Rename TYPE_TUSB6010 to TYPE_TUSB
  versatile: Rename TYPE_VERSATILE_PCI_HOST to TYPE_PCI_VPB_HOST
  versatile: Rename TYPE_VERSATILE_PCI to TYPE_PCI_VPB
  virtio-console: Rename TYPE_VIRTIO_CONSOLE_SERIAL_PORT to
    TYPE_VIRTIO_CONSOLE
  ahci: Rename ICH_AHCI to ICH9_AHCI
  net/can: Rename TYPE_CAN_CPI constants
  can: Rename KVASER_PCI_DEV to KVASER_PCI
  can: Rename MIOe3680_PCI_DEV to MIOE3680_PCI
  can: Rename PCM3680i_PCI_DEV to PCM3680_PCI
  ccid-card-emulated: Rename EMULATED_CCID_CARD to EMULATED_CCID
  ccid-card-passthru: Rename PASSTHRU_CCID_CARD to CCID_PASSTHRU
  esp: Rename ESP_STATE to ESP
  filter-rewriter: Rename FILTER_COLO_REWRITER to FILTER_REWRITER
  lance: Rename SYSBUS_PCNET to LANCE
  lasi_i82596: Rename SYSBUS_I82596 to LASI_82596
  pvpanic: Rename ISA_PVPANIC_DEVICE to PVPANIC
  renesas: Rename RCMT to RENESAS_CMT
  renesas: Rename RSCI to RENESAS_SCI
  renesas: Rename RTMR to RENESAS_TMR
  rs6000_mc: Rename RS6000MC_DEVICE to RS6000MC
  sabre: Rename SABRE_DEVICE to SABRE
  stm32f205: Rename STM32F2XXTIMER to STM32F2XX_TIMER
  tpm_crb: Rename CRB to TPM_CRB
  tpm_spapr: Rename VIO_SPAPR_VTPM to TPM_SPAPR
  usb: Rename USB_REDIRECT to USB_REDIR
  usb: Rename USB_SERIAL_DEV to USB_SERIAL
  usb: Rename USB_STORAGE_DEV to USB_STORAGE
  vfio: Rename PCI_VFIO to VFIO_PCI
  vmxnet3: Rename VMXNET3_DEVICE* to VMXNET3*
  xen: Rename XENBACKEND_DEVICE to XENBACKEND
  hvf: Rename HVF_STATE to HVF_ACCEL
  kvm: Rename KVM_STATE to KVM_ACCEL
  tcg: Rename TCG_STATE to TCG_ACCEL
  tests: Rename TYPE_DUMMY to TYPE_DUMMY_OBJECT
  tests: Rename DYNAMIC_TYPE to DYNAMIC_PROPS
  tests: Rename STATIC_TYPE to STATIC_PROPS

 chardev/chardev-internal.h          |  4 ++--
 hw/rdma/vmw/pvrdma.h                |  4 ++--
 hw/scsi/mptsas.h                    |  4 ++--
 hw/vfio/pci.h                       |  2 +-
 include/chardev/char-fd.h           |  4 ++--
 include/chardev/char-win-stdio.h    |  2 +-
 include/chardev/char-win.h          |  4 ++--
 include/chardev/char.h              | 30 ++++++++++++++---------------
 include/chardev/spice.h             |  8 ++++----
 include/hw/acpi/vmgenid.h           |  6 +++---
 include/hw/char/renesas_sci.h       |  2 +-
 include/hw/display/ramfb.h          |  2 +-
 include/hw/ide/ahci.h               |  2 +-
 include/hw/isa/pc87312.h            |  4 ++--
 include/hw/misc/vmcoreinfo.h        |  6 +++---
 include/hw/net/lance.h              |  2 +-
 include/hw/net/lasi_82596.h         |  2 +-
 include/hw/pci-host/gpex.h          |  2 +-
 include/hw/pci-host/sabre.h         |  2 +-
 include/hw/s390x/ap-device.h        |  4 ++--
 include/hw/scsi/esp.h               |  2 +-
 include/hw/timer/i8254.h            |  4 ++--
 include/hw/timer/renesas_cmt.h      |  2 +-
 include/hw/timer/renesas_tmr.h      |  6 +++---
 include/hw/timer/stm32f2xx_timer.h  |  2 +-
 include/hw/xen/xen-legacy-backend.h |  2 +-
 include/sysemu/hvf.h                |  2 +-
 include/sysemu/kvm.h                |  2 +-
 accel/kvm/kvm-all.c                 | 18 ++++++++---------
 accel/tcg/tcg-all.c                 | 14 +++++++-------
 chardev/baum.c                      |  6 +++---
 chardev/char-console.c              |  4 ++--
 chardev/char-fd.c                   |  2 +-
 chardev/char-file.c                 |  6 +++---
 chardev/char-mux.c                  |  2 +-
 chardev/char-null.c                 |  2 +-
 chardev/char-parallel.c             |  4 ++--
 chardev/char-pipe.c                 |  6 +++---
 chardev/char-pty.c                  |  4 ++--
 chardev/char-ringbuf.c              |  8 ++++----
 chardev/char-serial.c               |  6 +++---
 chardev/char-socket.c               |  4 ++--
 chardev/char-stdio.c                |  6 +++---
 chardev/char-udp.c                  |  4 ++--
 chardev/char-win-stdio.c            |  4 ++--
 chardev/char-win.c                  |  2 +-
 chardev/char.c                      |  2 +-
 chardev/msmouse.c                   |  6 +++---
 chardev/spice.c                     | 10 +++++-----
 chardev/testdev.c                   |  6 +++---
 chardev/wctablet.c                  |  6 +++---
 gdbstub.c                           |  6 +++---
 hw/acpi/vmgenid.c                   |  6 +++---
 hw/arm/sysbus-fdt.c                 |  2 +-
 hw/arm/virt.c                       |  2 +-
 hw/audio/intel-hda.c                | 10 +++++-----
 hw/char/etraxfs_ser.c               |  6 +++---
 hw/char/renesas_sci.c               | 18 ++++++++---------
 hw/char/virtio-console.c            |  8 ++++----
 hw/display/ramfb-standalone.c       |  4 ++--
 hw/display/vhost-user-gpu.c         |  2 +-
 hw/dma/sparc32_dma.c                |  2 +-
 hw/i386/kvm/i8259.c                 | 10 +++++-----
 hw/i386/pc_piix.c                   |  2 +-
 hw/i386/pc_q35.c                    |  2 +-
 hw/ide/ahci.c                       |  4 ++--
 hw/ide/ich.c                        |  8 ++++----
 hw/intc/i8259.c                     | 10 +++++-----
 hw/isa/pc87312.c                    |  2 +-
 hw/m68k/q800.c                      |  2 +-
 hw/mips/boston.c                    |  8 ++++----
 hw/mips/jazz.c                      |  2 +-
 hw/misc/edu.c                       |  6 +++---
 hw/misc/pvpanic.c                   |  6 +++---
 hw/misc/vmcoreinfo.c                |  6 +++---
 hw/net/can/can_kvaser_pci.c         | 16 +++++++--------
 hw/net/can/can_mioe3680_pci.c       | 16 +++++++--------
 hw/net/can/can_pcm3680_pci.c        | 16 +++++++--------
 hw/net/lance.c                      |  6 +++---
 hw/net/lasi_i82596.c                |  8 ++++----
 hw/net/vmxnet3.c                    |  6 +++---
 hw/pci-bridge/pci_expander_bridge.c | 12 ++++++------
 hw/pci-host/sabre.c                 |  8 ++++----
 hw/pci-host/versatile.c             | 16 +++++++--------
 hw/ppc/rs6000_mc.c                  |  4 ++--
 hw/rdma/vmw/pvrdma_main.c           |  2 +-
 hw/rx/rx62n.c                       |  2 +-
 hw/s390x/ap-device.c                |  2 +-
 hw/scsi/esp-pci.c                   | 14 +++++++-------
 hw/scsi/esp.c                       |  6 +++---
 hw/scsi/mptsas.c                    |  2 +-
 hw/sparc/sun4m.c                    | 10 +++++-----
 hw/sparc64/sun4u.c                  | 10 +++++-----
 hw/timer/etraxfs_timer.c            |  6 +++---
 hw/timer/i8254.c                    |  4 ++--
 hw/timer/renesas_cmt.c              |  4 ++--
 hw/timer/renesas_tmr.c              |  6 +++---
 hw/timer/stm32f2xx_timer.c          |  6 +++---
 hw/tpm/tpm_crb.c                    | 14 +++++++-------
 hw/tpm/tpm_spapr.c                  | 14 +++++++-------
 hw/usb/ccid-card-emulated.c         | 10 +++++-----
 hw/usb/ccid-card-passthru.c         |  8 ++++----
 hw/usb/dev-serial.c                 |  4 ++--
 hw/usb/dev-smartcard-reader.c       |  8 ++++----
 hw/usb/dev-storage.c                | 12 ++++++------
 hw/usb/redirect.c                   | 22 ++++++++++-----------
 hw/usb/tusb6010.c                   |  6 +++---
 hw/vfio/ap.c                        | 10 +++++-----
 hw/vfio/pci.c                       | 22 ++++++++++-----------
 net/filter-rewriter.c               | 18 ++++++++---------
 target/arm/kvm.c                    |  2 +-
 target/i386/kvm.c                   |  2 +-
 target/mips/kvm.c                   |  2 +-
 target/ppc/kvm.c                    |  4 ++--
 tests/check-qom-proplist.c          | 30 ++++++++++++++---------------
 tests/test-char.c                   |  4 ++--
 tests/test-io-task.c                | 14 +++++++-------
 tests/test-qdev-global-props.c      | 16 +++++++--------
 ui/console.c                        |  8 ++++----
 ui/gtk.c                            |  6 +++---
 ui/spice-app.c                      | 10 +++++-----
 121 files changed, 406 insertions(+), 406 deletions(-)

-- 
2.26.2

Comments

Thomas Huth Sept. 3, 2020, 11:35 a.m. UTC | #1
On 03/09/2020 00.42, Eduardo Habkost wrote:
> This will make the type name constant consistent with the name of
> the type checking macro.
> 
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: Cornelia Huck <cohuck@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
> Cc: Tony Krowiak <akrowiak@linux.ibm.com>
> Cc: Halil Pasic <pasic@linux.ibm.com>
> Cc: Pierre Morel <pmorel@linux.ibm.com>
> Cc: Alex Williamson <alex.williamson@redhat.com>
> Cc: qemu-s390x@nongnu.org
> Cc: qemu-devel@nongnu.org
> ---
>  hw/vfio/ap.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/hw/vfio/ap.c b/hw/vfio/ap.c
> index 68ed059b39..582c091a24 100644
> --- a/hw/vfio/ap.c
> +++ b/hw/vfio/ap.c
> @@ -30,7 +30,7 @@
>  #include "exec/address-spaces.h"
>  #include "qom/object.h"
>  
> -#define VFIO_AP_DEVICE_TYPE      "vfio-ap"
> +#define TYPE_VFIO_AP_DEVICE      "vfio-ap"
>  
>  struct VFIOAPDevice {
>      APDevice apdev;
> @@ -39,7 +39,7 @@ struct VFIOAPDevice {
>  typedef struct VFIOAPDevice VFIOAPDevice;
>  
>  DECLARE_INSTANCE_CHECKER(VFIOAPDevice, VFIO_AP_DEVICE,
> -                         VFIO_AP_DEVICE_TYPE)
> +                         TYPE_VFIO_AP_DEVICE)
>  
>  static void vfio_ap_compute_needs_reset(VFIODevice *vdev)
>  {
> @@ -72,7 +72,7 @@ static VFIOGroup *vfio_ap_get_group(VFIOAPDevice *vapdev, Error **errp)
>  
>      if (!group_path) {
>          error_setg(errp, "%s: no iommu_group found for %s: %s",
> -                   VFIO_AP_DEVICE_TYPE, vapdev->vdev.sysfsdev, gerror->message);
> +                   TYPE_VFIO_AP_DEVICE, vapdev->vdev.sysfsdev, gerror->message);
>          g_error_free(gerror);
>          return NULL;
>      }
> @@ -176,7 +176,7 @@ static void vfio_ap_class_init(ObjectClass *klass, void *data)
>  }
>  
>  static const TypeInfo vfio_ap_info = {
> -    .name = VFIO_AP_DEVICE_TYPE,
> +    .name = TYPE_VFIO_AP_DEVICE,
>      .parent = TYPE_AP_DEVICE,
>      .instance_size = sizeof(VFIOAPDevice),
>      .class_init = vfio_ap_class_init,
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>
Philippe Mathieu-Daudé Sept. 3, 2020, 12:48 p.m. UTC | #2
On 9/3/20 12:42 AM, Eduardo Habkost wrote:
> This looks like a copy/paste mistake: the instance type checking
> macro for TYPE_GPEX_ROOT_DEVICE was named MCH_PCI_DEVICE.
> 
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: qemu-devel@nongnu.org
> ---
>  include/hw/pci-host/gpex.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/hw/pci-host/gpex.h b/include/hw/pci-host/gpex.h
> index e8432eced8..2f4e852eee 100644
> --- a/include/hw/pci-host/gpex.h
> +++ b/include/hw/pci-host/gpex.h
> @@ -32,7 +32,7 @@ DECLARE_INSTANCE_CHECKER(GPEXHost, GPEX_HOST,
>  
>  #define TYPE_GPEX_ROOT_DEVICE "gpex-root"
>  typedef struct GPEXRootState GPEXRootState;
> -DECLARE_INSTANCE_CHECKER(GPEXRootState, MCH_PCI_DEVICE,
> +DECLARE_INSTANCE_CHECKER(GPEXRootState, GPEX_ROOT_DEVICE,
>                           TYPE_GPEX_ROOT_DEVICE)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

>  
>  #define GPEX_NUM_IRQS 4
>
Philippe Mathieu-Daudé Sept. 3, 2020, 12:53 p.m. UTC | #3
On 9/3/20 12:42 AM, Eduardo Habkost wrote:
> This will make the type name constant consistent with the name of
> the type checking macro.
> 
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: "Michael S. Tsirkin" <mst@redhat.com>
> Cc: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Richard Henderson <rth@twiddle.net>
> Cc: Eduardo Habkost <ehabkost@redhat.com>
> Cc: qemu-devel@nongnu.org
> ---
>  hw/i386/kvm/i8259.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/hw/i386/kvm/i8259.c b/hw/i386/kvm/i8259.c
> index 3f8bf69e9c..687c0cd536 100644
> --- a/hw/i386/kvm/i8259.c
> +++ b/hw/i386/kvm/i8259.c
> @@ -19,10 +19,10 @@
>  #include "sysemu/kvm.h"
>  #include "qom/object.h"
>  
> -#define TYPE_KVM_I8259 "kvm-i8259"
> +#define TYPE_KVM_PIC "kvm-i8259"

I disagree with this patch, as we have various KVM INTC and only one
KVM_I8259.

TYPE_KVM_ARM_GIC and TYPE_KVM_S390_FLIC are kind of TYPE_KVM_INTC ...

Can we rename it KVM_I8259_PIC?

>  typedef struct KVMPICClass KVMPICClass;
>  DECLARE_CLASS_CHECKERS(KVMPICClass, KVM_PIC,
> -                       TYPE_KVM_I8259)
> +                       TYPE_KVM_PIC)
>  
>  /**
>   * KVMPICClass:
> @@ -133,8 +133,8 @@ static void kvm_pic_realize(DeviceState *dev, Error **errp)
>  
>  qemu_irq *kvm_i8259_init(ISABus *bus)
>  {
> -    i8259_init_chip(TYPE_KVM_I8259, bus, true);
> -    i8259_init_chip(TYPE_KVM_I8259, bus, false);
> +    i8259_init_chip(TYPE_KVM_PIC, bus, true);
> +    i8259_init_chip(TYPE_KVM_PIC, bus, false);
>  
>      return qemu_allocate_irqs(kvm_pic_set_irq, NULL, ISA_NUM_IRQS);
>  }
> @@ -152,7 +152,7 @@ static void kvm_i8259_class_init(ObjectClass *klass, void *data)
>  }
>  
>  static const TypeInfo kvm_i8259_info = {
> -    .name = TYPE_KVM_I8259,
> +    .name = TYPE_KVM_PIC,
>      .parent = TYPE_PIC_COMMON,
>      .instance_size = sizeof(PICCommonState),
>      .class_init = kvm_i8259_class_init,
>
Philippe Mathieu-Daudé Sept. 3, 2020, 12:59 p.m. UTC | #4
On 9/3/20 12:42 AM, Eduardo Habkost wrote:
> Rename macros to names that are unique in the source tree, to
> reduce confusion and make automated conversion of the code
> easier.
> 
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: Jason Wang <jasowang@redhat.com>
> Cc: qemu-devel@nongnu.org
> ---
>  hw/net/can/can_kvaser_pci.c   | 6 +++---
>  hw/net/can/can_mioe3680_pci.c | 6 +++---
>  hw/net/can/can_pcm3680_pci.c  | 6 +++---
>  3 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/hw/net/can/can_kvaser_pci.c b/hw/net/can/can_kvaser_pci.c
> index 168b3a620d..12d4f8df33 100644
> --- a/hw/net/can/can_kvaser_pci.c
> +++ b/hw/net/can/can_kvaser_pci.c
> @@ -45,11 +45,11 @@
>  #include "can_sja1000.h"
>  #include "qom/object.h"
>  
> -#define TYPE_CAN_PCI_DEV "kvaser_pci"
> +#define TYPE_KVASER_PCI "kvaser_pci"

Why keep "PCI" and not keep the "CAN" part?
To be consistent we should use both busses or none.

[...]
Eduardo Habkost Sept. 3, 2020, 4:36 p.m. UTC | #5
On Thu, Sep 03, 2020 at 03:01:33PM +0200, Philippe Mathieu-Daudé wrote:
> On 9/3/20 12:42 AM, Eduardo Habkost wrote:
> > Make the type checking macro name consistent with the TYPE_*
> > constant.
> > 
> > Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> > ---
> > Cc: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> > Cc: Laurent Vivier <laurent@vivier.eu>
> > Cc: "Hervé Poussineau" <hpoussin@reactos.org>
> > Cc: Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>
> > Cc: Aleksandar Markovic <aleksandar.qemu.devel@gmail.com>
> > Cc: Aurelien Jarno <aurelien@aurel32.net>
> > Cc: Jiaxun Yang <jiaxun.yang@flygoat.com>
> > Cc: Paolo Bonzini <pbonzini@redhat.com>
> > Cc: Fam Zheng <fam@euphon.net>
> > Cc: Artyom Tarasenko <atar4qemu@gmail.com>
> > Cc: qemu-devel@nongnu.org
> > ---
> >  include/hw/scsi/esp.h | 2 +-
> >  hw/dma/sparc32_dma.c  | 2 +-
> >  hw/m68k/q800.c        | 2 +-
> >  hw/mips/jazz.c        | 2 +-
> >  hw/scsi/esp.c         | 6 +++---
> >  hw/sparc/sun4m.c      | 2 +-
> >  6 files changed, 8 insertions(+), 8 deletions(-)
> > 
> > diff --git a/include/hw/scsi/esp.h b/include/hw/scsi/esp.h
> > index f281625dd5..20800dbf5b 100644
> > --- a/include/hw/scsi/esp.h
> > +++ b/include/hw/scsi/esp.h
> > @@ -67,7 +67,7 @@ struct ESPState {
> >  
> >  #define TYPE_ESP "esp"
> >  typedef struct SysBusESPState SysBusESPState;
> > -DECLARE_INSTANCE_CHECKER(SysBusESPState, ESP_STATE,
> > +DECLARE_INSTANCE_CHECKER(SysBusESPState, ESP,
> >                           TYPE_ESP)
> 
> "ESPDMA" would less likely clash with eventual 'ESP' definition.

I want to make the changes less intrusive as possible.  If the
macro is already called ESP and the type name string is already
"esp", I will just keep the existing naming.
Eduardo Habkost Sept. 4, 2020, 4:12 p.m. UTC | #6
On Wed, Sep 02, 2020 at 06:42:54PM -0400, Eduardo Habkost wrote:
> This will make the type name constant consistent with the name of
> the type checking macro.
> 
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
> Cc: Magnus Damm <magnus.damm@gmail.com>
> Cc: qemu-devel@nongnu.org
> ---
[...]
> -#define TYPE_RENESAS_TMR "renesas-tmr"
> +#define TYPE_RTMR "renesas-tmr"
>  typedef struct RTMRState RTMRState;
> -DECLARE_INSTANCE_CHECKER(RTMRState, RTMR,
> -                         TYPE_RENESAS_TMR)
> +DECLARE_INSTANCE_CHECKER(RTMRState, RENESAS_TMR,
> +                         TYPE_RTMR)

Please ignore this patch.  I've made a mistake while rebasing the
series and accidentally squashed in a commit that renamed
TYPE_RENESAS_TMR to TYPE_RTMR.
Cornelia Huck Sept. 7, 2020, 7:29 a.m. UTC | #7
On Wed,  2 Sep 2020 18:42:16 -0400
Eduardo Habkost <ehabkost@redhat.com> wrote:

> This will make the type name constant consistent with the name of
> the type checking macro.
> 
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: Cornelia Huck <cohuck@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
> Cc: Tony Krowiak <akrowiak@linux.ibm.com>
> Cc: Halil Pasic <pasic@linux.ibm.com>
> Cc: Pierre Morel <pmorel@linux.ibm.com>
> Cc: Alex Williamson <alex.williamson@redhat.com>
> Cc: qemu-s390x@nongnu.org
> Cc: qemu-devel@nongnu.org
> ---
>  hw/vfio/ap.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)

Acked-by: Cornelia Huck <cohuck@redhat.com>
Xingtao Yao (Fujitsu)" via Sept. 28, 2020, 4:15 p.m. UTC | #8
On Wed, Sep 02, 2020 at 06:43:05PM -0400, Eduardo Habkost wrote:
> Make the type checking macro name consistent with the TYPE_*

> constant.

> 

> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>


Acked-by: Anthony PERARD <anthony.perard@citrix.com>


Thanks,

-- 
Anthony PERARD
Philippe Mathieu-Daudé Oct. 9, 2020, 3:25 p.m. UTC | #9
On 9/3/20 12:42 AM, Eduardo Habkost wrote:
> This will make the type name constant consistent with the name of

> the type checking macro.

> 

> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>

> ---

> Cc: Paul Burton <paulburton@kernel.org>

> Cc: Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>

> Cc: Aleksandar Markovic <aleksandar.qemu.devel@gmail.com>

> Cc: Aurelien Jarno <aurelien@aurel32.net>

> Cc: Jiaxun Yang <jiaxun.yang@flygoat.com>

> Cc: qemu-devel@nongnu.org

> ---

>   hw/mips/boston.c | 8 ++++----

>   1 file changed, 4 insertions(+), 4 deletions(-)


Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


And queued to mips-next, thanks.

Phil.