Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757957AbbEVRne (ORCPT ); Fri, 22 May 2015 13:43:34 -0400 Received: from mail-ie0-f175.google.com ([209.85.223.175]:35952 "EHLO mail-ie0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757518AbbEVRnd (ORCPT ); Fri, 22 May 2015 13:43:33 -0400 MIME-Version: 1.0 In-Reply-To: References: <1432241149-8762-1-git-send-email-mcgrof@do-not-panic.com> <20150521222129.GI3689@pd.tnic> <20150522065346.GA23022@pd.tnic> <1432282668.27695.24.camel@x220> From: "Luis R. Rodriguez" Date: Fri, 22 May 2015 10:43:12 -0700 X-Google-Sender-Auth: vlrm2nmBHKb16PBGzgcBupSvzKw Message-ID: Subject: Re: [RFC v1] tree-wide: remove "select FW_LOADER" uses To: Takashi Iwai Cc: Paul Bolle , Geert Uytterhoeven , Borislav Petkov , Greg KH , "David S. Miller" , clemens@ladisch.de, Dmitry Torokhov , JBottomley@odin.com, David Airlie , Mauro Carvalho Chehab , Herbert Xu , Marcel Holtmann , "Gustavo F. Padovan" , Johan Hedberg , Mikael Starvik , Jesper Nilsson , Imre Kaloz , khalasa@piap.pl, Ohad Ben-Cohen , Arnd Bergmann , 3chas3@gmail.com, Jiri Slaby , Bryan Wu , Richard Purdie , Jacek Anaszewski , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2356 Lines: 46 On Fri, May 22, 2015 at 1:44 AM, Takashi Iwai wrote: > At Fri, 22 May 2015 10:17:48 +0200, > Paul Bolle wrote: >> >> On Fri, 2015-05-22 at 09:11 +0200, Geert Uytterhoeven wrote: >> > On Fri, May 22, 2015 at 8:53 AM, Borislav Petkov wrote: >> > > One thing I forgot last night: what about randconfigs? All that >> > > functionality which selects FW_LOADER, won't boot anymore, right? I >> > > mean, there are provisions to build fine even with FW_LOADER unset but >> > > if you want to boot-test those kernels, you will artificially fail due >> > > to missing request_firmware* things... >> >> Luis also tried to explain to me that disabling FW_LOADER shouldn't make >> the build fail. (And, of course, we could decide to not care about >> randconfig builds that have EXPERT set. Maybe we could even special case >> EXPERT in randconfig. But that would make randconfig builds less useful. >> That's a separate issue, anyhow.) > > But FW_LOADER is a tristate, so it might be inconsistent if selected > randomly? Luis' patch doesn't add depends but just removes select. We could go both ways, either remove the "select" or replace it with "depends on". As you note keeping the "depends on" ensures run time sanity for the possible tristate mismatches, but this is an EXPERT concern. The crux of what option to go with is: Should we concern ourselves with run time configuration issues when folks enable EXPERT? Without EXPERT all run time configurations are vetted to run as FW_LOADER defaults to y. If we go down the path of removing the select completely we'd be taking a position that we could at least ensure EXPERT builds will work, but we cannot vet for not run time sanity of such build. I favor simplicity so would prefer to nuke the select completely and if we're really concerned about EXPERT users tristate mismatch misconfiguration why not just replace tristate with bool for FW_LOADER. That would do us the service of simplifying that code a bit, and leave only in place one way for folks that enable EXPERT to shoot themselves in the foot with FW_LOADER? Luis -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/