Juno: Use the generic ARM GIC driver
authorJuan Castillo <juan.castillo@arm.com>
Mon, 20 Oct 2014 11:29:58 +0000 (12:29 +0100)
committerJuan Castillo <juan.castillo@arm.com>
Fri, 31 Oct 2014 18:40:41 +0000 (18:40 +0000)
This patch replaces the usage of the GIC private driver in Juno with
the generic ARM GIC driver. The private driver is no longer necessary
and has been removed from the Juno port.

Fixes ARM-software/tf-issues#253

Change-Id: I6aaabc252e5e6fb5fcf44ab6d0febd9b38791056

plat/juno/aarch64/juno_common.c
plat/juno/bl31_plat_setup.c
plat/juno/juno_private.h
plat/juno/plat_gic.c [deleted file]
plat/juno/plat_pm.c
plat/juno/platform.mk
plat/juno/tsp/tsp-juno.mk
plat/juno/tsp/tsp_plat_setup.c

index 401f5fec8665f6d1a9232ca1874680f9c1c17c45..70b2f52fb834617fbeaf2829504aa503640bf112 100644 (file)
@@ -29,6 +29,7 @@
  */
 
 #include <arch_helpers.h>
+#include <arm_gic.h>
 #include <assert.h>
 #include <bl_common.h>
 #include <debug.h>
@@ -107,6 +108,28 @@ static const mmap_region_t juno_mmap[] = {
 };
 #endif
 
+/* Array of secure interrupts to be configured by the gic driver */
+const unsigned int irq_sec_array[] = {
+       IRQ_MHU,
+       IRQ_GPU_SMMU_0,
+       IRQ_GPU_SMMU_1,
+       IRQ_ETR_SMMU,
+       IRQ_TZC400,
+       IRQ_TZ_WDOG,
+       IRQ_SEC_PHY_TIMER,
+       IRQ_SEC_SGI_0,
+       IRQ_SEC_SGI_1,
+       IRQ_SEC_SGI_2,
+       IRQ_SEC_SGI_3,
+       IRQ_SEC_SGI_4,
+       IRQ_SEC_SGI_5,
+       IRQ_SEC_SGI_6,
+       IRQ_SEC_SGI_7
+};
+
+const unsigned int num_sec_irqs = sizeof(irq_sec_array) /
+       sizeof(irq_sec_array[0]);
+
 /*******************************************************************************
  * Macro generating the code for the function setting up the pagetables as per
  * the platform memory map & initialize the mmu, for the given exception level
@@ -157,3 +180,8 @@ uint64_t plat_get_syscnt_freq(void)
 
        return counter_base_frequency;
 }
+
+void plat_gic_init(void)
+{
+       arm_gic_init(GICC_BASE, GICD_BASE, 0, irq_sec_array, num_sec_irqs);
+}
index 4a92d44a8e3e977049d4fff16a15bdd3d765f756..c4504622529e21e6eb70a5bda4450a29a0bf1bff 100644 (file)
@@ -29,6 +29,7 @@
  */
 
 #include <arch.h>
+#include <arm_gic.h>
 #include <assert.h>
 #include <bl31.h>
 #include <bl_common.h>
@@ -151,7 +152,8 @@ void bl31_platform_setup(void)
        mhu_secure_init();
 
        /* Initialize the gic cpu and distributor interfaces */
-       gic_setup();
+       plat_gic_init();
+       arm_gic_setup();
 
        /* Enable and initialize the System level generic timer */
        mmio_write_32(SYS_CNTCTL_BASE + CNTCR_OFF, CNTCR_FCREQ(0) | CNTCR_EN);
index 0dac03a9dfdfc1b1233f96b2b6cd94a88b03fcbc..a6d1a1af1a42d1de3a2858c55dc6b974800990f4 100644 (file)
@@ -83,18 +83,7 @@ void plat_report_exception(unsigned long type);
 unsigned long plat_get_ns_image_entrypoint(void);
 unsigned long platform_get_stack(unsigned long mpidr);
 uint64_t plat_get_syscnt_freq(void);
