From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_PASS,USER_AGENT_NEOMUTT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DEB56C43381 for ; Wed, 6 Mar 2019 20:23:03 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id ADB3B206DD for ; Wed, 6 Mar 2019 20:23:03 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="UMWrZFm4"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="xaUnYiOU" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org ADB3B206DD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=armlinux.org.uk Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=iGCqg6mC6+j77ae0o9n1BRTwEO5fylA+NWYHSb1q8PM=; b=UMWrZFm4/lDINu GZsKz+Ku0UHFnd+Zq6wFfM6ahamnleWDTIhOId/AeCFlZHwyhqjFDbACYK7JSSIgQ0DUBlDDhb0Ko 3gyHsTXsXg8jp+qKm+aVr+kitiKc+FH8q79CHGYUeii60oTBHe2teqo+KhG4MxDsTxECT5Xr793Ib fQ/Bak+thRHc4pEyN5gZTEWrBgAILe6LruKzhOME8PTr4VdbQZqUVr8RP50IiBGeRw9lfpwralry6 0mFd1RdCnzff2ZPS6YfpjBD3fy1YwhIdgo14IgQmc0JDLOvjN7d2gjk7nEMav63KHCat0ezJ1t85g H2NDH7k8ypJRt6SdEHdw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1h1d43-0001sD-90; Wed, 06 Mar 2019 20:22:55 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:3201:214:fdff:fe10:1be6]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h1d3x-0001rH-0a for linux-arm-kernel@lists.infradead.org; Wed, 06 Mar 2019 20:22:52 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=y5ZURR5r/3LLx6aWN+bszh3KYmschxGAEvSeXhlTRzs=; b=xaUnYiOUXr98+jAF+wCQ2UQ03 W/nBn1C9oLDm0V/ziHRHpOug/LC2bYOZtA3oreola8hBtc7WnuBHPf4/uoaBUl4DPrRyJhMRgvKFg vI48mThBzaEvWfwxO+WamgfDqgh05XV9GPkLtXtUtbAKK9ugn+q0gq5OqSXhavtr2VcDXS6r52BSV eBDr2YtkfcHprcVS4nzRY3q26rhD2zKJBA/LnaaHgHj7jAU/Jz7C3aKAsFIIjVB3gJWE1Ynn3vEO0 gg5rdts0TI6VUfJexn0q2tTwNE6HuuLhrcIuu5h7/xLfwUnaD7GtIeYqMVmJCWOqKX6gPozN5pqfA Qtm/U36GQ==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:51636) by pandora.armlinux.org.uk with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) (envelope-from ) id 1h1d3c-0001QL-22; Wed, 06 Mar 2019 20:22:28 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.89) (envelope-from ) id 1h1d3V-0004Ka-6r; Wed, 06 Mar 2019 20:22:21 +0000 Date: Wed, 6 Mar 2019 20:22:21 +0000 From: Russell King - ARM Linux admin To: "Enrico Weigelt, metux IT consult" Subject: Re: Smarter Kconfig help Message-ID: <20190306202221.n654yxedjtdang5n@shell.armlinux.org.uk> References: <20190305173111.t3n6ycktht7kxuhq@shell.armlinux.org.uk> <1551865552.9298.24.camel@pengutronix.de> <20190306095147.xyf3d62bywx4v6ji@shell.armlinux.org.uk> <20190306113436.5ucdjppibwl6pahz@shell.armlinux.org.uk> <20190306124221.bf6qj6zx6jbdcsey@shell.armlinux.org.uk> <80d7c75a-f537-ee10-a139-d342c6b0f2b4@metux.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <80d7c75a-f537-ee10-a139-d342c6b0f2b4@metux.net> User-Agent: NeoMutt/20170113 (1.7.2) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190306_122249_058380_AECC4EC0 X-CRM114-Status: GOOD ( 16.88 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Peng Hao , Greg Kroah-Hartman , Linux Kernel Mailing List , Andy Shevchenko , linux-arm Mailing List , Lucas Stach Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Mar 06, 2019 at 09:16:02PM +0100, Enrico Weigelt, metux IT consult wrote: > On 06.03.19 13:42, Russell King - ARM Linux admin wrote: > > > In case it isn't clear, this is the *exact* point here - I don't know> whether this option should be enabled for iMX6 or not, and the only> > way I found out was to grep the dts files in arch/arm/boot/dts for> the > driver's compatible string. What that reveals is that *no* 32-bit> dts > files contain the compatible string, and so I summise that *no*> 32-bit > iMX SoC should have this driver enabled. > The problem is a bit more generic. I often have to spend lots of time > to find out which configs to enable on a specific board, to get certain > features (eg. network, sata, display, gpu, ...). Even worse: many > options require other stuff enabled before even showing up. And when > disabling unneeded stuff, it leaves lots of other things enabled. > (we don't have some `apt autoremove` kconfig counterpart :(). > > I've decided to cope w/ that on a higher level and written a little > config generator tool for that - here you can enable high level > features (eg. 'network' or 'display', etc) and it will generate the > actual .config: > > https://github.com/metux/kmct > > > The excuse that "we can't list the explicit SoCs" to me seems to be> a very lame excuse > > Maybe this actually means "nobody here volounteered to actually maintain > these help texts" ? > > > The best that I can come up with right now, given what little I know> from grepping the 32-bit DTS files, is that the help text should at> > least indicate that it *isn't* applicable to 32-bit SoCs, or if you> > prefer, *is* applicable to 64-bit SoCs. Beyond that, I have no> > information to formulate a better suggestion. > Perhaps just fix the text based on your knowledge and send a patch to > the maintainers. They'll propably tell you if it's incorrect. Frankly, no. I don't want to be going round endlessly writing help texts. We need the effort to be properly distributed - we need those who _know_ the feature they're developing to write a proper help text. One way to achieve that is to make a proper informative help text a pre-requisit of accepting the code. The quality of the help text is just as important as the quality of the code, and we really should be paying the same amount of attention to both. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel