From mboxrd@z Thu Jan 1 00:00:00 1970 From: Luciano Coelho Subject: Re: [PATCH] mfd: Combine MFD_SUPPORT and MFD_CORE Date: Fri, 02 Sep 2011 14:54:55 +0300 Message-ID: <1314964495.2296.632.camel@cumari> References: <20110829102732.03f0f05d.rdunlap@xenotime.net> <1314643307-17780-1-git-send-email-coelho@ti.com> <201108311849.37273.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201108311849.37273.arnd@arndb.de> Sender: linux-kernel-owner@vger.kernel.org To: Arnd Bergmann Cc: Randy Dunlap , matti.j.aaltonen@nokia.com, johannes@sipsolutions.net, linux-kernel@vger.kernel.org, sameo@linux.intel.com, mchehab@infradead.org, linux-media@vger.kernel.org, linux-omap@vger.kernel.org, Jean Delvare , Tony Lindgren , Grant Likely List-Id: linux-omap@vger.kernel.org On Wed, 2011-08-31 at 18:49 +0200, Arnd Bergmann wrote: > On Monday 29 August 2011, Luciano Coelho wrote: > > From: Randy Dunlap > > > > Combine MFD_SUPPORT (which only enabled the remainder of the MFD > > menu) and MFD_CORE. This allows other drivers to select MFD_CORE > > without needing to also select MFD_SUPPORT, which fixes some > > kconfig unmet dependency warnings. Modeled after I2C kconfig. > > > > [Forward-ported to 3.1-rc4. This fixes a warning when some drivers, > > such as RADIO_WL1273, are selected, but MFD_SUPPORT is not. -- Luca] > > > > Signed-off-by: Randy Dunlap > > Reported-by: Johannes Berg > > Cc: Jean Delvare > > Cc: Tony Lindgren > > Cc: Grant Likely > > Signed-off-by: Luciano Coelho > > --- > > > > I guess this should fix the problem. I've simple forward-ported > > Randy's patch to the latest mainline kernel. I don't know via which > > tree this should go in, though. > > > > NOTE: I have not tested this very thoroughly. But at least > > omap2plus stuff seems to work okay with this change. MFD_SUPPORT is > > also selected by a couple of "tile" platforms defconfigs, but I guess > > the Kconfig system should take care of it. > > Doing this is a good idea, but incidentally I have just spent some time > with the same problem and ended up with a solution that I like better, > which is removing CONFIG_MFD_SUPPORT altogether. > > The point is that there is no use enabling MFD_CORE if you don't also > enable any of the specific drivers. MFD_SUPPORT was added as a 'menuconfig' > before we had Kconfig warn about broken dependencies, so everything was > fine. Since Kconfig now issues the warnings, I think it would be better > to just turn the MFD menu into a plain 'menu' and remove all the > 'depends on MFD_SUPPORT' and 'select MFD_SUPPORT' lines from the other > Kconfig files. Yes, this makes sense. I think your solution is indeed cleaner. If you want to send it it's fine with me. I don't really have any preference, I just wanted to clean a problem that was reported to me. ;) If you send your changes, my patch can be ignored, otherwise, I can send a v2 with the changes Jean proposed. -- Cheers, Luca.