Re: [RFT 3/3] b43/legacy: port to cfg80211 rfkill

[Michael Buesch <mb@bu3sch.de>]

On Tuesday 02 June 2009 13:10:30 Johannes Berg wrote:
> This ports the b43/legacy rfkill code to the new API offered
> by cfg80211 and thus removes a lot of useless stuff.
> 
> Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
> Cc: Michael Buesch <mb@bu3sch.de>
> Cc: Larry Finger <Larry.Finger@lwfinger.net>
> ---
>  drivers/net/wireless/b43/Kconfig           |    7 --
>  drivers/net/wireless/b43/Makefile          |    2 
>  drivers/net/wireless/b43/b43.h             |    3 
>  drivers/net/wireless/b43/leds.c            |    2 
>  drivers/net/wireless/b43/main.c            |   27 ++-----
>  drivers/net/wireless/b43/phy_common.h      |    2 
>  drivers/net/wireless/b43/rfkill.c          |   94 +--------------------------
>  drivers/net/wireless/b43/rfkill.h          |   46 -------------
>  drivers/net/wireless/b43legacy/Kconfig     |    8 --
>  drivers/net/wireless/b43legacy/Makefile    |    2 
>  drivers/net/wireless/b43legacy/b43legacy.h |    3 
>  drivers/net/wireless/b43legacy/leds.c      |    2 
>  drivers/net/wireless/b43legacy/main.c      |   17 +---
>  drivers/net/wireless/b43legacy/rfkill.c    |   99 +----------------------------
>  drivers/net/wireless/b43legacy/rfkill.h    |   51 --------------
>  15 files changed, 29 insertions(+), 336 deletions(-)
> 
> --- wireless-testing.orig/drivers/net/wireless/b43/Kconfig	2009-06-02 12:53:36.000000000 +0200
> +++ wireless-testing/drivers/net/wireless/b43/Kconfig	2009-06-02 12:56:11.000000000 +0200
> @@ -98,13 +98,6 @@ config B43_LEDS
>  	depends on B43 && MAC80211_LEDS && (LEDS_CLASS = y || LEDS_CLASS = B43)
>  	default y
>  
> -# This config option automatically enables b43 RFKILL support,
> -# if it's possible.
> -config B43_RFKILL
> -	bool
> -	depends on B43 && (RFKILL = y || RFKILL = B43)
> -	default y

Don't you need to add a new "depends on" to "B43", if you remove this?
Or is rfkill always compiled in if wireless is present?

-- 
Greetings, Michael.