2013-04-04 16:01:42

by Wang YanQing

[permalink] [raw]
Subject: [PATCH] lib80211: make lib80211 can be enabled independently

Current we can only enable lib80211 by enable a driver
in tree use it which will select it, but some out tree's
drivers also use it, so I think it has sense to make lib80211
can be enabled independently.

A example of the out tree's drivers use lib80211 is:
hybird driver(wl) for Broadcom Corporation BCM43225 802.11b/g/n

Signed-off-by: Wang YanQing <[email protected]>
---
net/wireless/Kconfig | 22 +++++++++++++++++-----
1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/net/wireless/Kconfig b/net/wireless/Kconfig
index 16d08b3..6e83f0a 100644
--- a/net/wireless/Kconfig
+++ b/net/wireless/Kconfig
@@ -140,22 +140,34 @@ config CFG80211_WEXT
extensions with cfg80211-based drivers.

config LIB80211
- tristate
+ tristate "common routines used by IEEE802.11 wireless LAN drivers"
default n
help
This options enables a library of common routines used
by IEEE802.11 wireless LAN drivers.

- Drivers should select this themselves if needed.
+ Drivers could select this themselves if needed.

config LIB80211_CRYPT_WEP
- tristate
+ tristate "host-based WEP encryption implementation for lib80211"
+ depends on LIB80211
+ default n
+ ---help---
+ host-based WEP encryption implementation for lib80211

config LIB80211_CRYPT_CCMP
- tristate
+ tristate "host-based CCMP encryption implementation for lib80211"
+ depends on LIB80211
+ default n
+ ---help---
+ host-based CCMP encryption implementation for lib80211

config LIB80211_CRYPT_TKIP
- tristate
+ tristate "host-based TKIP encryption implementation for lib80211"
+ depends on LIB80211
+ default n
+ ---help---
+ host-based TKIP encryption implementation for lib80211

config LIB80211_DEBUG
bool "lib80211 debugging messages"
--
1.7.12.4.dirty


2013-04-04 23:51:52

by Julian Calaby

[permalink] [raw]
Subject: Re: [PATCH] lib80211: make lib80211 can be enabled independently

Hi Wang,

On Fri, Apr 5, 2013 at 3:01 AM, Wang YanQing <[email protected]> wrote:
>
> Current we can only enable lib80211 by enable a driver
> in tree use it which will select it, but some out tree's
> drivers also use it, so I think it has sense to make lib80211
> can be enabled independently.

Just as a bit of explanation for Johannes' NACK:

1. The only reason lib80211 still exists is because a couple of
in-tree drivers still use it. If this weren't the case, the code would
have been removed a long time ago as it's been completely replaced by
mac80211 and cfg80211. All modern drivers _must_ use mac80211 or
cfg80211 without exception.
2. In general, there is no official in-kernel support for
out-of-kernel drivers, regardless of quality, status or importance.
3. I believe that the in-tree brcmsmac driver already supports the
Broadcom card you reference.

Thanks,

--
Julian Calaby

Email: [email protected]
Profile: http://www.google.com/profiles/julian.calaby/
.Plan: http://sites.google.com/site/juliancalaby/

2013-04-05 07:23:34

by Arend van Spriel

[permalink] [raw]
Subject: Re: [PATCH] lib80211: make lib80211 can be enabled independently

On 04/05/2013 01:51 AM, Julian Calaby wrote:
> Hi Wang,
>
> On Fri, Apr 5, 2013 at 3:01 AM, Wang YanQing <[email protected]> wrote:
>>
>> Current we can only enable lib80211 by enable a driver
>> in tree use it which will select it, but some out tree's
>> drivers also use it, so I think it has sense to make lib80211
>> can be enabled independently.
>
> Just as a bit of explanation for Johannes' NACK:
>
> 3. I believe that the in-tree brcmsmac driver already supports the
> Broadcom card you reference.

That is correct.

Gr. AvS


2013-04-05 07:25:22

by Arend van Spriel

[permalink] [raw]
Subject: Re: [PATCH] lib80211: make lib80211 can be enabled independently

On 04/05/2013 06:01 AM, Randy Dunlap wrote:
> On 04/04/13 16:51, Julian Calaby wrote:
>> Hi Wang,
>>
>> On Fri, Apr 5, 2013 at 3:01 AM, Wang YanQing <[email protected]> wrote:
>>>
>>> Current we can only enable lib80211 by enable a driver
>>> in tree use it which will select it, but some out tree's
>>> drivers also use it, so I think it has sense to make lib80211
>>> can be enabled independently.
>>
>> Just as a bit of explanation for Johannes' NACK:
>>
>> 1. The only reason lib80211 still exists is because a couple of
>> in-tree drivers still use it. If this weren't the case, the code would
>> have been removed a long time ago as it's been completely replaced by
>> mac80211 and cfg80211. All modern drivers _must_ use mac80211 or
>> cfg80211 without exception.
>> 2. In general, there is no official in-kernel support for
>> out-of-kernel drivers, regardless of quality, status or importance.
>> 3. I believe that the in-tree brcmsmac driver already supports the
>> Broadcom card you reference.
>
>
> 3b. If 3. is not correct, someone should endeavor to merge this
> out-of-tree driver into the kernel tree.

Actually, it is correct and brcmsmac and the out-of-tree wl driver are
coming from the same code base although you should not compare the two
these days.

Gr. AvS



2013-04-05 04:01:55

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] lib80211: make lib80211 can be enabled independently

On 04/04/13 16:51, Julian Calaby wrote:
> Hi Wang,
>
> On Fri, Apr 5, 2013 at 3:01 AM, Wang YanQing <[email protected]> wrote:
>>
>> Current we can only enable lib80211 by enable a driver
>> in tree use it which will select it, but some out tree's
>> drivers also use it, so I think it has sense to make lib80211
>> can be enabled independently.
>
> Just as a bit of explanation for Johannes' NACK:
>
> 1. The only reason lib80211 still exists is because a couple of
> in-tree drivers still use it. If this weren't the case, the code would
> have been removed a long time ago as it's been completely replaced by
> mac80211 and cfg80211. All modern drivers _must_ use mac80211 or
> cfg80211 without exception.
> 2. In general, there is no official in-kernel support for
> out-of-kernel drivers, regardless of quality, status or importance.
> 3. I believe that the in-tree brcmsmac driver already supports the
> Broadcom card you reference.


3b. If 3. is not correct, someone should endeavor to merge this
out-of-tree driver into the kernel tree.


--
~Randy

2013-04-04 16:05:54

by Johannes Berg

[permalink] [raw]
Subject: Re: [PATCH] lib80211: make lib80211 can be enabled independently

On Fri, 2013-04-05 at 00:01 +0800, Wang YanQing wrote:
> Current we can only enable lib80211 by enable a driver
> in tree use it which will select it, but some out tree's
> drivers also use it, so I think it has sense to make lib80211
> can be enabled independently.
>
> A example of the out tree's drivers use lib80211 is:
> hybird driver(wl) for Broadcom Corporation BCM43225 802.11b/g/n

Tough luck.

NACK.

johannes


2013-04-05 05:45:33

by Wang YanQing

[permalink] [raw]
Subject: Re: [PATCH] lib80211: make lib80211 can be enabled independently

On Fri, Apr 05, 2013 at 10:51:30AM +1100, Julian Calaby wrote:
> Just as a bit of explanation for Johannes' NACK:
>
> 1. The only reason lib80211 still exists is because a couple of
> in-tree drivers still use it. If this weren't the case, the code would
> have been removed a long time ago as it's been completely replaced by
> mac80211 and cfg80211. All modern drivers _must_ use mac80211 or
> cfg80211 without exception.
> 2. In general, there is no official in-kernel support for
> out-of-kernel drivers, regardless of quality, status or importance.
> 3. I believe that the in-tree brcmsmac driver already supports the
> Broadcom card you reference.

Hi Julian.

Thanks for your complete explanation.
I have tried in-tree brcmsmac driver and it works well,
I use it to send out this email! This is wonderful, then
I don't need to fix compilation errors about the old, ugly
and closed wl driver for my notebook with BCM43225 802.11b/g/n
every time I upgrade kernel.

Thanks Julian and all.