[v4] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

[PATCH v4 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 v5
Download series mbox

Test checkpatch failed

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

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

There is a newer version of this series

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 | 79 +++++++++++++++++++++++++++--------
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, 161 insertions(+), 64 deletions(-)

Expand all Fold all

[PATCH v4 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.

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 | 79 +++++++++++++++++++++++++++--------
 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, 161 insertions(+), 64 deletions(-)

-- 
2.34.0

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

Posted by Zongyuan Li 2 years, 1 month ago

This version seems to have some code style errors. I will fix that soon.
Sorry for the bother.

Li

On Fri, Mar 25, 2022 at 1:18 AM 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.
>
> 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 | 79 +++++++++++++++++++++++++++--------
>  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, 161 insertions(+), 64 deletions(-)
>
> --
> 2.34.0
>