Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753101Ab3JRXtX (ORCPT ); Fri, 18 Oct 2013 19:49:23 -0400 Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:48735 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751009Ab3JRXtW (ORCPT ); Fri, 18 Oct 2013 19:49:22 -0400 X-Mail-Handler: Dyn Standard SMTP by Dyn X-Originating-IP: 50.131.214.131 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX1+fanWoUdRS6zbEASwiQUAI Date: Fri, 18 Oct 2013 16:49:14 -0700 From: Tony Lindgren To: Randy Dunlap Cc: Linus Walleij , Mark Brown , Thierry Reding , "pinctl-single.c" , "linux-kernel@vger.kernel.org" Subject: Re: linux-next: Tree for Oct 17 (pinctrl-single.c) Message-ID: <20131018234914.GV15154@atomide.com> References: <20131018003847.GH2443@sirena.org.uk> <5261792C.7030203@infradead.org> <20131018233951.GU15154@atomide.com> <5261C72D.1090302@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5261C72D.1090302@infradead.org> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1246 Lines: 36 * Randy Dunlap [131018 16:41]: > On 10/18/13 16:39, Tony Lindgren wrote: > > > > Looks like we need a little bit of arch specific handling > > with the generic IRQ. Fix the issue with an ifdef the > > same way as other drivers do. > > > > ARM needs things set to IRQF_VALID, which also then sets > > noprobe. Others seem to use just irq_set_noprobe(). > > > > Otherwise we can get: > > > > drivers/pinctrl/pinctrl-single.c: In function 'pcs_irqdomain_map': > > drivers/pinctrl/pinctrl-single.c:1750:2: error: implicit declaration of function 'set_irq_flags' [-Werror=implicit-function-declaration] > > Did you address this error? I don't see a fix for it... Yes set_irq_flags() is defined for ARM archs in arch/arm/include/asm/hw_irq.h which is included from include/linux/irq.h as . So the ifdef below takes care of that too. > > +#ifdef CONFIG_ARM > > + set_irq_flags(irq, IRQF_VALID); > > +#else > > + irq_set_noprobe(irq); > > +#endif Regards, Tony -- 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/