-
-/* Declarations for plat_gic.c */
-uint32_t ic_get_pending_interrupt_id(void);
-uint32_t ic_get_pending_interrupt_type(void);
-uint32_t ic_acknowledge_interrupt(void);
-uint32_t ic_get_interrupt_type(uint32_t id);
-void ic_end_of_interrupt(uint32_t id);
-void gic_cpuif_deactivate(unsigned int gicc_base);
-void gic_cpuif_setup(unsigned int gicc_base);
-void gic_pcpu_distif_setup(unsigned int gicd_base);
-void gic_setup(void);
-uint32_t plat_interrupt_type_to_line(uint32_t type, uint32_t security_state);
+void plat_gic_init(void);
 
 /* Declarations for plat_topology.c */
 int plat_setup_topology(void);
diff --git a/plat/juno/plat_gic.c b/plat/juno/plat_gic.c
deleted file mode 100644 (file)
index 9001519..0000000
+++ /dev/null
@@ -1,283 +0,0 @@
-/*
- * Copyright (c) 2013-2014, ARM Limited and Contributors. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are met:
- *
- * Redistributions of source code must retain the above copyright notice, this
- * list of conditions and the following disclaimer.
- *
- * Redistributions in binary form must reproduce the above copyright notice,
- * this list of conditions and the following disclaimer in the documentation
- * and/or other materials provided with the distribution.
- *
- * Neither the name of ARM nor the names of its contributors may be used
- * to endorse or promote products derived from this software without specific
- * prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
- * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
- * POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include <arch_helpers.h>
-#include <assert.h>
-#include <bl_common.h>
-#include <gic_v2.h>
-#include <interrupt_mgmt.h>
-#include <platform.h>
-#include "juno_def.h"
-#include "juno_private.h"
-
-
-/* Value used to initialise Non-Secure irq priorities four at a time */
-#define DEFAULT_NS_PRIORITY_X4 \
-       (GIC_HIGHEST_NS_PRIORITY | \
-       (GIC_HIGHEST_NS_PRIORITY << 8) | \
-       (GIC_HIGHEST_NS_PRIORITY << 16) | \
-       (GIC_HIGHEST_NS_PRIORITY << 24))
-
-
-/*******************************************************************************
- * Enable secure interrupts and use FIQs to route them. Disable legacy bypass
- * and set the priority mask register to allow all interrupts to trickle in.
- ******************************************************************************/
-void gic_cpuif_setup(unsigned int gicc_base)
-{
-       unsigned int val;
-
-       gicc_write_pmr(gicc_base, GIC_PRI_MASK);
-
-       val = ENABLE_GRP0 | FIQ_EN;
-       val |= FIQ_BYP_DIS_GRP0 | IRQ_BYP_DIS_GRP0;
-       val |= FIQ_BYP_DIS_GRP1 | IRQ_BYP_DIS_GRP1;
-       gicc_write_ctlr(gicc_base, val);
-}
-
-/*******************************************************************************
- * Place the cpu interface in a state where it can never make a cpu exit wfi as
- * as result of an asserted interrupt. This is critical for powering down a cpu
- ******************************************************************************/
-void gic_cpuif_deactivate(unsigned int gicc_base)
-{
-       unsigned int val;
-
-       /* Disable secure, non-secure interrupts and disable their bypass */
-       val = gicc_read_ctlr(gicc_base);
-       val &= ~(ENABLE_GRP0 | ENABLE_GRP1);
-       val |= FIQ_BYP_DIS_GRP1 | FIQ_BYP_DIS_GRP0;
-       val |= IRQ_BYP_DIS_GRP0 | IRQ_BYP_DIS_GRP1;
-       gicc_write_ctlr(gicc_base, val);
-}
-
-static void gic_set_secure(unsigned int gicd_base, unsigned id)
-{
-       /* Set interrupt as Group 0 */
-       gicd_clr_igroupr(gicd_base, id);
-
-       /* Set priority to max */
-       gicd_set_ipriorityr(gicd_base, id, GIC_HIGHEST_SEC_PRIORITY);
-}
-
-/*******************************************************************************
- * Per cpu gic distributor setup which will be done by all cpus after a cold
- * boot/hotplug. This marks out the secure interrupts & enables them.
- ******************************************************************************/
-void gic_pcpu_distif_setup(unsigned int gicd_base)
-{
-       unsigned i;
-
-       /* Mark all 32 PPI interrupts as Group 1 (non-secure) */
-       mmio_write_32(gicd_base + GICD_IGROUPR, 0xffffffffu);
-
-       /* Setup PPI priorities doing four at a time */
-       for (i = 0; i < 32; i += 4)
-               mmio_write_32(gicd_base + GICD_IPRIORITYR + i, DEFAULT_NS_PRIORITY_X4);
-
-       /* Configure those PPIs we want as secure, and enable them. */
-       static const char sec_irq[] = {
-               IRQ_SEC_PHY_TIMER,
-               IRQ_SEC_SGI_0,
-               IRQ_SEC_SGI_1,
-               IRQ_SEC_SGI_2,
-               IRQ_SEC_SGI_3,
-               IRQ_SEC_SGI_4,
-               IRQ_SEC_SGI_5,
-               IRQ_SEC_SGI_6,
-               IRQ_SEC_SGI_7
-       };
-       for (i = 0; i < sizeof(sec_irq) / sizeof(sec_irq[0]); i++) {
-               gic_set_secure(gicd_base, sec_irq[i]);
-               gicd_set_isenabler(gicd_base, sec_irq[i]);
-       }
-}
-
-/*******************************************************************************
- * Global gic distributor setup which will be done by the primary cpu after a
- * cold boot. It marks out the secure SPIs, PPIs & SGIs and enables them. It
- * then enables the secure GIC distributor interface.
- ******************************************************************************/
-static void gic_distif_setup(unsigned int gicd_base)
-{
-       unsigned int i, ctlr;
-       const unsigned int ITLinesNumber =
-                               gicd_read_typer(gicd_base) & IT_LINES_NO_MASK;
-
-       /* Disable the distributor before going further */
-       ctlr = gicd_read_ctlr(gicd_base);
-       ctlr &= ~(ENABLE_GRP0 | ENABLE_GRP1);
-       gicd_write_ctlr(gicd_base, ctlr);
-
-       /* Mark all lines of SPIs as Group 1 (non-secure) */
-       for (i = 0; i < ITLinesNumber; i++)
-               mmio_write_32(gicd_base + GICD_IGROUPR + 4 + i * 4, 0xffffffffu);
-
-       /* Setup SPI priorities doing four at a time */
-       for (i = 0; i < ITLinesNumber * 32; i += 4)
-               mmio_write_32(gicd_base + GICD_IPRIORITYR + 32 + i, DEFAULT_NS_PRIORITY_X4);
-
-       /* Configure the SPIs we want as secure */
-       static const char sec_irq[] = {
-               IRQ_MHU,
-               IRQ_GPU_SMMU_0,
-               IRQ_GPU_SMMU_1,
-               IRQ_ETR_SMMU,
-               IRQ_TZC400,
-               IRQ_TZ_WDOG
-       };
-       for (i = 0; i < sizeof(sec_irq) / sizeof(sec_irq[0]); i++)
-               gic_set_secure(gicd_base, sec_irq[i]);
-
-       /* Route watchdog interrupt to this CPU and enable it. */
-       gicd_set_itargetsr(gicd_base, IRQ_TZ_WDOG,
-                          platform_get_core_pos(read_mpidr()));
-       gicd_set_isenabler(gicd_base, IRQ_TZ_WDOG);
-
-       /* Now setup the PPIs */
-       gic_pcpu_distif_setup(gicd_base);
-
-       /* Enable Group 0 (secure) interrupts */
-       gicd_write_ctlr(gicd_base, ctlr | ENABLE_GRP0);
-}
-
-void gic_setup(void)
-{
-       gic_cpuif_setup(GICC_BASE);
-       gic_distif_setup(GICD_BASE);
-}
-
-/*******************************************************************************
- * An ARM processor signals interrupt exceptions through the IRQ and FIQ pins.
- * The interrupt controller knows which pin/line it uses to signal a type of
- * interrupt. The platform knows which interrupt controller type is being used
- * in a particular security state e.g. with an ARM GIC, normal world could use
- * the GICv2 features while the secure world could use GICv3 features and vice
- * versa.
- * This function is exported by the platform to let the interrupt management
- * framework determine for a type of interrupt and security state, which line
- * should be used in the SCR_EL3 to control its routing to EL3. The interrupt
- * line is represented as the bit position of the IRQ or FIQ bit in the SCR_EL3.
- ******************************************************************************/
-uint32_t plat_interrupt_type_to_line(uint32_t type, uint32_t security_state)
-{
-       assert(type == INTR_TYPE_S_EL1 ||
-              type == INTR_TYPE_EL3 ||
-              type == INTR_TYPE_NS);
-
-       assert(sec_state_is_valid(security_state));
-
-       /*
-        * We ignore the security state parameter because Juno is GICv2 only
-        * so both normal and secure worlds are using ARM GICv2.
-        */
-       return gicv2_interrupt_type_to_line(GICC_BASE, type);
-}
-
-/*******************************************************************************
- * This function returns the type of the highest priority pending interrupt at
- * the GIC cpu interface. INTR_TYPE_INVAL is returned when there is no
- * interrupt pending.
- ******************************************************************************/
-uint32_t plat_ic_get_pending_interrupt_type(void)
-{
-       uint32_t id;
-
-       id = gicc_read_hppir(GICC_BASE);
-
-       /* Assume that all secure interrupts are S-EL1 interrupts */
-       if (id < 1022)
-               return INTR_TYPE_S_EL1;
-
-       if (id == GIC_SPURIOUS_INTERRUPT)
-               return INTR_TYPE_INVAL;
-
-       return INTR_TYPE_NS;
-}
-
-/*******************************************************************************
- * This function returns the id of the highest priority pending interrupt at
- * the GIC cpu interface. INTR_ID_UNAVAILABLE is returned when there is no
- * interrupt pending.
- ******************************************************************************/
-uint32_t plat_ic_get_pending_interrupt_id(void)
-{
-       uint32_t id;
-
-       id = gicc_read_hppir(GICC_BASE);
-
-       if (id < 1022)
-               return id;
-
-       if (id == 1023)
-               return INTR_ID_UNAVAILABLE;
-
-       /*
-        * Find out which non-secure interrupt it is under the assumption that
-        * the GICC_CTLR.AckCtl bit is 0.
-        */
-       return gicc_read_ahppir(GICC_BASE);
-}
-
-/*******************************************************************************
- * This functions reads the GIC cpu interface Interrupt Acknowledge register
- * to start handling the pending interrupt. It returns the contents of the IAR.
- ******************************************************************************/
-uint32_t plat_ic_acknowledge_interrupt(void)
-{
-       return gicc_read_IAR(GICC_BASE);
-}
-
-/*******************************************************************************
- * This functions writes the GIC cpu interface End Of Interrupt register with
- * the passed value to finish handling the active interrupt
- ******************************************************************************/
-void plat_ic_end_of_interrupt(uint32_t id)
-{
-       gicc_write_EOIR(GICC_BASE, id);
-}
-
-/*******************************************************************************
- * This function returns the type of the interrupt id depending upon the group
- * this interrupt has been configured under by the interrupt controller i.e.
- * group0 or group1.
- ******************************************************************************/
-uint32_t plat_ic_get_interrupt_type(uint32_t id)
-{
-       uint32_t group;
-
-       group = gicd_get_igroupr(GICD_BASE, id);
-
-       /* Assume that all secure interrupts are S-EL1 interrupts */
-       if (group == GRP0)
-               return INTR_TYPE_S_EL1;
-       else
-               return INTR_TYPE_NS;
-}
index 74ce89fdfe5bbd41ad6d78ef49a7a41f4636eb92..8409b0c3c1fa8a2481082ec68a41d59cdeae5238 100644 (file)
@@ -30,6 +30,7 @@
 
 #include <assert.h>
 #include <arch_helpers.h>
