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=-2.8 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS,USER_AGENT_NEOMUTT autolearn=ham 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 47707C43381 for ; Wed, 6 Mar 2019 12:42:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 1769C204EC for ; Wed, 6 Mar 2019 12:42:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=armlinux.org.uk header.i=@armlinux.org.uk header.b="IlMxG5Mq" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730076AbfCFMmg (ORCPT ); Wed, 6 Mar 2019 07:42:36 -0500 Received: from pandora.armlinux.org.uk ([78.32.30.218]:59524 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728734AbfCFMmf (ORCPT ); Wed, 6 Mar 2019 07:42:35 -0500 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=5kfA31xI0xJ+P5rKkFZq8MZQMao/Gzu69qWDGP8i5HU=; b=IlMxG5MqtqDmbpStLv3ZWCDHL RnIxWPdbRlm6qvQH9BcVzLwHciY7azWkj+KgrZzZ3tZYghfPxo90RXdFFq8kXQDH4DoY9Wruafy+T vUgtBrhF/IT3830UvM8/PrbJkWCBAemhGpEMAPmgrRwMdnRq0jRX1ZQoFpyGfs8semlKumDgjqzit LsdhN/+1+bOACoS1oi0HOb+Z0pHiyYpIcKklVqMe+1rhMko/dCbvuymiDQx6hbocNz1MdRbcJVL5d ZErLoYbKu1vd9Ug8hPVF4IZXtp/tf7rOUlhdM5ZTlBcShNs2UOkC9fPCK6D3azZ9MH0L/JuelaQSw JKNuyunMw==; Received: from shell.armlinux.org.uk ([2002:4e20:1eda:1:5054:ff:fe00:4ec]:37280) by pandora.armlinux.org.uk with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) (envelope-from ) id 1h1VsR-0007hO-7X; Wed, 06 Mar 2019 12:42:27 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.89) (envelope-from ) id 1h1VsL-00044M-Sc; Wed, 06 Mar 2019 12:42:21 +0000 Date: Wed, 6 Mar 2019 12:42:21 +0000 From: Russell King - ARM Linux admin To: Andy Shevchenko Cc: Mark Rutland , Peng Hao , Greg Kroah-Hartman , Linux Kernel Mailing List , linux-arm Mailing List , Lucas Stach Subject: Re: Smarter Kconfig help Message-ID: <20190306124221.bf6qj6zx6jbdcsey@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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190306113436.5ucdjppibwl6pahz@shell.armlinux.org.uk> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 06, 2019 at 11:34:36AM +0000, Russell King - ARM Linux admin wrote: > On Wed, Mar 06, 2019 at 12:49:40PM +0200, Andy Shevchenko wrote: > > On Wed, Mar 6, 2019 at 11:52 AM Russell King - ARM Linux admin > > wrote: > > > On Wed, Mar 06, 2019 at 10:45:52AM +0100, Lucas Stach wrote: > > > > Am Dienstag, den 05.03.2019, 17:31 +0000 schrieb Russell King - ARM Linux admin: > > > > > > While I totally agree that the irqsteer driver should only default Y on > > > > 64bit i.MX SoCs, I'm not really sure what to do about the Kconfig help > > > > text. > > > > > > > > Enumerating SoCs in the Kconfig of a IP block driver is always prone to > > > > get outdated. Just as the first example I've been able to come up with, > > > > the MMC_SDHCI_ESDHC_IMX help text says: "This selects the Freescale > > > > eSDHC/uSDHC controller support found on i.MX25, i.MX35 i.MX5x and > > > > i.MX6x.", while the same IP block is to be found on i.MX7 and various > > > > i.MX8. > > > > > > > > For the Kconfig user, who needs to decide if an option is relevant, > > > > outdated SoC information is probably just as bad as no information at > > > > all. > > > > > > How about "This option does not apply to AArch32 based SoCs" or > > > > Negative is usually error prone. > > > > > "This option should be enabled for i.MX7 and later SoCs" ? > > > > Why it should be? It can / could be I guess. > > I've no idea. I'm just making suggestions - if you have anything > better, let's hear it. 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. Given that I don't know for certain, _I_ can't write a proper help text for this - I can only make suggestions which _might_ fit what the reality actually is. The excuse that "we can't list the explicit SoCs" to me seems to be a very lame excuse for a poor, one line help text that says absolutely nothing that can't already be gathered from the option line itself. So the current help text might as well be deleted - it's that "useful". 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. This is precisely why I say that we need to be smarter about Kconfig help text: if it gives no useful information, it might as well not even exist, and we might as well be honest and have Kconfig say that there is no help available for the option. -- 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