Re: [PATCH] genirq: Skip chained interrupt trigger configuration if type is IRQ_TYPE_NONE

From: Geert Uytterhoeven
Date: Mon Sep 19 2016 - 05:22:22 EST

Hi Marc,

On Mon, Sep 19, 2016 at 10:49 AM, Marc Zyngier <marc.zyngier@xxxxxxx> wrote:
> There is no point in trying to configure the trigger of a chained
> interrupt if no trigger information has been configured. At best
> this is ignored, and at the worse this confuses the underlying
> irqchip (which is likely not to handle such a thing), and
> unnecessarily alarms the user.
> Only apply the configuration if type is not IRQ_TYPE_NONE.


> Reported-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Fixes: 1e12c4a9393b ("genirq: Correctly configure the trigger on chained interrupts")
> Link:
> Signed-off-by: Marc Zyngier <marc.zyngier@xxxxxxx>

Tested-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>



Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds