Re: [PATCH] genirq: fix trigger flags check for shared irqs
From: Thomas Gleixner
Date: Tue Jan 26 2016 - 15:46:39 EST
On Tue, 26 Jan 2016, Brian Starkey wrote:
> For shared interrupts, if one requester passes in any IRQF_TRIGGER_*
> flags whilst another doesn't, __setup_irq() can erroneously fail.
>
> The no-flags case should be treated as "already configured", so change
> __setup_irq() to only check that the flags match if any have been
> provided.
What happens if that "already configured", i.e. the default setting, is
conflicting with the newly requested interrupt?
I rather prefer the failure than the resulting silent wreckage.
Thanks,
tglx