netdev
[Top] [All Lists]

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

To: akpm@xxxxxxxx
Subject: Re: [patch 1/3] fix buggy IEEE80211_CRYPT_* selects
From: Jeff Garzik <jgarzik@xxxxxxxxx>
Date: Fri, 04 Mar 2005 12:33:19 -0500
Cc: davem@xxxxxxxxxxxxx, netdev@xxxxxxxxxxx, bunk@xxxxxxxxx
In-reply-to: <200503041237.j24Cbb69026470@xxxxxxxxxxxxxxxxxxx>
References: <200503041237.j24Cbb69026470@xxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.3) Gecko/20040922
akpm@xxxxxxxx wrote:
From: Adrian Bunk <bunk@xxxxxxxxx>

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.

        Jeff



<Prev in Thread] Current Thread [Next in Thread>