From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965386AbcFMLJW (ORCPT ); Mon, 13 Jun 2016 07:09:22 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:16224 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965088AbcFMLJT (ORCPT ); Mon, 13 Jun 2016 07:09:19 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Mon, 13 Jun 2016 04:05:52 -0700 Subject: Re: [PATCH V6 2/9] genirq: Look-up trigger type if not specified by caller To: Marc Zyngier , Thomas Gleixner , Jason Cooper References: <1465312354-27778-1-git-send-email-jonathanh@nvidia.com> <1465312354-27778-3-git-send-email-jonathanh@nvidia.com> <575E8E29.3090808@arm.com> CC: Rob Herring , Pawel Moll , "Mark Rutland" , Ian Campbell , Kumar Gala , Stephen Warren , Thierry Reding , Kevin Hilman , Geert Uytterhoeven , Grygorii Strashko , Lars-Peter Clausen , "Linus Walleij" , , , From: Jon Hunter Message-ID: <575E9457.8010100@nvidia.com> Date: Mon, 13 Jun 2016 12:09:11 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: <575E8E29.3090808@arm.com> X-Originating-IP: [10.21.132.103] X-ClientProxiedBy: UKMAIL102.nvidia.com (10.26.138.15) To UKMAIL102.nvidia.com (10.26.138.15) Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 13/06/16 11:42, Marc Zyngier wrote: > On 07/06/16 16:12, Jon Hunter wrote: >> For some devices the IRQ trigger type for a device is read from >> firmware, such as device-tree. The IRQ trigger type is typically read >> when the mapping for IRQ is created, which is before the IRQ is >> requested. Hence, the IRQ trigger type is programmed when mapping the >> IRQ and not when requesting the IRQ. >> >> Although this works for most cases, in order to support IRQ chips which >> require runtime power management, which may not be accessible prior >> to requesting the IRQ, it is desirable to look-up the IRQ trigger type >> when it is requested. Therefore, if the IRQ trigger type is not >> specified when __setup_irq() is called, look-up the saved IRQ trigger >> type. This will allow us to defer the programming of the trigger type >> from when the IRQ is mapped to when it is actually requested. >> >> Signed-off-by: Jon Hunter >> Reviewed-by: Marc Zyngier >> --- >> kernel/irq/manage.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c >> index ef0bc02c3a70..eaedeb74b49d 100644 >> --- a/kernel/irq/manage.c >> +++ b/kernel/irq/manage.c >> @@ -1117,6 +1117,13 @@ __setup_irq(unsigned int irq, struct irq_desc *desc, struct irqaction *new) >> new->irq = irq; >> >> /* >> + * If the trigger type is not specified by the caller, >> + * then use the default for this interrupt. >> + */ >> + if (!(new->flags & IRQF_TRIGGER_MASK)) >> + new->flags |= irqd_get_trigger_type(&desc->irq_data); >> + >> + /* >> * Check whether the interrupt nests into another interrupt >> * thread. >> */ >> > > I've added the following patch to the queue, in order to deal with > percpu interrupts that were not handled by this patch: > > diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c > index f8fd1fb..00cfc85 100644 > --- a/kernel/irq/manage.c > +++ b/kernel/irq/manage.c > @@ -1751,7 +1751,14 @@ void enable_percpu_irq(unsigned int irq, unsigned int type) > if (!desc) > return; > > + /* > + * If the trigger type is not specified by the caller, then > + * use the default for this interrupt. > + */ > type &= IRQ_TYPE_SENSE_MASK; > + if (type == IRQ_TYPE_NONE) > + type = irqd_get_trigger_type(&desc->irq_data); > + I am wondering if you need this here because now __setup_irq(), called by request_percpu_irq(), will actually look-up the saved type even for a percpu-irq. So shouldn't this already be handled? Or am I missing something? Cheers Jon -- nvpublic