--- On Tue, 4/11/08, Herton Ronaldo Krzesinski <herton@xxxxxxxxxxxxxxx> wrote:
> Realtek 8187B has a receive command queue to feedback beacon
> interrupt
> and transmitted packet status. Use it to feedback mac80211
> about status
> of transmitted packets. Unfortunately in the course of
> testing I found
> that the sequence number reported by hardware includes
> entire sequence
> control in a 12 bit only field, so a workaround is done to
> check only
> lowest bits.
> ---
> v2: fix rate stuck at 1M reported by Larry Finger, Hin-Tak
> and also seen by me.
> btw, I noted now with a ralink device that rt2500usb
> (rt2x00) has same
> issue currently.
> - hdr->retry =
> cpu_to_le32((info->control.rates[0].count - 1) <<
> 8);
> + hdr->retry =
> cpu_to_le32(info->control.rates[0].count << 8);
> - hdr->retry =
> cpu_to_le32((info->control.rates[0].count - 1) <<
> 8);
> + hdr->retry =
> cpu_to_le32(info->control.rates[0].count << 8);
These are the only lines which changed between the two versions of the patch - if removing the two -1's fixes the rate stuck (and these two lines are newly introduced), shouldn't this be a separate commit if it fixes a problem introduced by an earlier commit?
I did *not* (and do not) have a rate-stuck problem, but I am running a slightly mod version of the patch against 2.6.27, which doesn't have "rates[x].count" yet, with the retry_count struct member which seems to have since been removed.
--
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
