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
next prev parent 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox