From: Pavel Roskin <proski@gnu.org>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Cc: Johannes Berg <johannes@sipsolutions.net>,
John Linville <linville@tuxdriver.com>,
linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] mac80211: allow disabling 40MHz on 2.4GHz
Date: Tue, 01 Dec 2009 20:19:14 -0500 [thread overview]
Message-ID: <1259716754.11611.6.camel@mj> (raw)
In-Reply-To: <43e72e890912011118w535781a4id4e4572df16267f4@mail.gmail.com>
On Tue, 2009-12-01 at 11:18 -0800, Luis R. Rodriguez wrote:
> >> That looks like a hack to me.
> >>
> >> Maybe it should be treated like other CRDA flags? In fact, you can find
> >> this in dbparse.py in the wireless-regdb sources:
> >>
> >> # hole at bit 9. FIXME: Where is NO-HT40 defined?
> >> 'NO-HT40': 1<<10,
> >>
> >> However, there are no other references to NO-HT40 in the wireless-regdb
> >> or CRDA sources. I assume it's not implemented.
> >
> > It's not a regulatory requirement, it's a coexistence requirement.
>
> To extend this text a little more, me and Johannes had reviewed usage
> of a NO-HT40 flag for regulatory purposes a while back. It was used
> mainly on the old Atheros HAL regulatory code and upon a lot of review
> even with our own regulatory folks determined that in fact there was
> no specific rules about disallowing 40 MHz, but that when this was
> disallowed it can be implied by the frequency range not fitting a 40
> MHz channel. This is currently computed dynamically now on cfg80211
> and its results are outputed through the file:
>
> /sys/kernel/debug/ieee80211/phy0/ht40allow_map
OK, good to know.
Anyway, I think that using module parameters for any configuration
creates a bad precedent. Disabling HT40 in the 2.4GHz band is a "fine
tuning" compared to other settings. But there may be many other such
"knobs" (e.g. ACK timeout), and if they all are controlled by mac80211
parameters, it would be a maintenance nightmare.
If module parameters are discourages for the drivers, mac80211 should be
held to the same standard.
--
Regards,
Pavel Roskin
next prev parent reply other threads:[~2009-12-02 1:19 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-01 17:29 [PATCH] mac80211: allow disabling 40MHz on 2.4GHz Johannes Berg
2009-12-01 18:17 ` Pavel Roskin
2009-12-01 18:27 ` Johannes Berg
2009-12-01 19:18 ` Luis R. Rodriguez
2009-12-02 1:19 ` Pavel Roskin [this message]
2009-12-02 1:43 ` Johannes Berg
2009-12-02 4:26 ` Pavel Roskin
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=1259716754.11611.6.camel@mj \
--to=proski@gnu.org \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=mcgrof@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.