Commit c191789c authored by Russell King's avatar Russell King

ARM: irq migration: update GIC migration code

This cleans up after the conversion to irq_data.  Rename the function
to match the method, and remove the now useless lookup of the irq
descriptor which is never used.  Move the bitmask calculation out of
the irq_controller_lock region.
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent 61791244
...@@ -142,25 +142,24 @@ static int gic_set_type(struct irq_data *d, unsigned int type) ...@@ -142,25 +142,24 @@ static int gic_set_type(struct irq_data *d, unsigned int type)
} }
#ifdef CONFIG_SMP #ifdef CONFIG_SMP
static int static int gic_set_affinity(struct irq_data *d, const struct cpumask *mask_val,
gic_set_cpu(struct irq_data *d, const struct cpumask *mask_val, bool force) bool force)
{ {
void __iomem *reg = gic_dist_base(d) + GIC_DIST_TARGET + (gic_irq(d) & ~3); void __iomem *reg = gic_dist_base(d) + GIC_DIST_TARGET + (gic_irq(d) & ~3);
unsigned int shift = (d->irq % 4) * 8; unsigned int shift = (d->irq % 4) * 8;
unsigned int cpu = cpumask_first(mask_val); unsigned int cpu = cpumask_first(mask_val);
u32 val; u32 val, mask, bit;
struct irq_desc *desc;
spin_lock(&irq_controller_lock); if (cpu >= 8)
desc = irq_to_desc(d->irq);
if (desc == NULL) {
spin_unlock(&irq_controller_lock);
return -EINVAL; return -EINVAL;
}
mask = 0xff << shift;
bit = 1 << (cpu + shift);
spin_lock(&irq_controller_lock);
d->node = cpu; d->node = cpu;
val = readl(reg) & ~(0xff << shift); val = readl(reg) & ~mask;
val |= 1 << (cpu + shift); writel(val | bit, reg);
writel(val, reg);
spin_unlock(&irq_controller_lock); spin_unlock(&irq_controller_lock);
return 0; return 0;
...@@ -203,7 +202,7 @@ static struct irq_chip gic_chip = { ...@@ -203,7 +202,7 @@ static struct irq_chip gic_chip = {
.irq_unmask = gic_unmask_irq, .irq_unmask = gic_unmask_irq,
.irq_set_type = gic_set_type, .irq_set_type = gic_set_type,
#ifdef CONFIG_SMP #ifdef CONFIG_SMP
.irq_set_affinity = gic_set_cpu, .irq_set_affinity = gic_set_affinity,
#endif #endif
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment