[OpenWrt-Devel] [PATCH v2 6/8] generic: replace request_irq by request_threaded_irq to fix non working ledtrig-gpio

[Guenther Kelleter]

> -----Original Message-----
> From: Felix Fietkau [mailto:nbd at openwrt.org]
> Sent: Monday, July 06, 2015 3:00 PM
> To: Guenther Kelleter; openwrt-devel at lists.openwrt.org
> Subject: Re: [OpenWrt-Devel] [PATCH v2 6/8] generic: replace request_irq by
> request_threaded_irq to fix non working ledtrig-gpio
> 
> On 2015-07-06 14:14, Guenther Kelleter wrote:
> >
> >> -----Original Message-----
> >> From: Felix Fietkau [mailto:nbd at openwrt.org]
> >> Sent: Monday, July 06, 2015 12:43 PM
> >> To: Guenther Kelleter; openwrt-devel at lists.openwrt.org
> >> Subject: Re: [OpenWrt-Devel] [PATCH v2 6/8] generic: replace
> >> request_irq by request_threaded_irq to fix non working ledtrig-gpio
> >>
> >> On 2015-07-02 13:47, Günther Kelleter wrote:
> >> > Signed-off-by: Günther Kelleter <guenther.kelleter at devolo.de>
> >> What issue does this fix?
> >>
> >> - Felix
> >
> > Without this change, gpio_trig_gpio_store() throws this error:
> > [   88.540000] leds devolo:status:dlan: request_irq failed with error -22
> > [   88.550000] leds devolo:error:dlan: request_irq failed with error -22
> If you rework the GPIO IRQ handler code to not require threading, you probably
> won't have to change this generic code.
> 
> - Felix

I don't see how I could rework this. It fails because of nested interrupt handlers.
Someone already tried to solve this issue on lkml
http://lkml.iu.edu/hypermail/linux/kernel/1409.1/00930.html
but this also requires generic code changes.

Günther
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel