From mboxrd@z Thu Jan 1 00:00:00 1970 From: olof@lixom.net (Olof Johansson) Date: Tue, 28 Jul 2015 14:00:53 +0200 Subject: [PATCH v3] ARM: kill off set_irq_flags usage In-Reply-To: <1438030523-15853-1-git-send-email-robh@kernel.org> References: <1438030523-15853-1-git-send-email-robh@kernel.org> Message-ID: <20150728120053.GB21866@localhost> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Jul 27, 2015 at 03:55:13PM -0500, Rob Herring 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 modify 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 > Cc: Russell King > Cc: Sekhar Nori > Cc: Kevin Hilman > Cc: Jason Cooper > Cc: Andrew Lunn > Cc: Sebastian Hesselbarth > Cc: Gregory Clement > Acked-by: Hans Ulli Kroll > Acked-by: Shawn Guo > Cc: Sascha Hauer > Cc: Imre Kaloz > Acked-by: Krzysztof Halasa > Cc: Greg Ungerer > Cc: Roland Stigge > Cc: Tony Lindgren > Cc: Daniel Mack > Cc: Haojian Zhuang > Cc: Robert Jarzmik > Cc: Simtec Linux Team > Cc: Kukjin Kim > Cc: Krzysztof Kozlowski > Acked-by: Wan ZongShun > Cc: linux-arm-kernel at lists.infradead.org > Cc: linux-omap at vger.kernel.org > Cc: linux-samsung-soc at vger.kernel.org > Tested-by: Kevin Hilman > --- > Thomas asked that this be merged thru subsystem trees instead of arm-soc, > so please apply just this patch to your tree. "this be merged through subsystem trees" + "please apply just this patch". I'm going to assume that the first "this" means a patch series. So I've applied this to next/cleanup, let me know if this was in error and should be removed, it's easy to drop at this time. -Olof