Search Linux Wireless

Re: [ath5k-devel] [PATCH v2 13/20] cfg80211: Add nl80211 antenna configuration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 04, 2010 at 12:30:09PM -0700, John W. Linville wrote:
> On Wed, May 19, 2010 at 10:31:48AM +0900, Bruno Randolf wrote:
> > Allow setting TX and RX antenna configuration via nl80211/cfg80211.
> > 
> > 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 multiple
> > antennas are selected, the driver may use diversity for receive and transmit.
> > This allows for a simple, yet flexible configuration interface for antennas,
> > while drivers may reject configurations they cannot support.
> > 
> > Signed-off-by: Bruno Randolf <br1@xxxxxxxxxxx>
> 
> This spawned a long thread, that isn't clearly resolved IMHO.  So, I
> haven't merged it, or the other patches that follow it in this series.
> 
> Am I mistaken?  Did this get resolved?  If not, are some of the later
> patches still interesting?
> 
> Unless I hear otherwise, I'll be dropping the rest of this series...

I don't think the API is defined clean enough yet and can yield
inconsistant interpretations. I'd like to see this clarified on
the documentation for both legacy and 802.11n. If the plan is
to support only legacy right now then I think the API can be
simpler and clearer with only three options passed, Fixed_A
Fixed_B and diversity.

  Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux