Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753682AbbG0Sj1 (ORCPT ); Mon, 27 Jul 2015 14:39:27 -0400 Received: from mail-yk0-f174.google.com ([209.85.160.174]:33386 "EHLO mail-yk0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750803AbbG0SjZ (ORCPT ); Mon, 27 Jul 2015 14:39:25 -0400 MIME-Version: 1.0 In-Reply-To: <55B3904B.3050408@free-electrons.com> References: <1436711211-18223-1-git-send-email-robh@kernel.org> <1436711211-18223-7-git-send-email-robh@kernel.org> <55B3904B.3050408@free-electrons.com> From: Rob Herring Date: Mon, 27 Jul 2015 13:39:05 -0500 X-Google-Sender-Auth: wrMux3z5B8O74QTdPJMnH5XzRWk Message-ID: Subject: Re: [PATCH v2 06/13] irqchip: kill off set_irq_flags usage To: Gregory CLEMENT Cc: "linux-arm-kernel@lists.infradead.org" , Russell King , Krzysztof Kozlowski , "linux-samsung-soc@vger.kernel.org" , "arm@kernel.org" , Alexander Shiyan , Stephen Warren , Lee Jones , "linux-kernel@vger.kernel.org" , Kukjin Kim , linux-rpi-kernel@lists.infradead.org, Thomas Gleixner , Maxime Ripard , Jason Cooper Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2846 Lines: 74 On Sat, Jul 25, 2015 at 8:34 AM, Gregory CLEMENT wrote: > Hi Rob, > > On 12/07/2015 16:26, 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 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 >> Cc: Thomas Gleixner >> Cc: Jason Cooper >> Cc: Kukjin Kim >> Cc: Krzysztof Kozlowski >> Cc: Stephen Warren >> Cc: Lee Jones >> Cc: Alexander Shiyan >> Cc: Maxime Ripard >> Cc: linux-arm-kernel@lists.infradead.org >> Cc: linux-samsung-soc@vger.kernel.org >> Cc: linux-rpi-kernel@lists.infradead.org >> --- >> v2: >> - Fix build error on clps711x >> > [...] > >> diff --git a/drivers/irqchip/irq-armada-370-xp.c b/drivers/irqchip/irq-armada-370-xp.c >> index 0d3b0fe..b8bf8b0 100644 >> --- a/drivers/irqchip/irq-armada-370-xp.c >> +++ b/drivers/irqchip/irq-armada-370-xp.c >> @@ -201,7 +201,6 @@ static int armada_370_xp_msi_map(struct irq_domain *domain, unsigned int virq, >> { >> irq_set_chip_and_handler(virq, &armada_370_xp_msi_irq_chip, >> handle_simple_irq); >> - set_irq_flags(virq, IRQF_VALID); > > OK > >> >> return 0; >> } >> @@ -318,7 +317,7 @@ static int armada_370_xp_mpic_irq_map(struct irq_domain *h, >> irq_set_chip_and_handler(virq, &armada_370_xp_irq_chip, >> handle_level_irq); >> } >> - set_irq_flags(virq, IRQF_VALID | IRQF_PROBE); >> + irq_set_noprobe(virq); > > I think it should be irq_set_probe(virq), I don't see why you inverted the probe flag. Yes, this translation and similar ones are messed up. I've gone back thru and fixed these. However, it is questionable whether you really want to enable probing on these lines or care either way. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/