Re: [PATCH] rtlwifi: Rework Kconfig

[Larry Finger <Larry.Finger@lwfinger.net>]

On 02/12/2013 01:54 PM, Joe Perches wrote:
> On Tue, 2013-02-12 at 13:45 -0600, Larry Finger wrote:
>> As the number of drivers in the rtlwifi family has grown, the Kconfig
>> section for them has grown unwieldy. This change has two effects: (1)
>> Variable RTLWIFI_DEBUG is documented, and (2) the entries for the
>> drivers that depend on RTLWIFI and indented.
>
> trivial note below:
>
>> +++ wireless-testing-new/drivers/net/wireless/rtlwifi/Kconfig
>> @@ -1,8 +1,26 @@
> []
>> +config RTLWIFI_DEBUG
>> +	bool "Debugging output for rtlwifi driver family"
>> +	depends on RTLWIFI
>> +	default y
>> +	---help---
>> +	This option is needed for the "debug=n" module option to produce any
>> +	output. For embedded or other memory-limited systems, choose "N".
>> +	For all other systems, or if in doubt, use "Y".
>
> using "debug=n" here can be a bit confusing given the
> "N" and "Y" that follow it.
>
> maybe debug=<0x...> module parameter to control debug
> message level output.

How about the following:

         This option must be "Y" for the module option for dynamic debugging
         to produce any output. For embedded or other memory-limited systems,
         choose "N". If in doubt, choose "Y".

Larry