mbox series

[0/4] hw: QOM housekeeping around IOTHREAD / IRQ types

Message ID 20230113200138.52869-1-philmd@linaro.org
Headers show
Series hw: QOM housekeeping around IOTHREAD / IRQ types | expand

Message

Philippe Mathieu-Daudé Jan. 13, 2023, 8:01 p.m. UTC 
- remove unused code
- use recent QOM macros
- use CamelCase typedef

Philippe Mathieu-Daudé (4):
  iothread: Remove unused IOThreadClass / IOTHREAD_CLASS
  hw/irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
  hw/or-irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
  hw: Replace qemu_or_irq typedef by OrIRQState

 hw/arm/exynos4210.c                  |  4 ++--
 hw/arm/mps2-tz.c                     |  2 +-
 hw/core/irq.c                        |  9 ++++-----
 hw/core/or-irq.c                     | 18 +++++++++---------
 hw/pci-host/raven.c                  |  2 +-
 include/hw/arm/armsse.h              |  6 +++---
 include/hw/arm/bcm2835_peripherals.h |  2 +-
 include/hw/arm/exynos4210.h          |  4 ++--
 include/hw/arm/stm32f205_soc.h       |  2 +-
 include/hw/arm/stm32f405_soc.h       |  2 +-
 include/hw/arm/xlnx-versal.h         |  6 +++---
 include/hw/arm/xlnx-zynqmp.h         |  2 +-
 include/hw/or-irq.h                  |  5 +----
 iothread.c                           |  4 ----
 14 files changed, 30 insertions(+), 38 deletions(-)

Comments

Stefan Hajnoczi Jan. 17, 2023, 4:15 p.m. UTC | #1 
On Fri, Jan 13, 2023 at 09:01:34PM +0100, Philippe Mathieu-Daudé wrote:
> - remove unused code
> - use recent QOM macros
> - use CamelCase typedef
> 
> Philippe Mathieu-Daudé (4):
>   iothread: Remove unused IOThreadClass / IOTHREAD_CLASS
>   hw/irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
>   hw/or-irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
>   hw: Replace qemu_or_irq typedef by OrIRQState
> 
>  hw/arm/exynos4210.c                  |  4 ++--
>  hw/arm/mps2-tz.c                     |  2 +-
>  hw/core/irq.c                        |  9 ++++-----
>  hw/core/or-irq.c                     | 18 +++++++++---------
>  hw/pci-host/raven.c                  |  2 +-
>  include/hw/arm/armsse.h              |  6 +++---
>  include/hw/arm/bcm2835_peripherals.h |  2 +-
>  include/hw/arm/exynos4210.h          |  4 ++--
>  include/hw/arm/stm32f205_soc.h       |  2 +-
>  include/hw/arm/stm32f405_soc.h       |  2 +-
>  include/hw/arm/xlnx-versal.h         |  6 +++---
>  include/hw/arm/xlnx-zynqmp.h         |  2 +-
>  include/hw/or-irq.h                  |  5 +----
>  iothread.c                           |  4 ----
>  14 files changed, 30 insertions(+), 38 deletions(-)
> 
> -- 
> 2.38.1
> 

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Philippe Mathieu-Daudé Feb. 7, 2023, 9:17 a.m. UTC | #2 
Hi Peter,

On 13/1/23 21:01, Philippe Mathieu-Daudé wrote:
> - remove unused code
> - use recent QOM macros
> - use CamelCase typedef
> 
> Philippe Mathieu-Daudé (4):
>    iothread: Remove unused IOThreadClass / IOTHREAD_CLASS
>    hw/irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
>    hw/or-irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
>    hw: Replace qemu_or_irq typedef by OrIRQState
> 
>   hw/arm/exynos4210.c                  |  4 ++--
>   hw/arm/mps2-tz.c                     |  2 +-
>   hw/core/irq.c                        |  9 ++++-----
>   hw/core/or-irq.c                     | 18 +++++++++---------
>   hw/pci-host/raven.c                  |  2 +-
>   include/hw/arm/armsse.h              |  6 +++---
>   include/hw/arm/bcm2835_peripherals.h |  2 +-
>   include/hw/arm/exynos4210.h          |  4 ++--
>   include/hw/arm/stm32f205_soc.h       |  2 +-
>   include/hw/arm/stm32f405_soc.h       |  2 +-
>   include/hw/arm/xlnx-versal.h         |  6 +++---
>   include/hw/arm/xlnx-zynqmp.h         |  2 +-
>   include/hw/or-irq.h                  |  5 +----
>   iothread.c                           |  4 ----
>   14 files changed, 30 insertions(+), 38 deletions(-)

Since most changes concern ARM machines, can this (reviewed)
series go via your tree?

Thanks,

Phil.
Peter Maydell Feb. 23, 2023, 1:36 p.m. UTC | #3 
On Tue, 7 Feb 2023 at 09:17, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> Hi Peter,
>
> On 13/1/23 21:01, Philippe Mathieu-Daudé wrote:
> > - remove unused code
> > - use recent QOM macros
> > - use CamelCase typedef
> >
> > Philippe Mathieu-Daudé (4):
> >    iothread: Remove unused IOThreadClass / IOTHREAD_CLASS
> >    hw/irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
> >    hw/or-irq: Declare QOM macros using OBJECT_DECLARE_SIMPLE_TYPE()
> >    hw: Replace qemu_or_irq typedef by OrIRQState
> >
> >   hw/arm/exynos4210.c                  |  4 ++--
> >   hw/arm/mps2-tz.c                     |  2 +-
> >   hw/core/irq.c                        |  9 ++++-----
> >   hw/core/or-irq.c                     | 18 +++++++++---------
> >   hw/pci-host/raven.c                  |  2 +-
> >   include/hw/arm/armsse.h              |  6 +++---
> >   include/hw/arm/bcm2835_peripherals.h |  2 +-
> >   include/hw/arm/exynos4210.h          |  4 ++--
> >   include/hw/arm/stm32f205_soc.h       |  2 +-
> >   include/hw/arm/stm32f405_soc.h       |  2 +-
> >   include/hw/arm/xlnx-versal.h         |  6 +++---
> >   include/hw/arm/xlnx-zynqmp.h         |  2 +-
> >   include/hw/or-irq.h                  |  5 +----
> >   iothread.c                           |  4 ----
> >   14 files changed, 30 insertions(+), 38 deletions(-)
>
> Since most changes concern ARM machines, can this (reviewed)
> series go via your tree?

Sure; applied to target-arm.next, thanks.

-- PMM