Search Linux Wireless

Re: [PATCH] d80211: Add API to generate RTS and CTS-to-self frames

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

 



On Monday 05 February 2007 15:58, Jouni Malinen wrote:
> > Index: bu3sch-wireless-dev/include/net/d80211.h
> > ===================================================================
> > --- bu3sch-wireless-dev.orig/include/net/d80211.h	2007-02-05 13:59:34.000000000 +0100
> > +++ bu3sch-wireless-dev/include/net/d80211.h	2007-02-05 14:01:30.000000000 +0100
> > @@ -194,7 +194,6 @@ struct ieee80211_tx_control {
> >  #define IEEE80211_TXCTL_TKIP_NEW_PHASE1_KEY (1<<9)
> >  	u32 flags;			       /* tx control flags defined
> >  						* above */
> > -	u16 rts_cts_duration;	/* duration field for RTS/CTS frame */
> >  	u8 retry_limit;		/* 1 = only first attempt, 2 = one retry, .. */
> >  	u8 power_level;		/* per-packet transmit power level, in dBm */
> >  	u8 antenna_sel; 	/* 0 = default/diversity, 1 = Ant0, 2 = Ant1 */
> 
> NAK. Why are you removing this? It is needed for hardware that generates
> RTS/CTS frames.

As I said in response to Johannes' comment, if a driver needs just
the duration, we should export the functions to calculate it
and remove the calculation from the tx_control and the tx_control
setup path.

Currently there is no driver using it, so I simply removed it.

BTW: Which cards need the duration, but not the whole frame?

-- 
Greetings Michael.
-
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