diff options
author | Jiang Liu <jiang.liu@linux.intel.com> | 2014-06-09 04:20:04 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2014-06-21 17:05:44 -0400 |
commit | 16ee7b3dcc56be14b9a813612cff2cc2339cdced (patch) | |
tree | d9af7dfef67c95ade1dacc7dc1fc9f1dac3aa944 /arch/x86/kernel/acpi | |
parent | 9f354b0252b81c870ac7534d71906280cb573f86 (diff) |
x86, irq: Simplify the way to handle ISA IRQ
On startup, setup_IO_APIC_irqs() will program all IOAPIC pins for ISA
IRQs. Later when mp_map_pin_to_irq() is called, it just returns ISA IRQ
number without programming corresponding IOAPIC pin.
This patch consolidates the way to program IOAPIC pins for both ISA and
non-ISA IRQs into mp_map_pin_to_irq() as below:
1) For ISA IRQs, mp_irqs array is used to map IOAPIC pin to IRQ and
mp_irqdomain_map() is used to actually program the pin.
2) For non-ISA IRQs, irqdomain is used to map IOAPIC pin to IRQ, and
mp_irqdomain_map() is also used to actually program the pin.
Signed-off-by: Jiang Liu <jiang.liu@linux.intel.com>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Joerg Roedel <joro@8bytes.org>
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Grant Likely <grant.likely@linaro.org>
Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
Cc: Bjorn Helgaas <bhelgaas@google.com>
Cc: Randy Dunlap <rdunlap@infradead.org>
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: Len Brown <len.brown@intel.com>
Cc: Pavel Machek <pavel@ucw.cz>
Link: http://lkml.kernel.org/r/1402302011-23642-36-git-send-email-jiang.liu@linux.intel.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/kernel/acpi')
-rw-r--r-- | arch/x86/kernel/acpi/boot.c | 15 |
1 files changed, 2 insertions, 13 deletions
diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c index 9add76f15cb0..fd4b6d2e436c 100644 --- a/arch/x86/kernel/acpi/boot.c +++ b/arch/x86/kernel/acpi/boot.c | |||
@@ -101,17 +101,6 @@ static u32 isa_irq_to_gsi[NR_IRQS_LEGACY] __read_mostly = { | |||
101 | 101 | ||
102 | #define ACPI_INVALID_GSI INT_MIN | 102 | #define ACPI_INVALID_GSI INT_MIN |
103 | 103 | ||
104 | static int map_gsi_to_irq(unsigned int gsi, unsigned int flags) | ||
105 | { | ||
106 | int i; | ||
107 | |||
108 | for (i = 0; i < nr_legacy_irqs(); i++) | ||
109 | if (isa_irq_to_gsi[i] == gsi) | ||
110 | return i; | ||
111 | |||
112 | return mp_map_gsi_to_irq(gsi, flags); | ||
113 | } | ||
114 | |||
115 | /* | 104 | /* |
116 | * This is just a simple wrapper around early_ioremap(), | 105 | * This is just a simple wrapper around early_ioremap(), |
117 | * with sanity checks for phys == 0 and size == 0. | 106 | * with sanity checks for phys == 0 and size == 0. |
@@ -422,7 +411,7 @@ static int mp_register_gsi(struct device *dev, u32 gsi, int trigger, | |||
422 | return -1; | 411 | return -1; |
423 | } | 412 | } |
424 | 413 | ||
425 | irq = map_gsi_to_irq(gsi, IOAPIC_MAP_ALLOC); | 414 | irq = mp_map_gsi_to_irq(gsi, IOAPIC_MAP_ALLOC); |
426 | if (irq < 0) | 415 | if (irq < 0) |
427 | return irq; | 416 | return irq; |
428 | 417 | ||
@@ -603,7 +592,7 @@ void __init acpi_pic_sci_set_trigger(unsigned int irq, u16 trigger) | |||
603 | 592 | ||
604 | int acpi_gsi_to_irq(u32 gsi, unsigned int *irqp) | 593 | int acpi_gsi_to_irq(u32 gsi, unsigned int *irqp) |
605 | { | 594 | { |
606 | int irq = map_gsi_to_irq(gsi, IOAPIC_MAP_ALLOC | IOAPIC_MAP_CHECK); | 595 | int irq = mp_map_gsi_to_irq(gsi, IOAPIC_MAP_ALLOC | IOAPIC_MAP_CHECK); |
607 | 596 | ||
608 | if (irq >= 0) { | 597 | if (irq >= 0) { |
609 | *irqp = irq; | 598 | *irqp = irq; |