qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: Eduardo Habkost <ehabkost@redhat.com>, qemu-devel@nongnu.org
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Andrew Jeffery" <andrew@aj.id.au>,
	"Jason Wang" <jasowang@redhat.com>,
	"Bin Meng" <bin.meng@windriver.com>,
	"Vijai Kumar K" <vijai@behindbytes.com>,
	"Taylor Simpson" <tsimpson@quicinc.com>,
	"Alistair Francis" <Alistair.Francis@wdc.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Klaus Jensen" <its@irrelevant.dk>,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
	"Vikram Garhwal" <fnu.vikram@xilinx.com>,
	qemu-block@nongnu.org, qemu-ppc@nongnu.org,
	"David Hildenbrand" <david@redhat.com>,
	"Havard Skinnemoen" <hskinnemoen@google.com>,
	"Halil Pasic" <pasic@linux.ibm.com>,
	"Christian Borntraeger" <borntraeger@de.ibm.com>,
	"Joel Stanley" <joel@jms.id.au>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	qemu-s390x@nongnu.org,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Greg Kurz" <groug@kaod.org>, "Alexander Bulekov" <alxndr@bu.edu>,
	"Bandan Das" <bsd@redhat.com>,
	qemu-arm@nongnu.org, "Cédric Le Goater" <clg@kaod.org>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Keith Busch" <kbusch@kernel.org>,
	qemu-riscv@nongnu.org,
	"David Gibson" <david@gibson.dropbear.id.au>,
	"Daniel P. Berrange" <berrange@redhat.com>,
	"Francisco Iglesias" <francisco.iglesias@xilinx.com>,
	"Thomas Huth" <huth@tuxfamily.org>,
	"Patrick Venture" <venture@google.com>,
	"Laurent Vivier" <laurent@vivier.eu>,
	"Tyrone Ting" <kfting@nuvoton.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Bastian Koppelmann" <kbastian@mail.uni-paderborn.de>,
	"Igor Mammedov" <imammedo@redhat.com>
Subject: Re: [PATCH for-6.2 07/12] [automated] Use DECLARE_*CHECKER* macros when possible
Date: Tue, 10 Aug 2021 08:09:55 +0200	[thread overview]
Message-ID: <871r71ygng.fsf@redhat.com> (raw)
In-Reply-To: <20210806211127.646908-8-ehabkost@redhat.com>

On Fri, Aug 06 2021, Eduardo Habkost <ehabkost@redhat.com> wrote:

> Converting existing QOM types to OBJECT_DECLARE_TYPE is not
> always trivial (due to inconsistent type/macro naming schemes),
> but at least converting existing manual QOM type checking macros
> to use DECLARE_*CHECKER* is a simpler process, and should at
> least discourage people from defining new QOM type checker macros
> manually.
>
> Generated using:
>
>   $ ./scripts/codeconverter/converter.py -i \
>     --pattern=TypeCheckMacro $(git grep -l '' -- '*.[ch]')
>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Patrick Venture <venture@google.com>
> Cc: Thomas Huth <huth@tuxfamily.org>
> Cc: "Michael S. Tsirkin" <mst@redhat.com>
> Cc: Igor Mammedov <imammedo@redhat.com>
> Cc: Alexander Bulekov <alxndr@bu.edu>
> Cc: Bandan Das <bsd@redhat.com>
> Cc: Stefan Hajnoczi <stefanha@redhat.com>
> Cc: Keith Busch <kbusch@kernel.org>
> Cc: Klaus Jensen <its@irrelevant.dk>
> Cc: Cornelia Huck <cohuck@redhat.com>
> Cc: Halil Pasic <pasic@linux.ibm.com>
> Cc: Christian Borntraeger <borntraeger@de.ibm.com>
> Cc: Richard Henderson <richard.henderson@linaro.org>
> Cc: David Hildenbrand <david@redhat.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Havard Skinnemoen <hskinnemoen@google.com>
> Cc: Tyrone Ting <kfting@nuvoton.com>
> Cc: Vijai Kumar K <vijai@behindbytes.com>
> Cc: Alistair Francis <Alistair.Francis@wdc.com>
> Cc: Bin Meng <bin.meng@windriver.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
> Cc: Peter Maydell <peter.maydell@linaro.org>
> Cc: Laurent Vivier <laurent@vivier.eu>
> Cc: "Cédric Le Goater" <clg@kaod.org>
> Cc: Andrew Jeffery <andrew@aj.id.au>
> Cc: Joel Stanley <joel@jms.id.au>
> Cc: Jason Wang <jasowang@redhat.com>
> Cc: Vikram Garhwal <fnu.vikram@xilinx.com>
> Cc: Francisco Iglesias <francisco.iglesias@xilinx.com>
> Cc: David Gibson <david@gibson.dropbear.id.au>
> Cc: Greg Kurz <groug@kaod.org>
> Cc: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
> Cc: Taylor Simpson <tsimpson@quicinc.com>
> Cc: qemu-devel@nongnu.org
> Cc: qemu-block@nongnu.org
> Cc: qemu-s390x@nongnu.org
> Cc: qemu-arm@nongnu.org
> Cc: qemu-riscv@nongnu.org
> Cc: qemu-ppc@nongnu.org
> ---
>  hw/nvme/nvme.h                              | 12 ++++++------
>  hw/usb/hcd-xhci-pci.h                       |  4 ++--
>  hw/usb/hcd-xhci-sysbus.h                    |  4 ++--
>  hw/usb/u2f.h                                |  8 ++------
>  include/hw/adc/npcm7xx_adc.h                |  4 ++--
>  include/hw/arm/npcm7xx.h                    | 15 ++++-----------
>  include/hw/char/shakti_uart.h               |  4 ++--
>  include/hw/dma/sifive_pdma.h                |  4 ++--
>  include/hw/dma/xlnx_csu_dma.h               |  4 ++--
>  include/hw/gpio/npcm7xx_gpio.h              |  4 ++--
>  include/hw/i2c/npcm7xx_smbus.h              |  4 ++--
>  include/hw/intc/m68k_irqc.h                 |  4 ++--
>  include/hw/intc/sifive_clint.h              |  4 ++--
>  include/hw/mem/npcm7xx_mc.h                 |  3 ++-
>  include/hw/misc/aspeed_lpc.h                |  3 ++-
>  include/hw/misc/mchp_pfsoc_dmc.h            | 10 ++++------
>  include/hw/misc/mchp_pfsoc_ioscb.h          |  4 ++--
>  include/hw/misc/mchp_pfsoc_sysreg.h         |  5 ++---
>  include/hw/misc/npcm7xx_clk.h               |  3 ++-
>  include/hw/misc/npcm7xx_gcr.h               |  3 ++-
>  include/hw/misc/npcm7xx_mft.h               |  4 ++--
>  include/hw/misc/npcm7xx_pwm.h               |  4 ++--
>  include/hw/misc/npcm7xx_rng.h               |  3 ++-
>  include/hw/misc/xlnx-versal-xramc.h         |  4 ++--
>  include/hw/net/npcm7xx_emc.h                |  4 ++--
>  include/hw/net/xlnx-zynqmp-can.h            |  4 ++--
>  include/hw/nvram/npcm7xx_otp.h              |  3 ++-
>  include/hw/ppc/spapr_drc.h                  | 13 ++++---------
>  include/hw/ppc/spapr_xive.h                 |  7 ++-----
>  include/hw/riscv/microchip_pfsoc.h          |  9 ++++-----
>  include/hw/riscv/shakti_c.h                 |  8 ++++----
>  include/hw/riscv/sifive_e.h                 |  8 ++++----
>  include/hw/riscv/sifive_u.h                 |  8 ++++----
>  include/hw/sd/cadence_sdhci.h               |  4 ++--
>  include/hw/ssi/npcm7xx_fiu.h                |  3 ++-
>  include/hw/ssi/sifive_spi.h                 |  3 ++-
>  include/hw/timer/npcm7xx_timer.h            |  4 ++--
>  include/hw/tricore/tricore_testdevice.h     |  4 ++--
>  include/hw/usb/hcd-dwc3.h                   |  4 ++--
>  include/hw/usb/xlnx-usb-subsystem.h         |  4 ++--
>  include/hw/usb/xlnx-versal-usb2-ctrl-regs.h |  4 ++--
>  include/hw/watchdog/sbsa_gwdt.h             |  4 ++--
>  include/qemu/accel.h                        |  8 ++------
>  target/hexagon/cpu.h                        |  8 ++------
>  chardev/char-parallel.c                     |  6 ++----
>  hw/i2c/i2c_mux_pca954x.c                    |  4 ++--
>  hw/m68k/mcf5206.c                           |  3 ++-
>  hw/mem/sparse-mem.c                         |  3 ++-
>  hw/misc/sbsa_ec.c                           |  3 ++-
>  hw/s390x/vhost-user-fs-ccw.c                |  4 ++--
>  hw/sensor/adm1272.c                         |  3 ++-
>  hw/sensor/max34451.c                        |  3 ++-
>  hw/usb/u2f-emulated.c                       |  4 ++--
>  hw/usb/u2f-passthru.c                       |  4 ++--
>  54 files changed, 126 insertions(+), 146 deletions(-)

Acked-by: Cornelia Huck <cohuck@redhat.com>



  reply	other threads:[~2021-08-10  6:11 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-06 21:11 [PATCH for-6.2 00/12] qom: Get rid of all manual usage of OBJECT_CHECK & friends Eduardo Habkost
2021-08-06 21:11 ` [PATCH for-6.2 01/12] accel: Rename TYPE_ACCEL to TYPE_ACCEL_BASE Eduardo Habkost
2021-08-06 21:11 ` [PATCH for-6.2 02/12] qom: Use DEVICE_*CLASS instead of OBJECT_*CLASS Eduardo Habkost
2021-08-07  8:17   ` Philippe Mathieu-Daudé
2021-08-10 11:56   ` Juan Quintela
2021-08-10 14:22     ` Eduardo Habkost
2021-08-06 21:11 ` [PATCH for-6.2 03/12] scripts/codeconverter: Update to latest version Eduardo Habkost
2021-08-06 21:11 ` [PATCH for-6.2 04/12] [automated] Add struct names to typedefs used by QOM types Eduardo Habkost
2021-08-07  8:03   ` Philippe Mathieu-Daudé
2021-08-06 21:11 ` [PATCH for-6.2 05/12] [automated] Move QOM typedefs and add missing includes Eduardo Habkost
2021-08-07  8:13   ` Philippe Mathieu-Daudé
2021-08-10  6:06   ` Cornelia Huck
2021-08-10 12:01   ` Juan Quintela
2021-08-10 13:06     ` Eduardo Habkost
2021-08-06 21:11 ` [PATCH for-6.2 06/12] [automated] Split QOM "typedef struct T { ... } T" declarations Eduardo Habkost
2021-08-10  6:08   ` Cornelia Huck
2021-08-06 21:11 ` [PATCH for-6.2 07/12] [automated] Use DECLARE_*CHECKER* macros when possible Eduardo Habkost
2021-08-10  6:09   ` Cornelia Huck [this message]
2021-08-06 21:11 ` [PATCH for-6.2 08/12] npcm7xx_clk: Use DECLARE_INSTANCE_CHECKER Eduardo Habkost
2021-08-07  7:59   ` Philippe Mathieu-Daudé
2021-08-06 21:11 ` [PATCH for-6.2 09/12] npcm7xx_otp: Use DECLARE_CLASS_CHECKERS Eduardo Habkost
2021-08-07  8:00   ` Philippe Mathieu-Daudé
2021-08-06 21:11 ` [PATCH for-6.2 10/12] [automated] Use DECLARE_OBJ_CHECKERS when possible Eduardo Habkost
2021-08-06 21:11 ` [PATCH for-6.2 11/12] [automated] Use OBJECT_DECLARE_TYPE " Eduardo Habkost
2021-08-07  8:02   ` Philippe Mathieu-Daudé
2021-08-06 21:11 ` [PATCH for-6.2 12/12] [automated] Use OBJECT_DECLARE_SIMPLE_TYPE " Eduardo Habkost
2021-08-07  8:09   ` Philippe Mathieu-Daudé
2021-08-10  6:12   ` Cornelia Huck
2021-08-07  8:15 ` [PATCH for-6.2 00/12] qom: Get rid of all manual usage of OBJECT_CHECK & friends Philippe Mathieu-Daudé
2021-08-09 18:00   ` Eduardo Habkost

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=871r71ygng.fsf@redhat.com \
    --to=cohuck@redhat.com \
    --cc=Alistair.Francis@wdc.com \
    --cc=alxndr@bu.edu \
    --cc=andrew@aj.id.au \
    --cc=berrange@redhat.com \
    --cc=bin.meng@windriver.com \
    --cc=borntraeger@de.ibm.com \
    --cc=bsd@redhat.com \
    --cc=clg@kaod.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=david@redhat.com \
    --cc=edgar.iglesias@gmail.com \
    --cc=ehabkost@redhat.com \
    --cc=fnu.vikram@xilinx.com \
    --cc=francisco.iglesias@xilinx.com \
    --cc=groug@kaod.org \
    --cc=hskinnemoen@google.com \
    --cc=huth@tuxfamily.org \
    --cc=imammedo@redhat.com \
    --cc=its@irrelevant.dk \
    --cc=jasowang@redhat.com \
    --cc=joel@jms.id.au \
    --cc=kbastian@mail.uni-paderborn.de \
    --cc=kbusch@kernel.org \
    --cc=kfting@nuvoton.com \
    --cc=kraxel@redhat.com \
    --cc=laurent@vivier.eu \
    --cc=marcandre.lureau@redhat.com \
    --cc=mst@redhat.com \
    --cc=palmer@dabbelt.com \
    --cc=pasic@linux.ibm.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=qemu-riscv@nongnu.org \
    --cc=qemu-s390x@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=stefanha@redhat.com \
    --cc=tsimpson@quicinc.com \
    --cc=venture@google.com \
    --cc=vijai@behindbytes.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).