Re: [patch 1/3] fix buggy IEEE80211_CRYPT_* selects

[Jeff Garzik <jgarzik@pobox.com>]

Adrian Bunk wrote:
> On Fri, Mar 04, 2005 at 12:33:19PM -0500, Jeff Garzik wrote:
> 
>>akpm@osdl.org wrote:
>>
>>>From: Adrian Bunk <bunk@stusta.de>
>>>
>>>Some of the options that needlessly wrote in their help text which options
>>>they do select (patch already sent) didn't obey the most important rule of
>>>select
>>>
>>> If you select something, you have to ensure that the dependencies
>>> of what you do select are fulfilled.
>>
>>>diff -puN net/ieee80211/Kconfig~fix-buggy-ieee80211_crypt_-selects 
>>>net/ieee80211/Kconfig
>>>--- 25/net/ieee80211/Kconfig~fix-buggy-ieee80211_crypt_-selects 2005-02-28 
>>>14:49:54.000000000 -0800
>>>+++ 25-akpm/net/ieee80211/Kconfig	2005-02-28 14:49:54.000000000 -0800
>>>@@ -44,6 +44,7 @@ config IEEE80211_CRYPT_WEP
>>>config IEEE80211_CRYPT_CCMP
>>>	tristate "IEEE 802.11i CCMP support"
>>>	depends on IEEE80211
>>>+	select CRYPTO
>>>	select CRYPTO_AES
>>>	---help---
>>>	Include software based cipher suites in support of IEEE 802.11i 
>>>@@ -56,6 +57,7 @@ config IEEE80211_CRYPT_CCMP
>>>config IEEE80211_CRYPT_TKIP
>>>	tristate "IEEE 802.11i TKIP encryption"
>>>	depends on IEEE80211
>>>+	select CRYPTO
>>>	select CRYPTO_MICHAEL_MIC
>>>	---help---
>>
>>
>>You are resending the old patch that is incorrect.  We don't need 
>>multiple selects, CRYPTO_AES and CRYPTO_MICHAEL_MIC should pull things in.
> 
> 
> As I already said, this implies that options like CRYPTO_AES and 
> CRYPTO_MICHAEL_MIC can no longer depend on CRYPTO.

No.  Because CRYPTO_AES and CRYPTO_MICHAEL_MIC __obviously__ depend on 
CRYPTO, it should select CRYPTO automatically given the existing entries.

Otherwise, we must start specifying dependency chains in every damn 
Kconfig entry, which is completely illogical and a maintenance nightmare.

	Jeff