From: Ben Nizette <bn@niasdigital.com>
To: linux-kernel <linux-kernel@vger.kernel.org>
Subject: [REGRESSION] threaded interrupt handler support breaks (some) irq handling on AVR32
Date: Tue, 14 Apr 2009 15:46:48 +1000 [thread overview]
Message-ID: <1239688008.29831.2.camel@linux-51e8.site> (raw)
[cc'ing *correct* lkml address]
After pulling in Linus' latest this morning, a tap on the touchscreen of
my AVR32-based board caused the system to hang hard. It's an ads7843
touchscreen controller driven by drivers/input/touchscreen/ads7846.c -
the tap triggers a pen-down IRQ. Note that I made a small mod to that
driver; due to limitations of the AVR32's gpio interrupt controller I
have to do
@@ -1129,7 +1129,7 @@ static int __devinit ads7846_probe(struct spi_device
*spi)
ts->last_msg = m;
- if (request_irq(spi->irq, ads7846_irq, IRQF_TRIGGER_FALLING,
+ if (request_irq(spi->irq, ads7846_irq, IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING,
spi->dev.driver->name, ts)) {
dev_dbg(&spi->dev, "irq %d busy?\n", spi->irq);
err = -EBUSY;
This doesn't effect the driver operation as the irq handler checks the
logic level of the line anyway.
I've bisected the problem to
commit 3aa551c9b4c40018f0e261a178e3d25478dc04a9
Author: Thomas Gleixner <tglx@linutronix.de>
Date: Mon Mar 23 18:28:15 2009 +0100
genirq: add threaded interrupt handler support
Add support for threaded interrupt handlers:
Because this doesn't revert cleanly I did the bisection again just to be
sure and it came out the same.
Just before the hang,
~ # cat /proc/interrupts
CPU0
0: 42 intc avr32_comparator
1: 0 intc atmel_lcdfb
2: 1 intc dw_dmac
4: 4 intc atmel_spi.1
9: 555 intc ttyS0
21: 0 intc rtc
22: 34037 intc tc_clkevt
24: 0 intc atmel_pwm
25: 2439 intc eth0
28: 156 intc atmel_mci.0
31: 0 intc atmel_usba_udc
131: 0 gpio ads7846
Note it's that very last line which, when triggered, causes the hang.
As you can see, it's the only one connected through the gpio interrupt
chip.
No sysrq action is possible because of the limited kinds of terminals I
can attach to this board.
Given the limited scope for debug (no logs or sysrq output available) I
can't think exactly what more I can tell you to help you get to the
bottom of it. I'll do whatever I can though, it's perfectly repeatable.
Thanks,
--Ben.
next reply other threads:[~2009-04-14 5:47 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-14 5:46 Ben Nizette [this message]
[not found] <1239685153.19815.12.camel@linux-51e8.site>
2009-04-14 8:37 ` [REGRESSION] threaded interrupt handler support breaks (some) irq handling on AVR32 Ben Nizette
2009-04-15 1:30 ` Ben Nizette
2009-04-15 7:57 ` Haavard Skinnemoen
2009-04-15 8:33 ` Haavard Skinnemoen
2009-04-15 16:01 ` Bill Gatliff
2009-04-16 8:40 ` Haavard Skinnemoen
2009-04-16 13:25 ` Bill Gatliff
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1239688008.29831.2.camel@linux-51e8.site \
--to=bn@niasdigital.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox