linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Mysterious IWLWIFI_SPECTRUM_MEASUREMENT config
@ 2009-02-18 17:33 Kalle Valo
  2009-02-18 21:40 ` reinette chatre
  0 siblings, 1 reply; 3+ messages in thread
From: Kalle Valo @ 2009-02-18 17:33 UTC (permalink / raw)
  To: linux-wireless

I was doing my usual practise of recompiling my kernel and again I was
asked this question:

Enable Spectrum Measurement in iwlagn driver
(IWLWIFI_SPECTRUM_MEASUREMENT) [N/y/?] (NEW)

Ok, so I chose '?' and got this:

"This option will enable spectrum measurement for the iwlagn driver."

Not a very helpful message. I have few kind requests:

1) Please write a proper description for the option.

2) Recommend a proper default value: "If you don't have a clue what
   this is, choose N".

3) Please consider even removing the option. Is it really needed?

-- 
Kalle Valo

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Mysterious IWLWIFI_SPECTRUM_MEASUREMENT config
  2009-02-18 17:33 Mysterious IWLWIFI_SPECTRUM_MEASUREMENT config Kalle Valo
@ 2009-02-18 21:40 ` reinette chatre
  2009-02-19  5:37   ` Kalle Valo
  0 siblings, 1 reply; 3+ messages in thread
From: reinette chatre @ 2009-02-18 21:40 UTC (permalink / raw)
  To: Kalle Valo; +Cc: linux-wireless@vger.kernel.org

On Wed, 2009-02-18 at 09:33 -0800, Kalle Valo wrote:
> I was doing my usual practise of recompiling my kernel and again I was
> asked this question:
> 
> Enable Spectrum Measurement in iwlagn driver
> (IWLWIFI_SPECTRUM_MEASUREMENT) [N/y/?] (NEW)
> 
> Ok, so I chose '?' and got this:
> 
> "This option will enable spectrum measurement for the iwlagn driver."
> 
> Not a very helpful message. I have few kind requests:
> 
> 1) Please write a proper description for the option.
Will do.

> 2) Recommend a proper default value: "If you don't have a clue what
>    this is, choose N".

These are 802.11h measurements ... so typical users may not be
interested. We'll make this a default N.

> 
> 3) Please consider even removing the option. Is it really needed?
> 

Some users may want this information so we'll keep it as an option for
now. 

Reinette


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Mysterious IWLWIFI_SPECTRUM_MEASUREMENT config
  2009-02-18 21:40 ` reinette chatre
@ 2009-02-19  5:37   ` Kalle Valo
  0 siblings, 0 replies; 3+ messages in thread
From: Kalle Valo @ 2009-02-19  5:37 UTC (permalink / raw)
  To: reinette chatre; +Cc: linux-wireless@vger.kernel.org

reinette chatre <reinette.chatre@intel.com> writes:

> On Wed, 2009-02-18 at 09:33 -0800, Kalle Valo wrote:
>> I was doing my usual practise of recompiling my kernel and again I was
>> asked this question:
>> 
>> Enable Spectrum Measurement in iwlagn driver
>> (IWLWIFI_SPECTRUM_MEASUREMENT) [N/y/?] (NEW)
>> 
>> Ok, so I chose '?' and got this:
>> 
>> "This option will enable spectrum measurement for the iwlagn driver."
>> 
>> Not a very helpful message. I have few kind requests:
>> 
>> 1) Please write a proper description for the option.
>
> Will do.

Thanks. You will make the life of the users easier.

>> 2) Recommend a proper default value: "If you don't have a clue what
>>    this is, choose N".
>
> These are 802.11h measurements ... so typical users may not be
> interested. We'll make this a default N.

Good choice.

>> 3) Please consider even removing the option. Is it really needed?
>> 
>
> Some users may want this information so we'll keep it as an option for
> now. 

Not a problem when the option is properly documented, like you are
planning to do.

-- 
Kalle Valo

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-02-19  5:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-02-18 17:33 Mysterious IWLWIFI_SPECTRUM_MEASUREMENT config Kalle Valo
2009-02-18 21:40 ` reinette chatre
2009-02-19  5:37   ` Kalle Valo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).