On 4/4/22 10:46, Peter Maydell wrote:
The exynos4210 code currently has two very similar arrays of IRQs:

  * board_irqs is a field of the Exynos4210Irq struct which is filled
    in by exynos4210_init_board_irqs() with the appropriate qemu_irqs
    for each IRQ the board/SoC can assert
  * irq_table is a set of qemu_irqs pointed to from the
    Exynos4210State struct.  It's allocated in exynos4210_init_irq,
    and the only behaviour these irqs have is that they pass on the
    level to the equivalent board_irqs[] irq

The extra indirection through irq_table is unnecessary, so coalesce
these into a single irq_table[] array as a direct field in
Exynos4210State which exynos4210_init_board_irqs() fills in.

Signed-off-by: Peter Maydell<peter.mayd...@linaro.org>
---
  include/hw/arm/exynos4210.h |  8 ++------
  hw/arm/exynos4210.c         |  6 +-----
  hw/intc/exynos4210_gic.c    | 32 ++++++++------------------------
  3 files changed, 11 insertions(+), 35 deletions(-)

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

r~

Reply via email to