diff options
author | Milton Miller <miltonm@bga.com> | 2011-05-10 15:30:26 -0400 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2011-05-19 01:31:53 -0400 |
commit | da0519800260a3c791b4fe3317a0c7560027a372 (patch) | |
tree | 8512a5dcee8eaf0dc78c4814aecf1413cc4b8869 /arch/powerpc/kernel/irq.c | |
parent | 23f73a5fb0dee5ab681bfeb8897bcfc57153ba9a (diff) |
powerpc: Remove irq_host_ops->remap hook
It was called from irq_create_mapping if that was called for a host
and hwirq that was previously mapped, "to update the flags". But the
only implementation was in beat_interrupt and all it did was repeat a
hypervisor call without error checking that was performed with error
checking at the beginning of the map hook. In addition, the comment on
the beat remap hook says it will only called once for a given mapping,
which would apply to map not remap.
All flags should be known by the time the match hook is called, before
we call the map hook. Removing this mostly unused hook will simpify
the requirements of irq_domain concept.
Signed-off-by: Milton Miller <miltonm@bga.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to 'arch/powerpc/kernel/irq.c')
-rw-r--r-- | arch/powerpc/kernel/irq.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/powerpc/kernel/irq.c b/arch/powerpc/kernel/irq.c index 4a5aa8ca97a5..0715a09a4101 100644 --- a/arch/powerpc/kernel/irq.c +++ b/arch/powerpc/kernel/irq.c | |||
@@ -732,8 +732,6 @@ unsigned int irq_create_mapping(struct irq_host *host, | |||
732 | */ | 732 | */ |
733 | virq = irq_find_mapping(host, hwirq); | 733 | virq = irq_find_mapping(host, hwirq); |
734 | if (virq != NO_IRQ) { | 734 | if (virq != NO_IRQ) { |
735 | if (host->ops->remap) | ||
736 | host->ops->remap(host, virq, hwirq); | ||
737 | pr_debug("irq: -> existing mapping on virq %d\n", virq); | 735 | pr_debug("irq: -> existing mapping on virq %d\n", virq); |
738 | return virq; | 736 | return virq; |
739 | } | 737 | } |