Merge branch 'irq-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip

Pull irq fixes from Ingo Molnar:
 "Diverse irqchip driver fixes"

* 'irq-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
  irqchip/gic-v3-its: Fix command queue pointer comparison bug
  irqchip/mips-gic: Use the correct local interrupt map registers
  irqchip/ti-sci-inta: Fix kernel crash if irq_create_fwspec_mapping fail
  irqchip/irq-csky-mpintc: Support auto irq deliver to all cpus

4 files changed, 41 insertions, 17 deletions

diff --git a/drivers/irqchip/irq-csky-mpintc.c b/drivers/irqchip/irq-csky-mpintc.c
index c67c961ab6cc..a4c1aacba1ff 100644
--- a/drivers/irqchip/irq-csky-mpintc.c
+++ b/drivers/irqchip/irq-csky-mpintc.c
@@ -89,8 +89,19 @@ static int csky_irq_set_affinity(struct irq_data *d,
         if (cpu >= nr_cpu_ids)
                 return -EINVAL;
 
-        /* Enable interrupt destination */
-        cpu |= BIT(31);
+        /*
+         * The csky,mpintc could support auto irq deliver, but it only
+         * could deliver external irq to one cpu or all cpus. So it
+         * doesn't support deliver external irq to a group of cpus
+         * with cpu_mask.
+         * SO we only use auto deliver mode when affinity mask_val is
+         * equal to cpu_present_mask.
+         *
+         */
+        if (cpumask_equal(mask_val, cpu_present_mask))
+                cpu = 0;
+        else
+                cpu |= BIT(31);
 
         writel_relaxed(cpu, INTCG_base + INTCG_CIDSTR + offset);
 
diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index d29b44b677e4..35500801dc2b 100644
--- a/drivers/irqchip/irq-gic-v3-its.c
+++ b/drivers/irqchip/irq-gic-v3-its.c
@@ -733,32 +733,43 @@ static void its_flush_cmd(struct its_node *its, struct its_cmd_block *cmd)
 }
 
 static int its_wait_for_range_completion(struct its_node *its,
-                                         struct its_cmd_block *from,
+                                         u64    prev_idx,
                                          struct its_cmd_block *to)
 {
-        u64 rd_idx, from_idx, to_idx;
+        u64 rd_idx, to_idx, linear_idx;
         u32 count = 1000000;    /* 1s! */
 
-        from_idx = its_cmd_ptr_to_offset(its, from);
+        /* Linearize to_idx if the command set has wrapped around */
         to_idx = its_cmd_ptr_to_offset(its, to);
+        if (to_idx < prev_idx)
+                to_idx += ITS_CMD_QUEUE_SZ;
+
+        linear_idx = prev_idx;
 
         while (1) {
+                s64 delta;
+
                 rd_idx = readl_relaxed(its->base + GITS_CREADR);
 
-                /* Direct case */
-                if (from_idx < to_idx && rd_idx >= to_idx)
-                        break;
+                /*
+                 * Compute the read pointer progress, taking the
+                 * potential wrap-around into account.
+                 */
+                delta = rd_idx - prev_idx;
+                if (rd_idx < prev_idx)
+                        delta += ITS_CMD_QUEUE_SZ;
 
-                /* Wrapped case */
-                if (from_idx >= to_idx && rd_idx >= to_idx && rd_idx < from_idx)
+                linear_idx += delta;
+                if (linear_idx >= to_idx)
                         break;
 
                 count--;
                 if (!count) {
-                        pr_err_ratelimited("ITS queue timeout (%llu %llu %llu)\n",
-                                           from_idx, to_idx, rd_idx);
+                        pr_err_ratelimited("ITS queue timeout (%llu %llu)\n",
+                                           to_idx, linear_idx);
                         return -1;
                 }
+                prev_idx = rd_idx;
                 cpu_relax();
                 udelay(1);
         }
@@ -775,6 +786,7 @@ void name(struct its_node *its,						\
         struct its_cmd_block *cmd, *sync_cmd, *next_cmd;                \
         synctype *sync_obj;                                             \
         unsigned long flags;                                            \
+        u64 rd_idx;                                                     \
                                                                         \
         raw_spin_lock_irqsave(&its->lock, flags);                       \
                                                                         \
@@ -796,10 +808,11 @@ void name(struct its_node *its,						\
         }                                                               \
                                                                         \
 post:                                                                   \
+        rd_idx = readl_relaxed(its->base + GITS_CREADR);                \
         next_cmd = its_post_commands(its);                              \
         raw_spin_unlock_irqrestore(&its->lock, flags);                  \
                                                                         \
-        if (its_wait_for_range_completion(its, cmd, next_cmd))          \
+        if (its_wait_for_range_completion(its, rd_idx, next_cmd))       \
                 pr_err_ratelimited("ITS cmd %ps failed\n", builder);    \
 }
 
diff --git a/drivers/irqchip/irq-mips-gic.c b/drivers/irqchip/irq-mips-gic.c
index d32268cc1174..f3985469c221 100644
--- a/drivers/irqchip/irq-mips-gic.c
+++ b/drivers/irqchip/irq-mips-gic.c
@@ -388,7 +388,7 @@ static void gic_all_vpes_irq_cpu_online(struct irq_data *d)
         intr = GIC_HWIRQ_TO_LOCAL(d->hwirq);
         cd = irq_data_get_irq_chip_data(d);
 
-        write_gic_vl_map(intr, cd->map);
+        write_gic_vl_map(mips_gic_vx_map_reg(intr), cd->map);
         if (cd->mask)
                 write_gic_vl_smask(BIT(intr));
 }
@@ -517,7 +517,7 @@ static int gic_irq_domain_map(struct irq_domain *d, unsigned int virq,
         spin_lock_irqsave(&gic_lock, flags);
         for_each_online_cpu(cpu) {
                 write_gic_vl_other(mips_cm_vp_id(cpu));
-                write_gic_vo_map(intr, map);
+                write_gic_vo_map(mips_gic_vx_map_reg(intr), map);
         }
         spin_unlock_irqrestore(&gic_lock, flags);
 
diff --git a/drivers/irqchip/irq-ti-sci-inta.c b/drivers/irqchip/irq-ti-sci-inta.c
index 011b60a49e3f..ef4d625d2d80 100644
--- a/drivers/irqchip/irq-ti-sci-inta.c
+++ b/drivers/irqchip/irq-ti-sci-inta.c
@@ -159,9 +159,9 @@ static struct ti_sci_inta_vint_desc *ti_sci_inta_alloc_parent_irq(struct irq_dom
         parent_fwspec.param[1] = vint_desc->vint_id;
 
         parent_virq = irq_create_fwspec_mapping(&parent_fwspec);
-        if (parent_virq <= 0) {
+        if (parent_virq == 0) {
                 kfree(vint_desc);
-                return ERR_PTR(parent_virq);
+                return ERR_PTR(-EINVAL);
         }
         vint_desc->parent_virq = parent_virq;