Search Linux Wireless

Re: [Rt2400-devel] rt2x00: cleanup Kconfig help

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > >  	  When compiled as a module, this driver will be called "rt2500usb.ko".
> > >  
> > >  config RT73USB
> > > -	tristate "Ralink rt73 usb support"
> > > +	tristate "Ralink RT2571W and RT2671 (USB) support"
> > >  	depends on RT2X00 && USB
> > >  	select RT2X00_LIB_USB
> > >  	select RT2X00_LIB_FIRMWARE
> > >  	---help---
> > > -	  This is an experimental driver for the Ralink rt73 wireless chip.
> > > +	  This is an experimental driver for the RT2571W and RT2671 MAC/BBP chips. It supports Ralink RT2501USB and RT5201USB wireless chipsets.
> > >  	  When compiled as a module, this driver will be called "rt73usb.ko".
> > 
> > *sigh* you seem to be absolutely sure you want to break any form of consistency in the rt2x00 KConfig entry.
> > So to show how I would like to see it I have the following patch. Unless you have objections against below changes
> > I will push this forward.
> No objection, but note that:
> * this has both "hardware that feature*s*" and  "hardware that feature"

I'll fix that.

> * The names of drivers are duplicated and not exactly the same. For example, "This is the experimental rt2500pci driver [...]" vs "[...] this driver will be called "rt2500pci.ko"".

I'll change that to:
This is the experimental rt2500pci driver [...]" vs "[...] the module will be called "rt2500pci.ko"".

> Here is the third version of my patch.

NACK

All my complaints about your previous patch still aplies to this new one,
and I am not going to repeat myself over and over again for exactly the same.

As for the bug this patch is supposedly going to fix,
the config option is named:
	tristate "Ralink rt2500 usb support"
How can people confuse that with pci support?
Ýour patch suggests the confusion is to blame because there is a space
between rt2500 and usb, and that usb isn't capitalized...

I'll update my previous patch with some additional chipset notices and send that upstream in a few days.
I'll consider this issue (including the bug report) closed after that.

Ivo

> ---
> 
> --- drivers/net/wireless/rt2x00/Kconfig	2008-01-24 17:58:37.000000000 -0500
> +++ drivers/net/wireless/rt2x00/Kconfig.USB	2008-04-14 02:29:37.000000000 -0400
> @@ -2,10 +2,10 @@
>  	tristate "Ralink driver support"
>  	depends on MAC80211 && WLAN_80211 && EXPERIMENTAL
>  	---help---
> -	  This will enable the experimental support for the Ralink drivers,
> +	  This enables the experimental support for the Ralink drivers,
>  	  developed in the rt2x00 project <http://rt2x00.serialmonkey.com>.
>  
> -	  These drivers will make use of the Devicescape ieee80211 stack.
> +	  These drivers use the mac80211 stack.
>  
>  	  When building one of the individual drivers, the rt2x00 library
>  	  will also be created. That library (when the driver is built as
> @@ -38,12 +38,12 @@
>  	select INPUT_POLLDEV
>  
>  config RT2400PCI
> -	tristate "Ralink rt2400 pci/pcmcia support"
> +	tristate "Ralink RT2400 (PCI/PCMCIA) support"
>  	depends on RT2X00 && PCI
>  	select RT2X00_LIB_PCI
>  	select EEPROM_93CX6
>  	---help---
> -	  This is an experimental driver for the Ralink rt2400 wireless chip.
> +	  This is an experimental driver for the Ralink RT2400 wireless chipset.
>  
>  	  When compiled as a module, this driver will be called "rt2400pci.ko".
>  
> @@ -52,17 +52,17 @@
>  	depends on RT2400PCI
>  	select RT2X00_LIB_RFKILL
>  	---help---
> -	  This adds support for integrated rt2400 devices that feature a
> +	  This adds support for integrated RT2400 devices that feature a
>  	  hardware button to control the radio state.
>  	  This feature depends on the RF switch subsystem rfkill.
>  
>  config RT2500PCI
> -	tristate "Ralink rt2500 pci/pcmcia support"
> +	tristate "Ralink RT2500 (PCI/PCMCIA) support"
>  	depends on RT2X00 && PCI
>  	select RT2X00_LIB_PCI
>  	select EEPROM_93CX6
>  	---help---
> -	  This is an experimental driver for the Ralink rt2500 wireless chip.
> +	  This is an experimental driver for the Ralink RT2500 wireless chipset.
>  
>  	  When compiled as a module, this driver will be called "rt2500pci.ko".
>  
> @@ -71,18 +71,18 @@
>  	depends on RT2500PCI
>  	select RT2X00_LIB_RFKILL
>  	---help---
> -	  This adds support for integrated rt2500 devices that feature a
> +	  This adds support for integrated RT2500 devices that feature a
>  	  hardware button to control the radio state.
>  	  This feature depends on the RF switch subsystem rfkill.
>  
>  config RT61PCI
> -	tristate "Ralink rt61 pci/pcmcia support"
> +	tristate "Ralink RT2561 and RT2661 (PCI/PCMCIA) support"
>  	depends on RT2X00 && PCI
>  	select RT2X00_LIB_PCI
>  	select RT2X00_LIB_FIRMWARE
>  	select EEPROM_93CX6
>  	---help---
> -	  This is an experimental driver for the Ralink rt61 wireless chip.
> +	  This is an experimental driver for the Ralink RT2561 and RT2661 MAC/BBP chips. It supports RT2600 and RT2501 [Turbo] wireless chipsets.
>  
>  	  When compiled as a module, this driver will be called "rt61pci.ko".
>  
> @@ -91,26 +91,26 @@
>  	depends on RT61PCI
>  	select RT2X00_LIB_RFKILL
>  	---help---
> -	  This adds support for integrated rt61 devices that feature a
> +	  This adds support for integrated RT2561 and RT2661 devices that feature a
>  	  hardware button to control the radio state.
>  	  This feature depends on the RF switch subsystem rfkill.
>  
>  config RT2500USB
> -	tristate "Ralink rt2500 usb support"
> +	tristate "Ralink RT2500USB support"
>  	depends on RT2X00 && USB
>  	select RT2X00_LIB_USB
>  	---help---
> -	  This is an experimental driver for the Ralink rt2500 wireless chip.
> +	  This is an experimental driver for the Ralink RT2500USB wireless chipset.
>  
>  	  When compiled as a module, this driver will be called "rt2500usb.ko".
>  
>  config RT73USB
> -	tristate "Ralink rt73 usb support"
> +	tristate "Ralink RT2571W and RT2671 (USB) support"
>  	depends on RT2X00 && USB
>  	select RT2X00_LIB_USB
>  	select RT2X00_LIB_FIRMWARE
>  	---help---
> -	  This is an experimental driver for the Ralink rt73 wireless chip.
> +	  This is an experimental driver for the Ralink RT2571W and RT2671 MAC/BBP chips. It supports RT2501USB and RT5201USB wireless chipsets.
>  
>  	  When compiled as a module, this driver will be called "rt73usb.ko".
>  
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
> Don't miss this year's exciting event. There's still time to save $100. 
> Use priority code J8TL2D2. 
> http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
> _______________________________________________
> Rt2400-devel mailing list
> Rt2400-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.sourceforge.net/lists/listinfo/rt2400-devel
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux