From patchwork Mon Feb 24 18:41:20 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Josh Cartwright X-Patchwork-Id: 3711371 Return-Path: X-Original-To: patchwork-linux-arm-msm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 59946BF13A for ; Mon, 24 Feb 2014 18:44:18 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 818DC20121 for ; Mon, 24 Feb 2014 18:44:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 975B220149 for ; Mon, 24 Feb 2014 18:44:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752477AbaBXSn5 (ORCPT ); Mon, 24 Feb 2014 13:43:57 -0500 Received: from smtp.codeaurora.org ([198.145.11.231]:55353 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752212AbaBXSn4 (ORCPT ); Mon, 24 Feb 2014 13:43:56 -0500 Received: from smtp.codeaurora.org (localhost [127.0.0.1]) by smtp.codeaurora.org (Postfix) with ESMTP id 021AC13ED82; Mon, 24 Feb 2014 18:43:56 +0000 (UTC) Received: by smtp.codeaurora.org (Postfix, from userid 486) id E6DF913EFD1; Mon, 24 Feb 2014 18:43:55 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 Received: from joshc.qualcomm.com (rrcs-67-52-129-61.west.biz.rr.com [67.52.129.61]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: joshc@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 81CEE13ED82; Mon, 24 Feb 2014 18:43:55 +0000 (UTC) Received: by joshc.qualcomm.com (Postfix, from userid 1000) id 5600B63536; Mon, 24 Feb 2014 12:41:20 -0600 (CST) Date: Mon, 24 Feb 2014 12:41:20 -0600 From: Josh Cartwright To: Randy Dunlap , Thomas Gleixner Cc: Bjorn Andersson , Linus Walleij , Stephen Rothwell , "linux-next@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-arm-msm@vger.kernel.org Subject: Re: linux-next: Tree for Feb 17 (pinctrl-msm) Message-ID: <20140224184120.GB18563@joshc.qualcomm.com> References: <20140217172329.2c61499f57842ec2ed177bc5@canb.auug.org.au> <530247FC.90606@infradead.org> <20140224174646.GA32621@sonymobile.com> <530B8C15.50100@infradead.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <530B8C15.50100@infradead.org> User-Agent: Mutt/1.5.21 (2012-12-30) X-Virus-Scanned: ClamAV using ClamSMTP Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Mon, Feb 24, 2014 at 10:14:45AM -0800, Randy Dunlap wrote: > On 02/24/2014 09:46 AM, Bjorn Andersson wrote: > > On Mon 24 Feb 06:46 PST 2014, Linus Walleij wrote: > >> On Mon, Feb 17, 2014 at 6:33 PM, Randy Dunlap wrote: > >>> On 02/16/2014 10:23 PM, Stephen Rothwell wrote: > >>> ERROR: "handle_bad_irq" [drivers/pinctrl/pinctrl-msm.ko] undefined! > >> > >> Weird, Björn do you know what may be causing this? > > > > I include linux/irq.h to get hold of handle_bad_irq() and as far as I can see > > there are no changes related to that in linux-next. > > > > Do we know if the build still fails? Or if something changed? > > > > > > Please let me know if I should pull down the -next tree and try to reproduce > > this. > > I guess that you get off free on this one since I didn't post the failing > .config file and I have already deleted it. > > I'm trying to reproduce it now but it's slow going. I'll let you know if > I get it to fail again. Without too much effort, I can get this to fail just by making CONFIG_PINCTRL_MSM=m. handle_bad_irq isn't marked EXPORT_SYMBOL*, so hence the warning. Whether or not this is intentional is not clear. Do we support modules installing chained irq handlers? For now, the patch below just makes this driver 'bool' instead of 'tristate'. Josh --8<-- Subject: [PATCH] pinctrl: msm: make PINCTRL_MSM bool instead of tristate Modular builds of pinctrl-msm break due to handle_bad_irq being unexported for module use. For now, make PINCTRL_MSM 'bool'. Signed-off-by: Josh Cartwright Reviewed-by: Bjorn Andersson --- drivers/pinctrl/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig index be361b7..1e4e693 100644 --- a/drivers/pinctrl/Kconfig +++ b/drivers/pinctrl/Kconfig @@ -217,7 +217,7 @@ config PINCTRL_IMX28 select PINCTRL_MXS config PINCTRL_MSM - tristate + bool select PINMUX select PINCONF select GENERIC_PINCONF