linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Luis R. Rodriguez" <rodrigue@qca.qualcomm.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Kalle Valo <kvalo@qca.qualcomm.com>,
	Arend van Spriel <arend@broadcom.com>,
	<linux-wireless@vger.kernel.org>, <me@bobcopeland.com>,
	<mickflemm@gmail.com>
Subject: Re: [RFC 0/3] wireless: add CONFIG_CFG80211_EXPERT
Date: Thu, 14 Jun 2012 12:31:04 -0700	[thread overview]
Message-ID: <20120614193104.GD32257@tux> (raw)
In-Reply-To: <1339425944.4520.27.camel@jlt3.sipsolutions.net>

On Mon, Jun 11, 2012 at 04:45:44PM +0200, Johannes Berg wrote:
> On Mon, 2012-06-11 at 17:40 +0300, Kalle Valo wrote:
> 
> > > I still don't see the need. What would you put under it in
> > > brcm[sf]mac? I certainly wouldn't see any reason to put anything
> > > under it in our driver since it's much simpler to .
> > 
> > Looks like something is missing here?
> 
> Sorry, yeah,  Iwas going to say it's simpler to put something under the
> driver's Kconfig.
> 
> > > Also, the argument about distros doesn't really work that way, if
> > > there are users interested in something then the distros will
> > > certainly enable this (CFG80211_EXPERT) option to get something
> > > hidden behind it.
> > 
> > I would compare this with NL80211_TESTMODE, we don't want distributions
> > to enable that either. Of course nothing prevents distros to enable
> > CFG80211_EXPERT but we need to be active to make sure it's not enabled
> > (ie. check the distro configs and file bugs etc).
> 
> Right, but that's actually a feature. I see little value in a pretty
> much meaningless "EXPERT wireless" Kconfig symbol that only groups
> others. If, as Arend suggested, it actually has some meaning, then it
> may make more sense.

This option should really be used by OEM/ODM that have assured proper
certification or scientists using a shielded room. Distros should be strongly
adviced not to use this as their users tend to be unaware or the distribution
ODM / OEM may not have done the necessary work to ensure proper regulatory
certification. Perhaps we can clarify that there is a heavier onus on
regulatory certification if these options are enabled. Linux distributions may
be eager to please their users, but I would expect them to have common sense as
well. We would just have to educate them about this option.

As for a name, I thought about it for a while and given that we have different
"wireless" technologies -- bluetooth, NFC, naming this CONFIG_WIRELESS_EXPERT
seemed odd, and given that our 802.11 framework is under cfg80211 naming it
CONFIG_CFG80211_EXPERT seemed appropriate. But even if its under cfg80211
perhaps something more explicit about the implications may be better, how
about CONFIG_CFG80211_MAY_BREAK_CERTIFICATION ?

  Luis

  parent reply	other threads:[~2012-06-14 19:31 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-09  0:51 [RFC 0/3] wireless: add CONFIG_CFG80211_EXPERT Luis R. Rodriguez
2012-06-09  0:51 ` [RFC 1/3] cfg80211: " Luis R. Rodriguez
2012-06-09  0:51 ` [RFC 2/3] ath5k: replace modparam_all_channels with CONFIG_ATH5K_TEST_CHANNELS Luis R. Rodriguez
2012-06-09  0:51 ` [RFC 3/3] ath9k: make CONFIG_ATH9K_DFS_CERTIFIED depend on CONFIG_CFG80211_EXPERT Luis R. Rodriguez
2012-06-09  7:55 ` [RFC 0/3] wireless: add CONFIG_CFG80211_EXPERT Johannes Berg
2012-06-09 19:51   ` Arend van Spriel
2012-06-11  7:32     ` Johannes Berg
2012-06-11 14:40       ` Kalle Valo
2012-06-11 14:45         ` Johannes Berg
2012-06-11 15:46           ` Kalle Valo
2012-06-14 19:31           ` Luis R. Rodriguez [this message]
2012-06-15  8:16             ` Johannes Berg
2012-06-15  8:21               ` Kalle Valo
2012-06-15 11:06                 ` Arend van Spriel
2012-06-15 16:09                   ` Kalle Valo
2012-06-15 16:53                     ` Luis R. Rodriguez
2012-06-16 12:13                       ` Arend van Spriel
2012-06-18 15:37                         ` Luis R. Rodriguez

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120614193104.GD32257@tux \
    --to=rodrigue@qca.qualcomm.com \
    --cc=arend@broadcom.com \
    --cc=johannes@sipsolutions.net \
    --cc=kvalo@qca.qualcomm.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=me@bobcopeland.com \
    --cc=mickflemm@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).