Skip to content

Commit e5adbd6

Browse files
committed
Merge tag 'irq-urgent-2020-04-05' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull irq fixes from Thomas Gleixner: "Two reverts addressing regressions of the Xilinx interrupt controller driver which affected the PPC users" * tag 'irq-urgent-2020-04-05' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: Revert "irqchip/xilinx: Enable generic irq multi handler" Revert "irqchip/xilinx: Do not call irq_set_default_host()"
2 parents 70fbdfe + a67cd1f commit e5adbd6

File tree

4 files changed

+38
-23
lines changed

4 files changed

+38
-23
lines changed

arch/microblaze/Kconfig

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,6 @@ config MICROBLAZE
4747
select CPU_NO_EFFICIENT_FFS
4848
select MMU_GATHER_NO_RANGE if MMU
4949
select SPARSE_IRQ
50-
select GENERIC_IRQ_MULTI_HANDLER
51-
select HANDLE_DOMAIN_IRQ
5250

5351
# Endianness selection
5452
choice

arch/microblaze/include/asm/irq.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,7 @@
1111
struct pt_regs;
1212
extern void do_IRQ(struct pt_regs *regs);
1313

14+
/* should be defined in each interrupt controller driver */
15+
extern unsigned int xintc_get_irq(void);
16+
1417
#endif /* _ASM_MICROBLAZE_IRQ_H */

arch/microblaze/kernel/irq.c

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,29 @@
2020
#include <linux/irqchip.h>
2121
#include <linux/of_irq.h>
2222

23+
static u32 concurrent_irq;
24+
2325
void __irq_entry do_IRQ(struct pt_regs *regs)
2426
{
27+
unsigned int irq;
28+
struct pt_regs *old_regs = set_irq_regs(regs);
2529
trace_hardirqs_off();
26-
handle_arch_irq(regs);
30+
31+
irq_enter();
32+
irq = xintc_get_irq();
33+
next_irq:
34+
BUG_ON(!irq);
35+
generic_handle_irq(irq);
36+
37+
irq = xintc_get_irq();
38+
if (irq != -1U) {
39+
pr_debug("next irq: %d\n", irq);
40+
++concurrent_irq;
41+
goto next_irq;
42+
}
43+
44+
irq_exit();
45+
set_irq_regs(old_regs);
2746
trace_hardirqs_on();
2847
}
2948

drivers/irqchip/irq-xilinx-intc.c

Lines changed: 15 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,20 @@ static unsigned int xintc_get_irq_local(struct xintc_irq_chip *irqc)
124124
return irq;
125125
}
126126

127+
unsigned int xintc_get_irq(void)
128+
{
129+
unsigned int irq = -1;
130+
u32 hwirq;
131+
132+
hwirq = xintc_read(primary_intc, IVR);
133+
if (hwirq != -1U)
134+
irq = irq_find_mapping(primary_intc->root_domain, hwirq);
135+
136+
pr_debug("irq-xilinx: hwirq=%d, irq=%d\n", hwirq, irq);
137+
138+
return irq;
139+
}
140+
127141
static int xintc_map(struct irq_domain *d, unsigned int irq, irq_hw_number_t hw)
128142
{
129143
struct xintc_irq_chip *irqc = d->host_data;
@@ -163,25 +177,6 @@ static void xil_intc_irq_handler(struct irq_desc *desc)
163177
chained_irq_exit(chip, desc);
164178
}
165179

166-
static void xil_intc_handle_irq(struct pt_regs *regs)
167-
{
168-
u32 hwirq;
169-
struct xintc_irq_chip *irqc = primary_intc;
170-
171-
do {
172-
hwirq = xintc_read(irqc, IVR);
173-
if (likely(hwirq != -1U)) {
174-
int ret;
175-
176-
ret = handle_domain_irq(irqc->root_domain, hwirq, regs);
177-
WARN_ONCE(ret, "Unhandled HWIRQ %d\n", hwirq);
178-
continue;
179-
}
180-
181-
break;
182-
} while (1);
183-
}
184-
185180
static int __init xilinx_intc_of_init(struct device_node *intc,
186181
struct device_node *parent)
187182
{
@@ -250,7 +245,7 @@ static int __init xilinx_intc_of_init(struct device_node *intc,
250245
}
251246
} else {
252247
primary_intc = irqc;
253-
set_handle_irq(xil_intc_handle_irq);
248+
irq_set_default_host(primary_intc->root_domain);
254249
}
255250

256251
return 0;

0 commit comments

Comments
 (0)