diff options
author | Marc Zyngier <marc.zyngier@arm.com> | 2016-02-19 10:00:29 -0500 |
---|---|---|
committer | Jason Cooper <jason@lakedaemon.net> | 2016-02-19 10:42:29 -0500 |
commit | 0407daceedfed003eaacb850d06cbbe359348367 (patch) | |
tree | c47fc329b56a23ce8d329c53357016359bc1e550 /drivers/irqchip/irq-gic.c | |
parent | c27f29bbbf02168c9b1e8ba0fe7a8cb917e5a50f (diff) |
irqchip/gic: Return IRQ_SET_MASK_OK_DONE in the set_affinity method
Moving an SPI around doesn't require any extra work from the rest
of the stack, and specially not for MSI-generated SPIs.
It is then worth returning IRQ_SET_MASK_OK_DONE instead of
IRQ_SET_MASK_OK, and simplify the other irqchips that rely on
this behaviour (GICv2m and Marvell's ODMI controller).
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Link: https://lkml.kernel.org/r/1455894029-17270-1-git-send-email-marc.zyngier@arm.com
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Diffstat (limited to 'drivers/irqchip/irq-gic.c')
-rw-r--r-- | drivers/irqchip/irq-gic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/irqchip/irq-gic.c b/drivers/irqchip/irq-gic.c index 911758c056c1..e14f2f2a7263 100644 --- a/drivers/irqchip/irq-gic.c +++ b/drivers/irqchip/irq-gic.c | |||
@@ -319,7 +319,7 @@ static int gic_set_affinity(struct irq_data *d, const struct cpumask *mask_val, | |||
319 | writel_relaxed(val | bit, reg); | 319 | writel_relaxed(val | bit, reg); |
320 | raw_spin_unlock_irqrestore(&irq_controller_lock, flags); | 320 | raw_spin_unlock_irqrestore(&irq_controller_lock, flags); |
321 | 321 | ||
322 | return IRQ_SET_MASK_OK; | 322 | return IRQ_SET_MASK_OK_DONE; |
323 | } | 323 | } |
324 | #endif | 324 | #endif |
325 | 325 | ||