Hello Linus, On 8/28/20 12:55 AM, Linus Walleij wrote: > Hi Ahmad, > > On Tue, Aug 4, 2020 at 11:18 AM Ahmad Fatoum <a.fat...@pengutronix.de> wrote: > >> Generic GPIO consumers like gpio-keys use request_any_context_irq() >> to request a threaded handler if irq_settings_is_nested_thread() == >> true or a hardirq handler otherwise. >> >> Drivers using handle_nested_irq() must be sure that the nested >> IRQs were requested with threaded handlers, because the IRQ >> is handled by calling action->thread_fn(). >> >> The gpio-siox driver dispatches IRQs via handle_nested_irq, >> but has irq_settings_is_nested_thread() == false. >> >> Set gpio_irq_chip::threaded to remedy this. >> >> Signed-off-by: Ahmad Fatoum <a.fat...@pengutronix.de> > > I think we concluded we want to apply this patch, but with > a fixed commit text, can you send a V2? (Or ask Uwe if he wants > to pick it up and write the text.)
Thanks for the reminder. This slipped through. I just sent out v2 (<20200907153135.3307-1-a.fat...@pengutronix.de>) Cheers, Ahmad > > Yours, > Linus Walleij > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
