From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751977AbcF0Vea (ORCPT ); Mon, 27 Jun 2016 17:34:30 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:56943 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751644AbcF0Ve1 (ORCPT ); Mon, 27 Jun 2016 17:34:27 -0400 From: Arnd Bergmann To: Randy Dunlap Cc: Stephen Rothwell , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, "devicetree@vger.kernel.org" , Linus Torvalds , "Linus Walleij (linus.walleij@linaro.org)" , Alexander Stein Subject: Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Date: Mon, 27 Jun 2016 23:36:29 +0200 Message-ID: <735717690.oeI6DkomRc@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-22-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <57716D3D.4070302@infradead.org> References: <20160627163939.20e454d5@canb.auug.org.au> <57716D3D.4070302@infradead.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:7lfRRqgUpj6zjFnxAn8R5zdEPIG5o4INh3iovkOScqo//uY80EV wdDUEHaMozNoLIEOCymTIi0cAM/DxOz7J82eQyNPqHqoEuLzAT+44G8zIwDopqBcs05E+h+ CcTyvf87B+joKzgGn7sUMYy9OXIKpYVvDhAZAw6EF5gBvnPWREkZuOH4GksYcWufiNaQ2zR 2dLFWE3iXT4GbmtQlfA4A== X-UI-Out-Filterresults: notjunk:1;V01:K0:h2ptMQ5MAUs=:Mz/liAl7GnLiQFIssmCFNJ hUEmr0tRAuoKDSg7YwzFAIpTw2MUmgzvuNZbRRoYXXeQvs3ocVKfe7HPbuKQv4A2T4uDFteI6 NdsaU1pnhgCxvJqMz/HwsW+Uj55V/+5LhJq1rlf9sR/+fKxNCI8EuHrMzA3uKP3VqT0h1O3Y3 vbTYXzyOmJi297wNwG1BroSufoTKDnYdq/OI8ljWrXBj+z/ecCvg0kdP5DOlDFcwm+JOk8DWj NljiK/lZt72KPi4Zpy1oVGMvWm/amSNNGZQFl5Q3kpZaI7c6I8Zx7s2MUcmgk8xhW8sIXDdoU DeTskx2UnncyTiQerF+x/4xxiqOqs5zonU4MdGjIJcDpY+Gbnbjay+/7qiK+8PtMAkOHKYEIi LOWxw13x97MqeYF34ohvHk4vrwZf4IDVCkJfShR2fJfohd6VRKi8VJMcGash4SrmETj2RjIdv ojN03qKbJCRs+1ZwsARwWgvv2Xa5EPTyfQgUqO/FYmATZV5iOF2yB7ot7ZPoe3giNTrBpClUM mL2xzbUieZ6CPyGbEILP8AlPS70O88TJd0sxzRgJXWveNBuRIGedVKsbd+sTF5u8qyjWs5Zjp Iy6bpZEHeWuLOvg4oSitj3rui+H1wq0IXFOSepqnt0Mj/BclwFn34aQv0Ei6H1IfvC5xE9jtW 6prJOF/tfK/9e/LP6/v7LZeucBTxV4/E8N9P5skF90E1IUNPwLxU8b+N7+xpjNNHA4UzxOsEb P4opHXtDpaw8efnN Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday, June 27, 2016 11:15:25 AM CEST Randy Dunlap wrote: > On 06/26/16 23:39, Stephen Rothwell wrote: > > Hi all, > > > > Changes since 20160624: > > > > on i386, when CONFIG_OF is not enabled ... > but OF_GPIO is enabled due to this in drivers/gpio/Kconfig: > > config OF_GPIO > def_bool y > depends on OF || COMPILE_TEST > > (above from commit 1e4a80640338924b9f9fd7a121ac31d08134410a > from Alexander Stein ) > > > ../drivers/pinctrl/bcm/pinctrl-iproc-gpio.c:381:20: error: 'pinconf_generic_dt_node_to_map_pin' undeclared here (not in a function) > ../drivers/pinctrl/bcm/pinctrl-cygnus-mux.c:739:20: error: 'pinconf_generic_dt_node_to_map_group' undeclared here (not in a function) > ../drivers/pinctrl/bcm/pinctrl-nsp-gpio.c:365:20: error: 'pinconf_generic_dt_node_to_map_pin' undeclared here (not in a function) > > because that function is only present when CONFIG_OF is enabled. > > > Also, why does that commit (1e4a80640338924b9f9fd7a121ac31d08134410a) > not have any other S-O-B lines in it? like whoever merged it? (adding Linus Walleij and Alexander Stein to Cc) Unfortunately, the changelog also doesn't say what triggered that change. I guess there was a driver that did 'select OF_GPIO' and had a dependency on COMPILE_TEST, though I don't see any driver doing that in linux-next. I think the commit should just be reverted, it clearly breaks stuff, and whatever needs it can be fixed in a better way. Specifically, we have three ways that the OF_GPIO symbol is used: a) it is implicitly enabled when both GPIOLIB and OF are turned on b) some drivers 'depends on OF_GPIO' c) some other drivers 'select OF_GPIO' The combination of b) and c) can easily lead to circular dependencies, and is also confusing. It would be better to change all of those to 'depends on'. Arnd