linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bruno Randolf <br1@einfach.org>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Cc: johannes@sipsolutions.net, linville@tuxdriver.com,
	linux-wireless@vger.kernel.org, holgerschurig@gmail.com
Subject: Re: [RFC PATCH 1/2] mac80211: Add nl80211 antenna configuration
Date: Wed, 12 May 2010 10:39:32 +0900	[thread overview]
Message-ID: <201005121039.32882.br1@einfach.org> (raw)
In-Reply-To: <AANLkTilowwAyqLrIJ0_mlZxnqY-TqSHM3xdXkbO8aH6-@mail.gmail.com>

On Wednesday 12 May 2010 03:14:30 Luis R. Rodriguez wrote:
> Subject should be for cfg80211, not mac80211. In fact can you submit
> the mac80211 stuff in a separate secondary patch? Some more comments
> below.

ok.

> I think we should call this TX / RX chainmask given that with 802.11n
> hardware this is what this is called.

from the following discussion, i'll stick with antenna...

> > The antenna configuration is defined as a bitmap of allowed antennas.
> > This bitmap is 8 bit at the moment, each bit representing one antenna.
> 
> If you use chainmask for this instead of 'antenna configuration' the
> wording would be something like:
> 
> The chainmask is defined as a bitmap of chain configurations used for
> TX/RX. The bitmap allows for configuring up to up to 4 chains for both TX
> and RX, 4 bits for each TX chain, 4 bits for each RX chain.

actually we have 8 bit for TX and 8 bit for RX.
 
> > If multiple
> > antennas are selected, the driver may use diversity for receive and
> > transmit.
> 
> For 802.11n this is called "selection diversity" but typically just
> referred to as "diversity", for legacy this is called "antenna
> diversity". It may be good to elaborate how selection diversity or
> antenna diversity might be enabled, ie, will this be another command,
> or what. I think for legacy another command makes sense, and it may be
> possible for us to use the same command for enabling selection
> diversity, I am not sure if we can fine tune the diversity algorithm
> at this time, I will have to review this and get back to you.

my idea was that if multiple antennas are selected in the bitmap this means 
that "antenna diversity" will be enabled. for RX this is clear. for TX this is 
a bit ambigous: it could mean "send on both antennas", which i believe is 
impossible on legacy hardware, so it means "let HW diversity choose TX 
antenna". for 802.11n - i think the TX part is different, because multiple TX 
chains will mean "send on multiple antennas". but otherwise can't we think of 
it as some form of advanced antenna diversity?

of course there are more possibillities of antenna configurations which cannot 
be configured with this api - like the "multiple sector antennas + RTS/CTS on 
omni" configuration of ath5k, but i don't think these are very common. 

the most important use case, imho, is to limit the antennas to one fixed 
antenna, if we know only one antenna is connected. this might well apply for 
802.11n too, no?

bruno

  parent reply	other threads:[~2010-05-12  1:39 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-11  8:38 [RFC PATCH 0/2] mac80211: antenna configuration Bruno Randolf
2010-05-11  8:39 ` [RFC PATCH 1/2] mac80211: Add nl80211 " Bruno Randolf
2010-05-11  8:50   ` Johannes Berg
2010-05-11  8:51   ` Johannes Berg
2010-05-11  8:52   ` Johannes Berg
2010-05-11 18:14   ` Luis R. Rodriguez
2010-05-11 18:19     ` Johannes Berg
2010-05-11 18:26       ` Luis R. Rodriguez
2010-05-12  1:39     ` Bruno Randolf [this message]
2010-05-12  1:51       ` Luis R. Rodriguez
2010-05-11  8:39 ` [RFC PATCH 2/2] ath5k: Add support for cfg80211 antenna setting Bruno Randolf
2010-05-11  8:50   ` Johannes Berg
2010-05-11  8:53 ` [RFC PATCH 0/2] mac80211: antenna configuration Johannes Berg
2010-05-11  9:34   ` Bruno Randolf
2010-05-11 18:17     ` 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=201005121039.32882.br1@einfach.org \
    --to=br1@einfach.org \
    --cc=holgerschurig@gmail.com \
    --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 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).