Message-ID: <1432581417.27695.190.camel@x220>
Subject: Re: [RFC v1] tree-wide: remove "select FW_LOADER" uses
From: Paul Bolle <pebolle@tiscali.nl>
To: Josh Triplett <josh@joshtriplett.org>
Cc: "Luis R. Rodriguez" <mcgrof@suse.com>,
        Dmitry Torokhov <dmitry.torokhov@gmail.com>,
        Takashi Iwai <tiwai@suse.de>,
        Geert Uytterhoeven <geert@linux-m68k.org>,
        Borislav Petkov <bp@alien8.de>, Greg KH <gregkh@linuxfoundation.org>,
        "David S. Miller" <davem@davemloft.net>, clemens@ladisch.de,
        JBottomley@odin.com, David Airlie <airlied@linux.ie>,
        Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
        Herbert Xu <herbert@gondor.apana.org.au>,
        Marcel Holtmann <marcel@holtmann.org>,
        "Gustavo F. Padovan" <gustavo@padovan.org>,
        Johan Hedberg <johan.hedberg@gmail.com>,
        Mikael Starvik <starvik@axis.com>,
        Jesper Nilsson <jesper.nilsson@axis.com>,
        Imre Kaloz <kaloz@openwrt.org>, khalasa@piap.pl,
        Ohad Ben-Cohen <ohad@wizery.com>, Arnd Bergmann <arnd@arndb.de>,
        3chas3@gmail.com, Jiri Slaby <jslaby@suse.cz>,
        Bryan Wu <cooloney@gmail.com>, Richard Purdie <rpurdie@rpsys.net>,
        Jacek Anaszewski <j.anaszewski@samsung.com>, mcgrof@do-not-panic.com,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Date: Mon, 25 May 2015 21:16:57 +0200
In-Reply-To: <20150525190647.GG1037@x>
References: <s5h7fs1gfxl.wl-tiwai@suse.de>
	 <CAB=NE6WLsOoWOBwcHW5FC4k4GEtW-JWap52Zs34H-JZNtFxrhA@mail.gmail.com>
	 <20150522175711.GE40101@dtor-ws> <20150522181924.GN23057@wotan.suse.de>
	 <20150522185207.GG40101@dtor-ws> <20150522192814.GO23057@wotan.suse.de>
	 <20150522214357.GA24129@cloud> <1432551316.27695.166.camel@x220>
	 <20150525175431.GD1037@x> <1432578335.27695.186.camel@x220>
	 <20150525190647.GG1037@x>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1177
Lines: 29

On Mon, 2015-05-25 at 12:06 -0700, Josh Triplett wrote:
> On Mon, May 25, 2015 at 08:25:35PM +0200, Paul Bolle wrote:
> > On Mon, 2015-05-25 at 10:54 -0700, Josh Triplett wrote:
> For clarity: "recursive dependency resolution" as in "recursive
> resolution of dependencies", not "resolution of recursive dependencies".

I feel a headache coming up.

> For an initial solution, I'd suggest just recursing through "select"
> dependencies; if C has "select B", and B has "select A", enabling C
> should enable B and A.  Since you can't "select A || D", that seems
> straightforward enough.  You can "select A if X", but for that, just
> bail out if the value of the expression X is changed by any of the
> select statements.

Actually, kconfig already recurses through select statements. (I haven't
checked, but I'd expect it to handle the "select A if X" case correctly
while doing the recursive select dance.)

Thanks,


Paul Bolle

--
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/