* [PATCH] libertas: separate libertas' Kconfig in it's own file
@ 2009-10-06 14:31 Holger Schurig
2009-10-06 22:39 ` Dan Williams
0 siblings, 1 reply; 3+ messages in thread
From: Holger Schurig @ 2009-10-06 14:31 UTC (permalink / raw)
To: linux-wireless; +Cc: John W Linville, Dan Williams
Also sorts all "source" lines in the wireless/Kconfig.
Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Index: linux-wl/drivers/net/wireless/Kconfig
===================================================================
--- linux-wl.orig/drivers/net/wireless/Kconfig 2009-10-06 07:55:26.000000000 +0200
+++ linux-wl/drivers/net/wireless/Kconfig 2009-10-06 07:58:05.000000000 +0200
@@ -139,47 +139,6 @@ config PCMCIA_RAYCS
To compile this driver as a module, choose M here: the module will be
called ray_cs. If unsure, say N.
-config LIBERTAS
- tristate "Marvell 8xxx Libertas WLAN driver support"
- depends on WLAN_80211
- select WIRELESS_EXT
- select WEXT_SPY
- select LIB80211
- select FW_LOADER
- ---help---
- A library for Marvell Libertas 8xxx devices.
-
-config LIBERTAS_USB
- tristate "Marvell Libertas 8388 USB 802.11b/g cards"
- depends on LIBERTAS && USB
- ---help---
- A driver for Marvell Libertas 8388 USB devices.
-
-config LIBERTAS_CS
- tristate "Marvell Libertas 8385 CompactFlash 802.11b/g cards"
- depends on LIBERTAS && PCMCIA
- select FW_LOADER
- ---help---
- A driver for Marvell Libertas 8385 CompactFlash devices.
-
-config LIBERTAS_SDIO
- tristate "Marvell Libertas 8385/8686/8688 SDIO 802.11b/g cards"
- depends on LIBERTAS && MMC
- ---help---
- A driver for Marvell Libertas 8385/8686/8688 SDIO devices.
-
-config LIBERTAS_SPI
- tristate "Marvell Libertas 8686 SPI 802.11b/g cards"
- depends on LIBERTAS && SPI
- ---help---
- A driver for Marvell Libertas 8686 SPI devices.
-
-config LIBERTAS_DEBUG
- bool "Enable full debugging output in the Libertas module."
- depends on LIBERTAS
- ---help---
- Debugging support.
-
config LIBERTAS_THINFIRM
tristate "Marvell 8xxx Libertas WLAN driver support with thin firmware"
depends on WLAN_80211 && MAC80211
@@ -491,17 +450,18 @@ config MWL8K
To compile this driver as a module, choose M here: the module
will be called mwl8k. If unsure, say N.
-source "drivers/net/wireless/p54/Kconfig"
source "drivers/net/wireless/ath/Kconfig"
-source "drivers/net/wireless/ipw2x00/Kconfig"
-source "drivers/net/wireless/iwlwifi/Kconfig"
-source "drivers/net/wireless/hostap/Kconfig"
source "drivers/net/wireless/b43/Kconfig"
source "drivers/net/wireless/b43legacy/Kconfig"
-source "drivers/net/wireless/zd1211rw/Kconfig"
-source "drivers/net/wireless/rt2x00/Kconfig"
+source "drivers/net/wireless/hostap/Kconfig"
+source "drivers/net/wireless/ipw2x00/Kconfig"
+source "drivers/net/wireless/iwlwifi/Kconfig"
+source "drivers/net/wireless/iwmc3200wifi/Kconfig"
+source "drivers/net/wireless/libertas/Kconfig"
source "drivers/net/wireless/orinoco/Kconfig"
+source "drivers/net/wireless/p54/Kconfig"
+source "drivers/net/wireless/rt2x00/Kconfig"
source "drivers/net/wireless/wl12xx/Kconfig"
-source "drivers/net/wireless/iwmc3200wifi/Kconfig"
+source "drivers/net/wireless/zd1211rw/Kconfig"
endif # WLAN
Index: linux-wl/drivers/net/wireless/libertas/Makefile
===================================================================
--- linux-wl.orig/drivers/net/wireless/libertas/Makefile 2009-10-06 07:55:26.000000000 +0200
+++ linux-wl/drivers/net/wireless/libertas/Makefile 2009-10-06 07:58:25.000000000 +0200
@@ -1,5 +1,15 @@
-libertas-objs := main.o wext.o rx.o tx.o cmd.o cmdresp.o scan.o 11d.o \
- debugfs.o persistcfg.o ethtool.o assoc.o
+libertas-y += 11d.o
+libertas-y += assoc.o
+libertas-y += cmd.o
+libertas-y += cmdresp.o
+libertas-y += debugfs.o
+libertas-y += ethtool.o
+libertas-y += main.o
+libertas-y += persistcfg.o
+libertas-y += rx.o
+libertas-y += scan.o
+libertas-y += tx.o
+libertas-y += wext.o
usb8xxx-objs += if_usb.o
libertas_cs-objs += if_cs.o
Index: linux-wl/drivers/net/wireless/libertas/Kconfig
===================================================================
--- /dev/null 1970-01-01 00:00:00.000000000 +0000
+++ linux-wl/drivers/net/wireless/libertas/Kconfig 2009-10-06 07:58:15.000000000 +0200
@@ -0,0 +1,40 @@
+config LIBERTAS
+ tristate "Marvell 8xxx Libertas WLAN driver support"
+ depends on WLAN_80211
+ select WIRELESS_EXT
+ select WEXT_SPY
+ select LIB80211
+ select FW_LOADER
+ ---help---
+ A library for Marvell Libertas 8xxx devices.
+
+config LIBERTAS_USB
+ tristate "Marvell Libertas 8388 USB 802.11b/g cards"
+ depends on LIBERTAS && USB
+ ---help---
+ A driver for Marvell Libertas 8388 USB devices.
+
+config LIBERTAS_CS
+ tristate "Marvell Libertas 8385 CompactFlash 802.11b/g cards"
+ depends on LIBERTAS && PCMCIA
+ select FW_LOADER
+ ---help---
+ A driver for Marvell Libertas 8385 CompactFlash devices.
+
+config LIBERTAS_SDIO
+ tristate "Marvell Libertas 8385/8686/8688 SDIO 802.11b/g cards"
+ depends on LIBERTAS && MMC
+ ---help---
+ A driver for Marvell Libertas 8385/8686/8688 SDIO devices.
+
+config LIBERTAS_SPI
+ tristate "Marvell Libertas 8686 SPI 802.11b/g cards"
+ depends on LIBERTAS && SPI
+ ---help---
+ A driver for Marvell Libertas 8686 SPI devices.
+
+config LIBERTAS_DEBUG
+ bool "Enable full debugging output in the Libertas module."
+ depends on LIBERTAS
+ ---help---
+ Debugging support.
--
http://www.holgerschurig.de
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] libertas: separate libertas' Kconfig in it's own file
2009-10-06 14:31 [PATCH] libertas: separate libertas' Kconfig in it's own file Holger Schurig
@ 2009-10-06 22:39 ` Dan Williams
2009-10-07 1:15 ` Andrey Yurovsky
0 siblings, 1 reply; 3+ messages in thread
From: Dan Williams @ 2009-10-06 22:39 UTC (permalink / raw)
To: Holger Schurig; +Cc: linux-wireless, John W Linville
On Tue, 2009-10-06 at 16:31 +0200, Holger Schurig wrote:
> Also sorts all "source" lines in the wireless/Kconfig.
>
> Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Yeah, we've got a lot more options now.
Acked-by: Dan Williams <dcbw@redhat.com>
>
> Index: linux-wl/drivers/net/wireless/Kconfig
> ===================================================================
> --- linux-wl.orig/drivers/net/wireless/Kconfig 2009-10-06 07:55:26.000000000 +0200
> +++ linux-wl/drivers/net/wireless/Kconfig 2009-10-06 07:58:05.000000000 +0200
> @@ -139,47 +139,6 @@ config PCMCIA_RAYCS
> To compile this driver as a module, choose M here: the module will be
> called ray_cs. If unsure, say N.
>
> -config LIBERTAS
> - tristate "Marvell 8xxx Libertas WLAN driver support"
> - depends on WLAN_80211
> - select WIRELESS_EXT
> - select WEXT_SPY
> - select LIB80211
> - select FW_LOADER
> - ---help---
> - A library for Marvell Libertas 8xxx devices.
> -
> -config LIBERTAS_USB
> - tristate "Marvell Libertas 8388 USB 802.11b/g cards"
> - depends on LIBERTAS && USB
> - ---help---
> - A driver for Marvell Libertas 8388 USB devices.
> -
> -config LIBERTAS_CS
> - tristate "Marvell Libertas 8385 CompactFlash 802.11b/g cards"
> - depends on LIBERTAS && PCMCIA
> - select FW_LOADER
> - ---help---
> - A driver for Marvell Libertas 8385 CompactFlash devices.
> -
> -config LIBERTAS_SDIO
> - tristate "Marvell Libertas 8385/8686/8688 SDIO 802.11b/g cards"
> - depends on LIBERTAS && MMC
> - ---help---
> - A driver for Marvell Libertas 8385/8686/8688 SDIO devices.
> -
> -config LIBERTAS_SPI
> - tristate "Marvell Libertas 8686 SPI 802.11b/g cards"
> - depends on LIBERTAS && SPI
> - ---help---
> - A driver for Marvell Libertas 8686 SPI devices.
> -
> -config LIBERTAS_DEBUG
> - bool "Enable full debugging output in the Libertas module."
> - depends on LIBERTAS
> - ---help---
> - Debugging support.
> -
> config LIBERTAS_THINFIRM
> tristate "Marvell 8xxx Libertas WLAN driver support with thin firmware"
> depends on WLAN_80211 && MAC80211
> @@ -491,17 +450,18 @@ config MWL8K
> To compile this driver as a module, choose M here: the module
> will be called mwl8k. If unsure, say N.
>
> -source "drivers/net/wireless/p54/Kconfig"
> source "drivers/net/wireless/ath/Kconfig"
> -source "drivers/net/wireless/ipw2x00/Kconfig"
> -source "drivers/net/wireless/iwlwifi/Kconfig"
> -source "drivers/net/wireless/hostap/Kconfig"
> source "drivers/net/wireless/b43/Kconfig"
> source "drivers/net/wireless/b43legacy/Kconfig"
> -source "drivers/net/wireless/zd1211rw/Kconfig"
> -source "drivers/net/wireless/rt2x00/Kconfig"
> +source "drivers/net/wireless/hostap/Kconfig"
> +source "drivers/net/wireless/ipw2x00/Kconfig"
> +source "drivers/net/wireless/iwlwifi/Kconfig"
> +source "drivers/net/wireless/iwmc3200wifi/Kconfig"
> +source "drivers/net/wireless/libertas/Kconfig"
> source "drivers/net/wireless/orinoco/Kconfig"
> +source "drivers/net/wireless/p54/Kconfig"
> +source "drivers/net/wireless/rt2x00/Kconfig"
> source "drivers/net/wireless/wl12xx/Kconfig"
> -source "drivers/net/wireless/iwmc3200wifi/Kconfig"
> +source "drivers/net/wireless/zd1211rw/Kconfig"
>
> endif # WLAN
> Index: linux-wl/drivers/net/wireless/libertas/Makefile
> ===================================================================
> --- linux-wl.orig/drivers/net/wireless/libertas/Makefile 2009-10-06 07:55:26.000000000 +0200
> +++ linux-wl/drivers/net/wireless/libertas/Makefile 2009-10-06 07:58:25.000000000 +0200
> @@ -1,5 +1,15 @@
> -libertas-objs := main.o wext.o rx.o tx.o cmd.o cmdresp.o scan.o 11d.o \
> - debugfs.o persistcfg.o ethtool.o assoc.o
> +libertas-y += 11d.o
> +libertas-y += assoc.o
> +libertas-y += cmd.o
> +libertas-y += cmdresp.o
> +libertas-y += debugfs.o
> +libertas-y += ethtool.o
> +libertas-y += main.o
> +libertas-y += persistcfg.o
> +libertas-y += rx.o
> +libertas-y += scan.o
> +libertas-y += tx.o
> +libertas-y += wext.o
>
> usb8xxx-objs += if_usb.o
> libertas_cs-objs += if_cs.o
> Index: linux-wl/drivers/net/wireless/libertas/Kconfig
> ===================================================================
> --- /dev/null 1970-01-01 00:00:00.000000000 +0000
> +++ linux-wl/drivers/net/wireless/libertas/Kconfig 2009-10-06 07:58:15.000000000 +0200
> @@ -0,0 +1,40 @@
> +config LIBERTAS
> + tristate "Marvell 8xxx Libertas WLAN driver support"
> + depends on WLAN_80211
> + select WIRELESS_EXT
> + select WEXT_SPY
> + select LIB80211
> + select FW_LOADER
> + ---help---
> + A library for Marvell Libertas 8xxx devices.
> +
> +config LIBERTAS_USB
> + tristate "Marvell Libertas 8388 USB 802.11b/g cards"
> + depends on LIBERTAS && USB
> + ---help---
> + A driver for Marvell Libertas 8388 USB devices.
> +
> +config LIBERTAS_CS
> + tristate "Marvell Libertas 8385 CompactFlash 802.11b/g cards"
> + depends on LIBERTAS && PCMCIA
> + select FW_LOADER
> + ---help---
> + A driver for Marvell Libertas 8385 CompactFlash devices.
> +
> +config LIBERTAS_SDIO
> + tristate "Marvell Libertas 8385/8686/8688 SDIO 802.11b/g cards"
> + depends on LIBERTAS && MMC
> + ---help---
> + A driver for Marvell Libertas 8385/8686/8688 SDIO devices.
> +
> +config LIBERTAS_SPI
> + tristate "Marvell Libertas 8686 SPI 802.11b/g cards"
> + depends on LIBERTAS && SPI
> + ---help---
> + A driver for Marvell Libertas 8686 SPI devices.
> +
> +config LIBERTAS_DEBUG
> + bool "Enable full debugging output in the Libertas module."
> + depends on LIBERTAS
> + ---help---
> + Debugging support.
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] libertas: separate libertas' Kconfig in it's own file
2009-10-06 22:39 ` Dan Williams
@ 2009-10-07 1:15 ` Andrey Yurovsky
0 siblings, 0 replies; 3+ messages in thread
From: Andrey Yurovsky @ 2009-10-07 1:15 UTC (permalink / raw)
To: Dan Williams; +Cc: Holger Schurig, linux-wireless, John W Linville
On Tue, Oct 6, 2009 at 3:39 PM, Dan Williams <dcbw@redhat.com> wrote:
> On Tue, 2009-10-06 at 16:31 +0200, Holger Schurig wrote:
>> Also sorts all "source" lines in the wireless/Kconfig.
>>
>> Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
>
> Yeah, we've got a lot more options now.
>
> Acked-by: Dan Williams <dcbw@redhat.com>
LIBERTAS_CS does a "select FW_LOADER" that's already selected by
LIBERTAS, so we should probably remove that line. Probably worth a
separate (trivial) patch?
-Andrey
>> Index: linux-wl/drivers/net/wireless/Kconfig
>> ===================================================================
>> --- linux-wl.orig/drivers/net/wireless/Kconfig 2009-10-06 07:55:26.000000000 +0200
>> +++ linux-wl/drivers/net/wireless/Kconfig 2009-10-06 07:58:05.000000000 +0200
>> @@ -139,47 +139,6 @@ config PCMCIA_RAYCS
>> To compile this driver as a module, choose M here: the module will be
>> called ray_cs. If unsure, say N.
>>
>> -config LIBERTAS
>> - tristate "Marvell 8xxx Libertas WLAN driver support"
>> - depends on WLAN_80211
>> - select WIRELESS_EXT
>> - select WEXT_SPY
>> - select LIB80211
>> - select FW_LOADER
>> - ---help---
>> - A library for Marvell Libertas 8xxx devices.
>> -
>> -config LIBERTAS_USB
>> - tristate "Marvell Libertas 8388 USB 802.11b/g cards"
>> - depends on LIBERTAS && USB
>> - ---help---
>> - A driver for Marvell Libertas 8388 USB devices.
>> -
>> -config LIBERTAS_CS
>> - tristate "Marvell Libertas 8385 CompactFlash 802.11b/g cards"
>> - depends on LIBERTAS && PCMCIA
>> - select FW_LOADER
>> - ---help---
>> - A driver for Marvell Libertas 8385 CompactFlash devices.
>> -
>> -config LIBERTAS_SDIO
>> - tristate "Marvell Libertas 8385/8686/8688 SDIO 802.11b/g cards"
>> - depends on LIBERTAS && MMC
>> - ---help---
>> - A driver for Marvell Libertas 8385/8686/8688 SDIO devices.
>> -
>> -config LIBERTAS_SPI
>> - tristate "Marvell Libertas 8686 SPI 802.11b/g cards"
>> - depends on LIBERTAS && SPI
>> - ---help---
>> - A driver for Marvell Libertas 8686 SPI devices.
>> -
>> -config LIBERTAS_DEBUG
>> - bool "Enable full debugging output in the Libertas module."
>> - depends on LIBERTAS
>> - ---help---
>> - Debugging support.
>> -
>> config LIBERTAS_THINFIRM
>> tristate "Marvell 8xxx Libertas WLAN driver support with thin firmware"
>> depends on WLAN_80211 && MAC80211
>> @@ -491,17 +450,18 @@ config MWL8K
>> To compile this driver as a module, choose M here: the module
>> will be called mwl8k. If unsure, say N.
>>
>> -source "drivers/net/wireless/p54/Kconfig"
>> source "drivers/net/wireless/ath/Kconfig"
>> -source "drivers/net/wireless/ipw2x00/Kconfig"
>> -source "drivers/net/wireless/iwlwifi/Kconfig"
>> -source "drivers/net/wireless/hostap/Kconfig"
>> source "drivers/net/wireless/b43/Kconfig"
>> source "drivers/net/wireless/b43legacy/Kconfig"
>> -source "drivers/net/wireless/zd1211rw/Kconfig"
>> -source "drivers/net/wireless/rt2x00/Kconfig"
>> +source "drivers/net/wireless/hostap/Kconfig"
>> +source "drivers/net/wireless/ipw2x00/Kconfig"
>> +source "drivers/net/wireless/iwlwifi/Kconfig"
>> +source "drivers/net/wireless/iwmc3200wifi/Kconfig"
>> +source "drivers/net/wireless/libertas/Kconfig"
>> source "drivers/net/wireless/orinoco/Kconfig"
>> +source "drivers/net/wireless/p54/Kconfig"
>> +source "drivers/net/wireless/rt2x00/Kconfig"
>> source "drivers/net/wireless/wl12xx/Kconfig"
>> -source "drivers/net/wireless/iwmc3200wifi/Kconfig"
>> +source "drivers/net/wireless/zd1211rw/Kconfig"
>>
>> endif # WLAN
>> Index: linux-wl/drivers/net/wireless/libertas/Makefile
>> ===================================================================
>> --- linux-wl.orig/drivers/net/wireless/libertas/Makefile 2009-10-06 07:55:26.000000000 +0200
>> +++ linux-wl/drivers/net/wireless/libertas/Makefile 2009-10-06 07:58:25.000000000 +0200
>> @@ -1,5 +1,15 @@
>> -libertas-objs := main.o wext.o rx.o tx.o cmd.o cmdresp.o scan.o 11d.o \
>> - debugfs.o persistcfg.o ethtool.o assoc.o
>> +libertas-y += 11d.o
>> +libertas-y += assoc.o
>> +libertas-y += cmd.o
>> +libertas-y += cmdresp.o
>> +libertas-y += debugfs.o
>> +libertas-y += ethtool.o
>> +libertas-y += main.o
>> +libertas-y += persistcfg.o
>> +libertas-y += rx.o
>> +libertas-y += scan.o
>> +libertas-y += tx.o
>> +libertas-y += wext.o
>>
>> usb8xxx-objs += if_usb.o
>> libertas_cs-objs += if_cs.o
>> Index: linux-wl/drivers/net/wireless/libertas/Kconfig
>> ===================================================================
>> --- /dev/null 1970-01-01 00:00:00.000000000 +0000
>> +++ linux-wl/drivers/net/wireless/libertas/Kconfig 2009-10-06 07:58:15.000000000 +0200
>> @@ -0,0 +1,40 @@
>> +config LIBERTAS
>> + tristate "Marvell 8xxx Libertas WLAN driver support"
>> + depends on WLAN_80211
>> + select WIRELESS_EXT
>> + select WEXT_SPY
>> + select LIB80211
>> + select FW_LOADER
>> + ---help---
>> + A library for Marvell Libertas 8xxx devices.
>> +
>> +config LIBERTAS_USB
>> + tristate "Marvell Libertas 8388 USB 802.11b/g cards"
>> + depends on LIBERTAS && USB
>> + ---help---
>> + A driver for Marvell Libertas 8388 USB devices.
>> +
>> +config LIBERTAS_CS
>> + tristate "Marvell Libertas 8385 CompactFlash 802.11b/g cards"
>> + depends on LIBERTAS && PCMCIA
>> + select FW_LOADER
>> + ---help---
>> + A driver for Marvell Libertas 8385 CompactFlash devices.
>> +
>> +config LIBERTAS_SDIO
>> + tristate "Marvell Libertas 8385/8686/8688 SDIO 802.11b/g cards"
>> + depends on LIBERTAS && MMC
>> + ---help---
>> + A driver for Marvell Libertas 8385/8686/8688 SDIO devices.
>> +
>> +config LIBERTAS_SPI
>> + tristate "Marvell Libertas 8686 SPI 802.11b/g cards"
>> + depends on LIBERTAS && SPI
>> + ---help---
>> + A driver for Marvell Libertas 8686 SPI devices.
>> +
>> +config LIBERTAS_DEBUG
>> + bool "Enable full debugging output in the Libertas module."
>> + depends on LIBERTAS
>> + ---help---
>> + Debugging support.
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-10-07 1:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-10-06 14:31 [PATCH] libertas: separate libertas' Kconfig in it's own file Holger Schurig
2009-10-06 22:39 ` Dan Williams
2009-10-07 1:15 ` Andrey Yurovsky
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox