From: "Philippe Mathieu-Daudé" <philmd@linaro.org>
To: qemu-devel@nongnu.org
Cc: "Mark Cave-Ayland" <mark.cave-ayland@ilande.co.uk>,
"Thomas Huth" <thuth@redhat.com>,
"Markus Armbruster" <armbru@redhat.com>,
qemu-ppc@nongnu.org, "Eduardo Habkost" <eduardo@habkost.net>,
qemu-arm@nongnu.org,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>
Subject: [PATCH v2 0/7] hw/sysbus: Add sysbus_init_irqs and reduce SYSBUS_DEVICE_GPIO_IRQ scope
Date: Thu, 1 Jun 2023 11:46:18 +0200 [thread overview]
Message-ID: <20230601094625.39569-1-philmd@linaro.org> (raw)
Since v1:
- Removed pointless QOM casts on a preliminary series [*] (Markus)
- Added Richard R-b tags
This series:
- Remove uses (out of sysbus.c) to the SYSBUS_DEVICE_GPIO_IRQ
definition, using proper SysBus API methods,
- Reduce SYSBUS_DEVICE_GPIO_IRQ scope, making it SysBus API
internal,
- Convert various for() loops iterating over sysbus_init_irq()
by calling a single sysbus_init_irqs() instead. Mostly an
automatic convertion except 4 files.
The resulting code looks nicer IMHO, and is possibly less
bug prone.
Based-on: <20230601093452.38972-1-philmd@linaro.org>
"bulk: Remove pointless QOM casts"
[*] https://lore.kernel.org/qemu-devel/20230601093452.38972-1-philmd@linaro.org/
Philippe Mathieu-Daudé (7):
hw/arm/xlnx-versal: Do not open-code sysbus_connect_irq()
hw/usb/xlnx: Do not open-code sysbus_pass_irq()
hw/sysbus: Introduce sysbus_init_irqs()
hw/usb/hcd-xhci: Use sysbus_init_irqs()
hw/sysbus: Make SYSBUS_DEVICE_GPIO_IRQ API internal
hw: Simplify using sysbus_init_irqs() [automatic]
hw: Simplify using sysbus_init_irqs() [manual]
include/hw/sysbus.h | 3 +--
hw/arm/smmuv3.c | 6 +-----
hw/arm/stellaris.c | 5 +----
hw/arm/strongarm.c | 5 +----
hw/arm/versatilepb.c | 5 +----
hw/arm/xlnx-versal.c | 4 +---
hw/char/pl011.c | 5 +----
hw/char/renesas_sci.c | 5 +----
hw/core/platform-bus.c | 5 +----
hw/core/sysbus.c | 9 ++++++++-
hw/dma/pl330.c | 4 +---
hw/dma/sifive_pdma.c | 5 +----
hw/gpio/sifive_gpio.c | 5 +----
hw/i386/kvm/xen_evtchn.c | 5 +----
hw/intc/arm_gic_common.c | 20 +++++---------------
hw/intc/arm_gicv2m.c | 5 +----
hw/intc/exynos4210_combiner.c | 5 +----
hw/intc/loongarch_extioi.c | 7 ++-----
hw/intc/loongson_liointc.c | 5 +----
hw/intc/omap_intc.c | 3 +--
hw/intc/openpic.c | 6 ++----
hw/intc/slavio_intctl.c | 6 ++----
hw/misc/avr_power.c | 4 +---
hw/misc/macio/gpio.c | 5 +----
hw/misc/stm32f4xx_exti.c | 5 +----
hw/net/cadence_gem.c | 5 +----
hw/net/mcf_fec.c | 5 +----
hw/pci-host/designware.c | 5 +----
hw/pci-host/gpex.c | 2 +-
hw/pci-host/ppce500.c | 4 +---
hw/pci-host/raven.c | 4 +---
hw/pci-host/sh_pci.c | 5 +----
hw/pci-host/versatile.c | 4 +---
hw/ppc/ppc405_uc.c | 10 ++--------
hw/ppc/ppc440_uc.c | 6 ++----
hw/ppc/ppc4xx_devs.c | 4 +---
hw/ppc/ppc4xx_pci.c | 5 +----
hw/ssi/ibex_spi_host.c | 5 +----
hw/ssi/imx_spi.c | 5 +----
hw/ssi/sifive_spi.c | 5 +----
hw/ssi/xilinx_spi.c | 5 +----
hw/ssi/xilinx_spips.c | 4 +---
hw/ssi/xlnx-versal-ospi.c | 4 +---
hw/timer/allwinner-a10-pit.c | 4 +---
hw/timer/exynos4210_mct.c | 4 +---
hw/timer/hpet.c | 4 +---
hw/timer/renesas_cmt.c | 5 +----
hw/timer/renesas_tmr.c | 9 +++------
hw/timer/sifive_pwm.c | 5 +----
hw/usb/hcd-xhci-sysbus.c | 5 ++---
hw/usb/xlnx-usb-subsystem.c | 2 +-
51 files changed, 70 insertions(+), 197 deletions(-)
--
2.38.1
next reply other threads:[~2023-06-01 9:48 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-01 9:46 Philippe Mathieu-Daudé [this message]
2023-06-01 9:46 ` [PATCH v2 1/7] hw/arm/xlnx-versal: Do not open-code sysbus_connect_irq() Philippe Mathieu-Daudé
2023-06-01 9:46 ` [PATCH v2 2/7] hw/usb/xlnx: Do not open-code sysbus_pass_irq() Philippe Mathieu-Daudé
2023-06-01 9:46 ` [PATCH v2 3/7] hw/sysbus: Introduce sysbus_init_irqs() Philippe Mathieu-Daudé
2023-06-01 9:46 ` [PATCH v2 4/7] hw/usb/hcd-xhci: Use sysbus_init_irqs() Philippe Mathieu-Daudé
2023-06-01 9:46 ` [PATCH v2 5/7] hw/sysbus: Make SYSBUS_DEVICE_GPIO_IRQ API internal Philippe Mathieu-Daudé
2023-06-01 9:46 ` [PATCH v2 6/7] hw: Simplify using sysbus_init_irqs() [automatic] Philippe Mathieu-Daudé
2023-06-01 9:46 ` [PATCH v2 7/7] hw: Simplify using sysbus_init_irqs() [manual] Philippe Mathieu-Daudé
2023-06-01 10:19 ` Peter Maydell
2023-06-01 10:34 ` Philippe Mathieu-Daudé
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=20230601094625.39569-1-philmd@linaro.org \
--to=philmd@linaro.org \
--cc=armbru@redhat.com \
--cc=eduardo@habkost.net \
--cc=marcandre.lureau@redhat.com \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=thuth@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.