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

[Michael Buesch <mb@bu3sch.de>]

On Monday 05 February 2007 14:12, Michael Buesch wrote:
> +static u16 ieee80211_rts_duration(struct ieee80211_local *local,
> +				  size_t frame_len, int rate,
> +				  int erp, int short_preamble)
> +{
> +	u16 dur;
> +
> +	/* Data frame duration */
> +	dur = ieee80211_frame_duration(local, frame_len, rate, erp, short_preamble);
> +	/* ACK duration */
> +	dur += ieee80211_frame_duration(local, 14, rate, erp, short_preamble);
> +	/* CTS duration */
> +	dur += ieee80211_frame_duration(local, 14, rate, erp, short_preamble);

Ok, I just figured out that this is wrong.
It should be length 10, as the FCS length is added in ieee80211_frame_duration.

> +	return dur;
> +}
> +
> +static u16 ieee80211_ctstoself_duration(struct ieee80211_local *local,
> +					size_t frame_len, int rate,
> +					int erp, int short_preamble,
> +					int data_requires_ack)
> +{
> +	u16 dur;
> +
> +	/* Data frame duration */
> +	dur = ieee80211_frame_duration(local, frame_len, rate, erp, short_preamble);
> +	if (data_requires_ack) {
> +		/* ACK duration */
> +		dur += ieee80211_frame_duration(local, 14, rate, erp, short_preamble);
> +	}

Same here.


I will submit a corrected patch, soon.
But please still review this one for other bugs.

-- 
Greetings Michael.