On Tue, 2009-08-11 at 08:47 +0200, Gabor Juhos wrote: > The current code uses wrong binary operator for masking, > and the shift values for the 'tx_max_num_spatial_streams' and > 'tx_unequal_modulation' fields are off-by-one. > - tx_max_num_spatial_streams = (mcs[12] | ((1 << 3) | (1 << 4))) + 1; > - tx_unequal_modulation = !!(mcs[12] & (1 << 5)); > + tx_max_num_spatial_streams = (mcs[12] & ((1 << 2) | (1 << 3))) + 1; > + tx_unequal_modulation = !!(mcs[12] & (1 << 4)); Are you sure? The "Supported MCS Set" field is defined as a series of _bits_, and the bits are transmitted the other way around within each byte. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
