[PATCH v4 1/6] mfd: fsl imx25 Touchscreen ADC driver

Linus Walleij linus.walleij at linaro.org
Mon Aug 17 02:11:17 PDT 2015


On Fri, Jun 27, 2014 at 11:18 AM, Linus Walleij
<linus.walleij at linaro.org> wrote:
> On Tue, Jun 24, 2014 at 4:46 PM, Denis Carikli <denis at eukrea.com> wrote:

>> +       irq_set_chip_and_handler(irq, &dummy_irq_chip,
>> +                                handle_level_irq);
>> +       set_irq_flags(irq, IRQF_VALID);
>
> This function only exists on ARM. Unless Kconfig ensures
> this is only available on ARM platforms, use
>
> #ifdef CONFIG_ARM
>       set_irq_flags(irq, IRQF_VALID);
> #else
>       irq_set_noprobe(irq);
> #endif
>
> (Sorry we have no better way of doing it.)

Since I wrote this, there have been cleanups from Rob Herring merged
to the kernel to get rid of this oddity. Please check in linux-next how
to get this right, see c.f. commit
e8d36d5dbb6a6ec4f5222f8775d664ec29d5527d
"ARM: kill off set_irq_flags usage"

Yours,
Linus Walleij



More information about the linux-arm-kernel mailing list