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

[Michael Buesch <mb@bu3sch.de>]

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.