On Tue January 25 2011 19:38:03 Bob Copeland wrote: > On Tue, Jan 25, 2011 at 02:23:31PM +0900, Bruno Randolf wrote: > > > The ath5k version of ieee80211_generic_frame_duration() returns > > > an __le16 for standard modes but a cpu-endian int for turbo/half/ > > > quarter rates. Make it always return cpu-endian values. > > > > I wonder what effect this has in ath5k_hw_set_ifs_intervals() where > > ack_tx_time was little endian before... > > Good point, I didn't think about that. My guess is EIFS would have > been much too large on big endian arches before the patch. But then > EIFS resets after successful rx so maybe our many BE users (heh) > wouldn't have noticed except on very idle channels... Allright, just the EIFS is affected - shouldn't be a big deal. Anyhow it's correct now and was wrong before. But you meant on a very busy channel, right? I'm on a BE board ;) I can try to see what difference it makes tomorrow. bruno -- 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