[v2] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

[RFC PATCH v2 0/1] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

Zongyuan Li posted 1 patch 2 years, 1 month ago

Diff against v1 v3 v4 v5
Download series mbox

Test checkpatch passed

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

Maintainers: Peter Maydell <peter.maydell@linaro.org>

There is a newer version of this series

hw/arm/realview.c | 41 ++++++++++++++++++++++++++++++++---------
1 file changed, 32 insertions(+), 9 deletions(-)

Expand all Fold all

[RFC PATCH v2 0/1] Replace 'qemu_irq_split' with 'TYPE_SPLIT_IRQ'

Posted by Zongyuan Li 2 years, 1 month ago

This is a WIP patch set trying to resolve

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

which will totally replace 'qemu_irq_split' with use 
QOM object 'TYPE_SPLIT_IRQ' device.

I just want to make sure I'm on the right way, so
only 'realview' module is modified.

Changes since v1:

1. Remove unnecessary headers realview.c
2. Rewrite the splitter helper function. I agree that previous valist
   implementation is too complicated, but keeping it will make code
   more readable. Or any other reason we need to write the code
   directly into the init function? If that is true, I will change that
3. Use 'qdev_*' rather than 'Object' directly

Thanks for reviewing.

Zongyuan Li (1):
  hw/arm/realview: replace 'qemu_split_irq' with 'TYPE_SPLIT_IRQ'

 hw/arm/realview.c | 41 ++++++++++++++++++++++++++++++++---------
 1 file changed, 32 insertions(+), 9 deletions(-)

-- 
2.34.0