Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030599AbcCQPGk (ORCPT ); Thu, 17 Mar 2016 11:06:40 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:14961 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935868AbcCQPGh (ORCPT ); Thu, 17 Mar 2016 11:06:37 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Thu, 17 Mar 2016 08:05:06 -0700 Subject: Re: [PATCH 07/15] irqdomain: Don't set type when mapping an IRQ To: Thomas Gleixner References: <1458224359-32665-1-git-send-email-jonathanh@nvidia.com> <1458224359-32665-8-git-send-email-jonathanh@nvidia.com> CC: Jason Cooper , Marc Zyngier , =?UTF-8?Q?Beno=c3=aet_Cousson?= , Tony Lindgren , 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: <56EAC7F6.8000504@nvidia.com> Date: Thu, 17 Mar 2016 15:06:30 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: X-Originating-IP: [10.21.132.114] X-ClientProxiedBy: UKMAIL102.nvidia.com (10.26.138.15) To UKMAIL101.nvidia.com (10.26.138.13) 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 Content-Length: 683 Lines: 23 On 17/03/16 14:55, Thomas Gleixner wrote: > On Thu, 17 Mar 2016, Jon Hunter wrote: >> --- 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); >> + > > This change is independent of the irqdomain part and should be in a seperate > patch because it affects all callers. Ok, I will put this into a separate patch. Cheers Jon