irqchip/gic-v3: Add redistributor iterator
authorMarc Zyngier <marc.zyngier@arm.com>
Mon, 19 Dec 2016 17:00:38 +0000 (17:00 +0000)
committerMarc Zyngier <marc.zyngier@arm.com>
Wed, 23 Aug 2017 10:09:14 +0000 (11:09 +0100)
In order to discover the VLPI properties, we need to iterate over
the redistributor regions. As we already have code that does this,
let's factor it out and make it slightly more generic.

Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
drivers/irqchip/irq-gic-v3.c

index afae0a9d0df9b6c217fe474122a199bdcae18f95..fba5f668be59319eceea0420f68ce5316fafeddd 100644 (file)
@@ -421,24 +421,14 @@ static void __init gic_dist_init(void)
                gic_write_irouter(affinity, base + GICD_IROUTER + i * 8);
 }
 
-static int gic_populate_rdist(void)
+static int gic_iterate_rdists(int (*fn)(struct redist_region *, void __iomem *))
 {
-       unsigned long mpidr = cpu_logical_map(smp_processor_id());
-       u64 typer;
-       u32 aff;
+       int ret = -ENODEV;
        int i;
 
-       /*
-        * Convert affinity to a 32bit value that can be matched to
-        * GICR_TYPER bits [63:32].
-        */
-       aff = (MPIDR_AFFINITY_LEVEL(mpidr, 3) << 24 |
-              MPIDR_AFFINITY_LEVEL(mpidr, 2) << 16 |
-              MPIDR_AFFINITY_LEVEL(mpidr, 1) << 8 |
-              MPIDR_AFFINITY_LEVEL(mpidr, 0));
-
        for (i = 0; i < gic_data.nr_redist_regions; i++) {
                void __iomem *ptr = gic_data.redist_regions[i].redist_base;
+               u64 typer;
                u32 reg;
 
                reg = readl_relaxed(ptr + GICR_PIDR2) & GIC_PIDR2_ARCH_MASK;
@@ -450,15 +440,9 @@ static int gic_populate_rdist(void)
 
                do {
                        typer = gic_read_typer(ptr + GICR_TYPER);
-                       if ((typer >> 32) == aff) {
-                               u64 offset = ptr - gic_data.redist_regions[i].redist_base;
-                               gic_data_rdist_rd_base() = ptr;
-                               gic_data_rdist()->phys_base = gic_data.redist_regions[i].phys_base + offset;
-                               pr_info("CPU%d: found redistributor %lx region %d:%pa\n",
-                                       smp_processor_id(), mpidr, i,
-                                       &gic_data_rdist()->phys_base);
+                       ret = fn(gic_data.redist_regions + i, ptr);
+                       if (!ret)
                                return 0;
-                       }
 
                        if (gic_data.redist_regions[i].single_redist)
                                break;
@@ -473,9 +457,50 @@ static int gic_populate_rdist(void)
                } while (!(typer & GICR_TYPER_LAST));
        }
 
+       return ret ? -ENODEV : 0;
+}
+
+static int __gic_populate_rdist(struct redist_region *region, void __iomem *ptr)
+{
+       unsigned long mpidr = cpu_logical_map(smp_processor_id());
+       u64 typer;
+       u32 aff;
+
+       /*
+        * Convert affinity to a 32bit value that can be matched to
+        * GICR_TYPER bits [63:32].
+        */
+       aff = (MPIDR_AFFINITY_LEVEL(mpidr, 3) << 24 |
+              MPIDR_AFFINITY_LEVEL(mpidr, 2) << 16 |
+              MPIDR_AFFINITY_LEVEL(mpidr, 1) << 8 |
+              MPIDR_AFFINITY_LEVEL(mpidr, 0));
+
+       typer = gic_read_typer(ptr + GICR_TYPER);
+       if ((typer >> 32) == aff) {
+               u64 offset = ptr - region->redist_base;
+               gic_data_rdist_rd_base() = ptr;
+               gic_data_rdist()->phys_base = region->phys_base + offset;
+
+               pr_info("CPU%d: found redistributor %lx region %d:%pa\n",
+                       smp_processor_id(), mpidr,
+                       (int)(region - gic_data.redist_regions),
+                       &gic_data_rdist()->phys_base);
+               return 0;
+       }
+
+       /* Try next one */
+       return 1;
+}
+
+static int gic_populate_rdist(void)
+{
+       if (gic_iterate_rdists(__gic_populate_rdist) == 0)
+               return 0;
+
        /* We couldn't even deal with ourselves... */
        WARN(true, "CPU%d: mpidr %lx has no re-distributor!\n",
-            smp_processor_id(), mpidr);
+            smp_processor_id(),
+            (unsigned long)cpu_logical_map(smp_processor_id()));
        return -ENODEV;
 }