* [PATCH] microblaze: Fix level/edge irq sensibility
@ 2011-03-30 11:19 Michal Simek
2011-03-30 11:21 ` Michal Simek
0 siblings, 1 reply; 2+ messages in thread
From: Michal Simek @ 2011-03-30 11:19 UTC (permalink / raw)
To: tglx; +Cc: linux-kernel, Michal Simek
Patches:
"microblaze: Convert to new irq function names"
sha (4adc192ec7d977c74c750320f289af9d61c1caca)
and
"microblaze: Use generic show_interrupts()"
sha(9d61c18b25726306c9231428c17db42e3ff29ba7)
should also setup edge/level in irq_set_chip_and_handler_name
name parameter.
Error log:
~ # cat /proc/interrupts
CPU0
2: 2 Xilinx INTC-Xilinx INTC eth0
3: 2 Xilinx INTC-Xilinx INTC eth0
4: 241 Xilinx INTC-Xilinx INTC timer
6: 108 Xilinx INTC-Xilinx INTC serial
Fixed:
~ # cat /proc/interrupts
CPU0
2: 2 Xilinx INTC-edge eth0
3: 2 Xilinx INTC-edge eth0
4: 241 Xilinx INTC-level timer
6: 108 Xilinx INTC-edge serial
Signed-off-by: Michal Simek <monstr@monstr.eu>
---
arch/microblaze/kernel/intc.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/microblaze/kernel/intc.c b/arch/microblaze/kernel/intc.c
index 5ba7e16..71efa51 100644
--- a/arch/microblaze/kernel/intc.c
+++ b/arch/microblaze/kernel/intc.c
@@ -158,11 +158,11 @@ void __init init_IRQ(void)
for (i = 0; i < nr_irq; ++i) {
if (intr_type & (0x00000001 << i)) {
irq_set_chip_and_handler_name(i, &intc_dev,
- handle_edge_irq, intc_dev.name);
+ handle_edge_irq, "level");
irq_clear_status_flags(i, IRQ_LEVEL);
} else {
irq_set_chip_and_handler_name(i, &intc_dev,
- handle_level_irq, intc_dev.name);
+ handle_level_irq, "edge");
irq_set_status_flags(i, IRQ_LEVEL);
}
}
--
1.5.5.6
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] microblaze: Fix level/edge irq sensibility
2011-03-30 11:19 [PATCH] microblaze: Fix level/edge irq sensibility Michal Simek
@ 2011-03-30 11:21 ` Michal Simek
0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2011-03-30 11:21 UTC (permalink / raw)
To: tglx; +Cc: linux-kernel
Michal Simek wrote:
> Patches:
> "microblaze: Convert to new irq function names"
> sha (4adc192ec7d977c74c750320f289af9d61c1caca)
> and
> "microblaze: Use generic show_interrupts()"
> sha(9d61c18b25726306c9231428c17db42e3ff29ba7)
>
> should also setup edge/level in irq_set_chip_and_handler_name
> name parameter.
>
> Error log:
> ~ # cat /proc/interrupts
> CPU0
> 2: 2 Xilinx INTC-Xilinx INTC eth0
> 3: 2 Xilinx INTC-Xilinx INTC eth0
> 4: 241 Xilinx INTC-Xilinx INTC timer
> 6: 108 Xilinx INTC-Xilinx INTC serial
>
> Fixed:
> ~ # cat /proc/interrupts
> CPU0
> 2: 2 Xilinx INTC-edge eth0
> 3: 2 Xilinx INTC-edge eth0
> 4: 241 Xilinx INTC-level timer
> 6: 108 Xilinx INTC-edge serial
>
> Signed-off-by: Michal Simek <monstr@monstr.eu>
> ---
> arch/microblaze/kernel/intc.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/microblaze/kernel/intc.c b/arch/microblaze/kernel/intc.c
> index 5ba7e16..71efa51 100644
> --- a/arch/microblaze/kernel/intc.c
> +++ b/arch/microblaze/kernel/intc.c
> @@ -158,11 +158,11 @@ void __init init_IRQ(void)
> for (i = 0; i < nr_irq; ++i) {
> if (intr_type & (0x00000001 << i)) {
> irq_set_chip_and_handler_name(i, &intc_dev,
> - handle_edge_irq, intc_dev.name);
> + handle_edge_irq, "level");
> irq_clear_status_flags(i, IRQ_LEVEL);
> } else {
> irq_set_chip_and_handler_name(i, &intc_dev,
> - handle_level_irq, intc_dev.name);
> + handle_level_irq, "edge");
oou - vice-versa - this is level and above is edge.
Will send v2
M
--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-03-30 11:21 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-30 11:19 [PATCH] microblaze: Fix level/edge irq sensibility Michal Simek
2011-03-30 11:21 ` Michal Simek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox