From mboxrd@z Thu Jan 1 00:00:00 1970 From: Robert Jarzmik Subject: Re: [PATCH v2 01/13] ARM: kill off set_irq_flags usage Date: Tue, 14 Jul 2015 23:14:45 +0200 Message-ID: <87pp3ujvay.fsf@belgarion.home> References: <1436711211-18223-1-git-send-email-robh@kernel.org> <1436711211-18223-2-git-send-email-robh@kernel.org> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <1436711211-18223-2-git-send-email-robh@kernel.org> (Rob Herring's message of "Sun, 12 Jul 2015 09:26:39 -0500") Sender: linux-kernel-owner@vger.kernel.org To: Rob Herring Cc: linux-arm-kernel@lists.infradead.org, Russell King , linux-kernel@vger.kernel.org, arm@kernel.org, Sekhar Nori , Kevin Hilman , Jason Cooper , Andrew Lunn , Sebastian Hesselbarth , Gregory Clement , Shawn Guo , Sascha Hauer , Imre Kaloz , Greg Ungerer , Roland Stigge , Tony Lindgren , Daniel Mack , Haojian Zhuang , Simtec Linux Team , Kukjin Kim , Krzysztof Kozlowski , linux-omap@vger.kernel.org, linux-samsung-soc@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org Rob Herring writes: > 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. As this works for several pxa boards : Acked-by: Robert Jarzmik -- Robert