All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ivo van Doorn <ivdoorn@gmail.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: linux-wireless@vger.kernel.org
Subject: Re: Antenna software diversity
Date: Mon, 14 Jan 2008 00:18:43 +0100	[thread overview]
Message-ID: <200801140018.43316.IvDoorn@gmail.com> (raw)
In-Reply-To: <1200266014.5887.33.camel@johannes.berg>

Hi,

> > However I would like to know if this is something should remain inside
> > the driver, or if it should be moved into mac80211. Possibly integrated
> > into the rate selection module, or otherwise as a seperate module which
> > could allow the user to select a prefered antenna selection method similar
> > as to how he can select the prefered rate algorithm.
> 
> I guess it would be useful for other drivers inside mac80211. Not sure
> whether it should be in the rate control algorithm or not.

My personal preference would be making it a seperate module.

> If it is in the rate control algorithm it could keep track of antenna
> per-station to optimise the link to the station (obviously the station
> still has to be in range for both antennae when it sends a frame, but
> one could in theory achieve better data rates to the station with the
> better antenna). On the other hand, I guess nobody would want to
> implement such a thing anyway.
> 
> I think a basic implementation within mac80211 would be sufficient if
> any other drivers need it.

Well at least rt2x00 need it, I searched a bit through the other drivers
and couldn't find software diversity related code. So either it is not needed
for them, or they didn't implement it (yet).

> > In mac80211 this would mean that the antenna_sel should probably become
> > an enum looking like this:
> > 
> > enum antenna_sel {
> > 	ANTENNA_SEL_DEFAULT,
> > 	ANTENNA_SEL_A,
> > 	ANTENNA_SEL_B,
> > 	ANTENNA_SEL_DIVERSITY,
> > };
> > 
> > I am not really a fan of the ANTENNA_SEL_DEFAULT, but this is already present
> > in the current mac80211 implementation. So to remove that, we could use some
> > method for the driver to let the default antenna known to mac80211, probably
> > through an extra field in ieee80211_hw perhaps.
> 
> Sounds like a good plan, that way we could maybe even report what
> antenna is used to userland.

Exactly :)

> > The ANTENNA_SEL_DIVERSITY would in this case be hardware diversity unless
> > the driver has indicated that software diversity is in action, through a flag in
> > ieee80211_hw.
> 
> Of course, with 11n all that is moot. Or does 11n hw use diversity for
> legacy frames?

No idea, I still have to check how rt2800pci and rt2800usb handle antenna configuration.

Ivo

  reply	other threads:[~2008-01-13 23:18 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-13 23:04 Antenna software diversity Ivo van Doorn
2008-01-13 23:13 ` Johannes Berg
2008-01-13 23:18   ` Ivo van Doorn [this message]
2008-01-14  3:46     ` Nick Kossifidis

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=200801140018.43316.IvDoorn@gmail.com \
    --to=ivdoorn@gmail.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    /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.