On Tue, 9 Jun 2015 13:26:29 -0500 Rob Herring <[email protected]> wrote:
> set_irq_flags is ARM specific with custom flags which have genirq > equivalents. Convert drivers to use the genirq interfaces directly, so we > can kill off set_irq_flags. The translation of flags is as follows: > > IRQF_VALID -> !IRQ_NOREQUEST > IRQF_PROBE -> !IRQ_NOPROBE > IRQF_NOAUTOEN -> IRQ_NOAUTOEN > > For IRQs managed by an irqdomain, the irqdomain core code handles clearing > and setting IRQ_NOREQUEST already, so there is no need to do this in > .map() functions and we can simply remove the set_irq_flags calls. Some > users also set IRQ_NOPROBE and this has been maintained although it is not > clear that is really needed. There appears to be a great deal of blind > copy and paste of this code. > > Signed-off-by: Rob Herring <[email protected]> Acked-by: Boris Brezillon <[email protected]> > Cc: Boris Brezillon <[email protected]> > Cc: Mike Turquette <[email protected]> > Cc: Stephen Boyd <[email protected]> > Cc: [email protected] > --- > drivers/clk/at91/pmc.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/clk/at91/pmc.c b/drivers/clk/at91/pmc.c > index 3f27d21..828b196 100644 > --- a/drivers/clk/at91/pmc.c > +++ b/drivers/clk/at91/pmc.c > @@ -125,7 +125,6 @@ static int pmc_irq_map(struct irq_domain *h, unsigned int > virq, > > irq_set_chip_and_handler(virq, &pmc_irq, > handle_level_irq); > - set_irq_flags(virq, IRQF_VALID); > irq_set_chip_data(virq, pmc); > > return 0; -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
