[v5] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

[PATCH v5 0/4] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

Zongyuan Li posted 4 patches 2 years, 1 month ago

Diff against v1 v2 v3 v4
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220324181557.203805-1-zongyuan.li@smartx.com

Maintainers: Igor Mitsyanko <i.mitsyanko@gmail.com>, Peter Maydell <peter.maydell@linaro.org>

hw/arm/exynos4210.c           | 26 +++++++++++
hw/arm/realview.c             | 33 ++++++++++----
hw/arm/stellaris.c            | 15 ++++++-
hw/core/irq.c                 | 15 -------
hw/intc/exynos4210_combiner.c | 81 +++++++++++++++++++++++++++--------
hw/intc/exynos4210_gic.c      | 36 +++++++++++++---
include/hw/arm/exynos4210.h   | 11 ++---
include/hw/core/split-irq.h   |  5 +--
include/hw/irq.h              |  5 ---
9 files changed, 163 insertions(+), 64 deletions(-)

Expand all Fold all

[PATCH v5 0/4] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

Posted by Zongyuan Li 2 years, 1 month ago

This patch set tries to replace 'qemu_irq_split' function with QOM
object 'TYPE_SPLIT_IRQ' and totally remove this call.

If this patch set is applied, issue:

https://gitlab.com/qemu-project/qemu/-/issues/811

can be closed.

Changes since v3:

1. Squash Patch 3 & 4 into one, since they would affect each other.
2. Use `object_initialize_with_child` and `qdev_realize` for device code.
3. Code style fixes.
4. Remove unnecessary `if` statement used with `qdev_realize_and_unref.
5. Narrow scope of some variables.

Changes since v4

1. Code style fixes.

Zongyuan Li (4):
  hw/arm/realview: replace 'qemu_split_irq' with 'TYPE_SPLIT_IRQ'
  hw/arm/stellaris: replace 'qemu_split_irq' with 'TYPE_SPLIT_IRQ'
  hw/intc/exynos4210: replace 'qemu_split_irq' in combiner and gic
  hw/core/irq: remove unused 'qemu_irq_split' function

 hw/arm/exynos4210.c           | 26 +++++++++++
 hw/arm/realview.c             | 33 ++++++++++----
 hw/arm/stellaris.c            | 15 ++++++-
 hw/core/irq.c                 | 15 -------
 hw/intc/exynos4210_combiner.c | 81 +++++++++++++++++++++++++++--------
 hw/intc/exynos4210_gic.c      | 36 +++++++++++++---
 include/hw/arm/exynos4210.h   | 11 ++---
 include/hw/core/split-irq.h   |  5 +--
 include/hw/irq.h              |  5 ---
 9 files changed, 163 insertions(+), 64 deletions(-)

-- 
2.34.0

Re: [PATCH v5 0/4] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

Posted by Peter Maydell 2 years ago

On Thu, 24 Mar 2022 at 18:22, Zongyuan Li <zongyuan.li@smartx.com> wrote:
>
> This patch set tries to replace 'qemu_irq_split' function with QOM
> object 'TYPE_SPLIT_IRQ' and totally remove this call.
>
> If this patch set is applied, issue:
>
> https://gitlab.com/qemu-project/qemu/-/issues/811
>
> can be closed.
>
> Changes since v3:
>
> 1. Squash Patch 3 & 4 into one, since they would affect each other.
> 2. Use `object_initialize_with_child` and `qdev_realize` for device code.
> 3. Code style fixes.
> 4. Remove unnecessary `if` statement used with `qdev_realize_and_unref.
> 5. Narrow scope of some variables.
>
> Changes since v4
>
> 1. Code style fixes.
>
> Zongyuan Li (4):
>   hw/arm/realview: replace 'qemu_split_irq' with 'TYPE_SPLIT_IRQ'
>   hw/arm/stellaris: replace 'qemu_split_irq' with 'TYPE_SPLIT_IRQ'
>   hw/intc/exynos4210: replace 'qemu_split_irq' in combiner and gic
>   hw/core/irq: remove unused 'qemu_irq_split' function

I've applied patches 1, 2 and 4 to target-arm.next, thanks
(after my exynos4210 cleanup series).

-- PMM