summaryrefslogtreecommitdiffstats
path: root/drivers/irqchip/irq-mvebu-odmi.c
diff options
context:
space:
mode:
authorMarc Zyngier <marc.zyngier@arm.com>2016-02-19 16:00:29 +0100
committerJason Cooper <jason@lakedaemon.net>2016-02-19 16:42:29 +0100
commit0407daceedfed003eaacb850d06cbbe359348367 (patch)
treec47fc329b56a23ce8d329c53357016359bc1e550 /drivers/irqchip/irq-mvebu-odmi.c
parentirqchip/mvebu-odmi: Add new driver for platform MSI on Marvell 7K/8K (diff)
downloadlinux-0407daceedfed003eaacb850d06cbbe359348367.tar.xz
linux-0407daceedfed003eaacb850d06cbbe359348367.zip
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-mvebu-odmi.c')
-rw-r--r--drivers/irqchip/irq-mvebu-odmi.c14
1 files changed, 1 insertions, 13 deletions
diff --git a/drivers/irqchip/irq-mvebu-odmi.c b/drivers/irqchip/irq-mvebu-odmi.c
index 0ae98b24836f..b4d367868dbb 100644
--- a/drivers/irqchip/irq-mvebu-odmi.c
+++ b/drivers/irqchip/irq-mvebu-odmi.c
@@ -47,18 +47,6 @@ static unsigned int odmis_count;
/* Protects odmis_bm */
static DEFINE_SPINLOCK(odmis_bm_lock);
-static int odmi_set_affinity(struct irq_data *d,
- const struct cpumask *mask, bool force)
-{
- int ret;
-
- ret = irq_chip_set_affinity_parent(d, mask, force);
- if (ret == IRQ_SET_MASK_OK)
- ret = IRQ_SET_MASK_OK_DONE;
-
- return ret;
-}
-
static void odmi_compose_msi_msg(struct irq_data *d, struct msi_msg *msg)
{
struct odmi_data *odmi;
@@ -83,7 +71,7 @@ static struct irq_chip odmi_irq_chip = {
.irq_mask = irq_chip_mask_parent,
.irq_unmask = irq_chip_unmask_parent,
.irq_eoi = irq_chip_eoi_parent,
- .irq_set_affinity = odmi_set_affinity,
+ .irq_set_affinity = irq_chip_set_affinity_parent,
.irq_compose_msi_msg = odmi_compose_msi_msg,
};