+#include <arm_gic.h>
 #include <debug.h>
 #include <cci400.h>
 #include <errno.h>
@@ -133,10 +134,10 @@ int32_t juno_affinst_on_finish(uint64_t mpidr, uint32_t afflvl, uint32_t state)
 
 
        /* Enable the gic cpu interface */
-       gic_cpuif_setup(GICC_BASE);
+       arm_gic_cpuif_setup();
 
        /* Juno todo: Is this setup only needed after a cold boot? */
-       gic_pcpu_distif_setup(GICD_BASE);
+       arm_gic_pcpu_distif_setup();
 
        /* Clear the mailbox for this cpu. */
        juno_program_mailbox(mpidr, 0);
@@ -155,7 +156,7 @@ static int32_t juno_power_down_common(uint32_t afflvl)
        uint32_t cluster_state = scpi_power_on;
 
        /* Prevent interrupts from spuriously waking up this cpu */
-       gic_cpuif_deactivate(GICC_BASE);
+       arm_gic_cpuif_deactivate();
 
        /* Cluster is to be turned off, so disable coherency */
        if (afflvl > MPIDR_AFFLVL0) {
index 47465360dbb00b5042ef9848d2ad2c3428602b17..f0904ef0946b9a0cbb7f0ee0f966d7106c3ef380 100644 (file)
@@ -36,6 +36,7 @@ PLAT_BL_COMMON_SOURCES        :=      drivers/arm/pl011/pl011_console.S       \
                                drivers/io/io_storage.c                 \
                                lib/aarch64/xlat_tables.c               \
                                plat/common/aarch64/plat_common.c       \
+                               plat/common/plat_gic.c                  \
                                plat/juno/plat_io_storage.c
 
 BL1_SOURCES            +=      drivers/arm/cci400/cci400.c             \
@@ -57,7 +58,9 @@ BL2_SOURCES           +=      lib/locks/bakery/bakery_lock.c          \
                                plat/juno/scpi.c
 
 BL31_SOURCES           +=      drivers/arm/cci400/cci400.c             \
+                               drivers/arm/gic/arm_gic.c               \
                                drivers/arm/gic/gic_v2.c                \
+                               drivers/arm/gic/gic_v3.c                \
                                lib/cpus/aarch64/cortex_a53.S           \
                                lib/cpus/aarch64/cortex_a57.S           \
                                plat/common/aarch64/platform_mp_stack.S \
@@ -67,7 +70,6 @@ BL31_SOURCES          +=      drivers/arm/cci400/cci400.c             \
                                plat/juno/aarch64/juno_common.c         \
                                plat/juno/plat_pm.c                     \
                                plat/juno/plat_topology.c               \
-                               plat/juno/plat_gic.c                    \
                                plat/juno/scpi.c
 
 ifneq (${RESET_TO_BL31},0)
index d0d29d77370a0464ee30f0f5c02e31daa3dd1f8f..4d56ea2d6912765ba2ef5b569d9ad99413e57fb6 100644 (file)
@@ -29,9 +29,9 @@
 #
 
 # TSP source files specific to Juno platform
-BL32_SOURCES           +=      drivers/arm/gic/gic_v2.c                \
+BL32_SOURCES           +=      drivers/arm/gic/arm_gic.c               \
+                               drivers/arm/gic/gic_v2.c                \
                                plat/common/aarch64/platform_mp_stack.S \
                                plat/juno/aarch64/juno_common.c         \
                                plat/juno/aarch64/plat_helpers.S        \
-                               plat/juno/tsp/tsp_plat_setup.c          \
-                               plat/juno/plat_gic.c
+                               plat/juno/tsp/tsp_plat_setup.c
index 2d4ab81d6094f9bb3e621eff1c668c2943f57fd2..0a9d4cbe8bd850a15cc78764d9fabb81058c07c7 100644 (file)
@@ -80,6 +80,7 @@ void tsp_early_platform_setup(void)
  ******************************************************************************/
 void tsp_platform_setup(void)
 {
+       plat_gic_init();
 }
 
 /*******************************************************************************