From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754880Ab2HAKw2 (ORCPT ); Wed, 1 Aug 2012 06:52:28 -0400 Received: from smtp-out-181.synserver.de ([212.40.185.181]:1045 "EHLO smtp-out-178.synserver.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753874Ab2HAKw1 (ORCPT ); Wed, 1 Aug 2012 06:52:27 -0400 X-SynServer-TrustedSrc: 1 X-SynServer-AuthUser: lars@metafoo.de X-SynServer-PPID: 20933 Message-ID: <50190B67.5050108@metafoo.de> Date: Wed, 01 Aug 2012 12:56:39 +0200 From: Lars-Peter Clausen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.5) Gecko/20120624 Icedove/10.0.5 MIME-Version: 1.0 To: Borislav Petkov , Thierry Reding , Steven Rostedt , Dave Jones , Linux Kernel Subject: Re: awful kconfig help texts. References: <20120731151600.GA19412@redhat.com> <20120731170741.GA4735@x1.osrc.amd.com> <20120731172638.GE16873@home.goodmis.org> <20120731174249.GB4735@x1.osrc.amd.com> <20120801074742.GH29673@avionic-0098.adnet.avionic-design.de> <20120801085644.GA21303@liondog.tnic> <5018F537.209@metafoo.de> <20120801092625.GB21303@liondog.tnic> <5018F908.9090607@metafoo.de> <20120801100455.GD21303@liondog.tnic> In-Reply-To: <20120801100455.GD21303@liondog.tnic> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/01/2012 12:04 PM, Borislav Petkov wrote: > On Wed, Aug 01, 2012 at 11:38:16AM +0200, Lars-Peter Clausen wrote: >> You don't see any drivers, because the subsystem is still young and no >> such arch independent drivers have been added yet, but they will get >> added in the future. The arch independent companion or PWM expander >> chips usually interface via I2S or SPI and I would consider it quite >> likely that you'll also find them on some embedded X86 boards. If we >> add a arch restriction to the config item now we'd quite likely have >> to remove it again in the next release. > > Yes please. > > Kconfig is overcrowded as it is now and adding yet another option which > is irrelevant for some arches (for now, as you say) simply causes > confusion to people with absolutely no gain. > > Simply take a look at all arch//Kconfig files and look at all > the "select ..." statements right at the beginning of the respective > Kconfig file. Yes and these select statements make sense, what you suggest though doesn't, at least from my point of view. What you want is that you don't get presented the option to select the PWM system if there no PWM driver available based on your other config options. But whether a PWM driver is available or not is not a issue of which arch you are building for. You could do that by using a construct like below, but well... config HAS_PWM bool config PWM bool "PWM depends on HAS_PWM config PWM_CAN_BUILD_DRIVER_X defbool I2C select HAS_PWM config PWM_DRIVER_X tristate "PWM chip X support" depends on PWM && PWM_CAN_BUILD_DRIVER_X config PWM_CAN_BUILD_DRIVER_Y defbool ARCH_Y select HAS_PWM config PWM_DRIVER_Y tristate "PWM chip Y support" depends on PWM && PWM_CAN_BUILD_DRIVER_Y