[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 05/13] hw/ppc: Remove unused ppcuic_init()
From: |
David Gibson |
Subject: |
[PULL 05/13] hw/ppc: Remove unused ppcuic_init() |
Date: |
Tue, 19 Jan 2021 17:23:10 +1100 |
From: Peter Maydell <peter.maydell@linaro.org>
Now we've converted all the callsites to directly create the QOM UIC
device themselves, the ppcuic_init() function is unused and can be
removed. The enum defining PPCUIC symbolic constants can be moved
to the ppc-uic.h header where it more naturally belongs.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Tested-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Message-Id: <20210108171212.16500-5-peter.maydell@linaro.org>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
hw/ppc/ppc4xx_devs.c | 38 --------------------------------------
include/hw/intc/ppc-uic.h | 7 +++++++
include/hw/ppc/ppc4xx.h | 9 ---------
3 files changed, 7 insertions(+), 47 deletions(-)
diff --git a/hw/ppc/ppc4xx_devs.c b/hw/ppc/ppc4xx_devs.c
index ffe4cf43e8..fe9d4f7155 100644
--- a/hw/ppc/ppc4xx_devs.c
+++ b/hw/ppc/ppc4xx_devs.c
@@ -77,44 +77,6 @@ PowerPCCPU *ppc4xx_init(const char *cpu_type,
return cpu;
}
-/*****************************************************************************/
-/* "Universal" Interrupt controller */
-
-qemu_irq *ppcuic_init (CPUPPCState *env, qemu_irq *irqs,
- uint32_t dcr_base, int has_ssr, int has_vr)
-{
- DeviceState *uicdev = qdev_new(TYPE_PPC_UIC);
- SysBusDevice *uicsbd = SYS_BUS_DEVICE(uicdev);
- qemu_irq *uic_irqs;
- int i;
-
- qdev_prop_set_uint32(uicdev, "dcr-base", dcr_base);
- qdev_prop_set_bit(uicdev, "use-vectors", has_vr);
- object_property_set_link(OBJECT(uicdev), "cpu", OBJECT(env_cpu(env)),
- &error_fatal);
- sysbus_realize_and_unref(uicsbd, &error_fatal);
-
- sysbus_connect_irq(uicsbd, PPCUIC_OUTPUT_INT, irqs[PPCUIC_OUTPUT_INT]);
- sysbus_connect_irq(uicsbd, PPCUIC_OUTPUT_CINT, irqs[PPCUIC_OUTPUT_CINT]);
-
- /*
- * Return an allocated array of the UIC's input IRQ lines.
- * This is an ugly temporary API to retain compatibility with
- * the ppcuic_init() interface from the pre-QOM-conversion UIC.
- * None of the callers free this array, so it is leaked -- but
- * so was the array allocated by qemu_allocate_irqs() in the
- * old code.
- *
- * The callers should just instantiate the UIC and wire it up
- * themselves rather than passing qemu_irq* in and out of this function.
- */
- uic_irqs = g_new0(qemu_irq, UIC_MAX_IRQ);
- for (i = 0; i < UIC_MAX_IRQ; i++) {
- uic_irqs[i] = qdev_get_gpio_in(uicdev, i);
- }
- return uic_irqs;
-}
-
/*****************************************************************************/
/* SDRAM controller */
typedef struct ppc4xx_sdram_t ppc4xx_sdram_t;
diff --git a/include/hw/intc/ppc-uic.h b/include/hw/intc/ppc-uic.h
index e614e2ffd8..22dd5e5ac2 100644
--- a/include/hw/intc/ppc-uic.h
+++ b/include/hw/intc/ppc-uic.h
@@ -47,6 +47,13 @@ OBJECT_DECLARE_SIMPLE_TYPE(PPCUIC, PPC_UIC)
#define UIC_MAX_IRQ 32
+/* Symbolic constants for the sysbus IRQ outputs */
+enum {
+ PPCUIC_OUTPUT_INT = 0,
+ PPCUIC_OUTPUT_CINT = 1,
+ PPCUIC_OUTPUT_NB,
+};
+
struct PPCUIC {
/*< private >*/
SysBusDevice parent_obj;
diff --git a/include/hw/ppc/ppc4xx.h b/include/hw/ppc/ppc4xx.h
index cc19c8da5b..980f964b5a 100644
--- a/include/hw/ppc/ppc4xx.h
+++ b/include/hw/ppc/ppc4xx.h
@@ -33,15 +33,6 @@ PowerPCCPU *ppc4xx_init(const char *cpu_model,
clk_setup_t *cpu_clk, clk_setup_t *tb_clk,
uint32_t sysclk);
-/* PowerPC 4xx universal interrupt controller */
-enum {
- PPCUIC_OUTPUT_INT = 0,
- PPCUIC_OUTPUT_CINT = 1,
- PPCUIC_OUTPUT_NB,
-};
-qemu_irq *ppcuic_init (CPUPPCState *env, qemu_irq *irqs,
- uint32_t dcr_base, int has_ssr, int has_vr);
-
void ppc4xx_sdram_banks(MemoryRegion *ram, int nr_banks,
MemoryRegion ram_memories[],
hwaddr ram_bases[], hwaddr ram_sizes[],
--
2.29.2
- [PULL 00/13] ppc-for-6.0 queue 20210119, David Gibson, 2021/01/19
- [PULL 03/13] hw/intc/ppc-uic: Make default dcr-base 0xc0, not 0x30, David Gibson, 2021/01/19
- [PULL 02/13] hw/ppc: Delete unused ppc405cr_init() code, David Gibson, 2021/01/19
- [PULL 01/13] hw/ppc/sam460ex: Drop use of ppcuic_init(), David Gibson, 2021/01/19
- [PULL 04/13] hw/ppc/ppc405_uc: Drop use of ppcuic_init(), David Gibson, 2021/01/19
- [PULL 08/13] sam460ex: Use type cast macro instead of simple cast, David Gibson, 2021/01/19
- [PULL 12/13] spapr_rtas.c: fix identation of rtas_ibm_suspend_me() args, David Gibson, 2021/01/19
- [PULL 10/13] spapr.h: fix trailing whitespace in phb_placement, David Gibson, 2021/01/19
- [PULL 05/13] hw/ppc: Remove unused ppcuic_init(),
David Gibson <=
- [PULL 06/13] Revert "sam460ex: Remove FDT_PPC dependency from KConfig", David Gibson, 2021/01/19
- [PULL 13/13] spapr_cpu_core.c: use g_auto* in spapr_create_vcpu(), David Gibson, 2021/01/19
- [PULL 07/13] Revert "ppc4xx: Move common dependency on serial to common option", David Gibson, 2021/01/19
- [PULL 09/13] spapr: Improve handling of memory unplug with old guests, David Gibson, 2021/01/19
- [PULL 11/13] spapr_hcall.c: make do_client_architecture_support static, David Gibson, 2021/01/19
- Re: [PULL 00/13] ppc-for-6.0 queue 20210119, Peter Maydell, 2021/01/19