On Wed, Dec 22, 2010 at 23:36, Luciano Coelho <luciano.coelho@xxxxxxxxx> wrote: > On Wed, 2010-12-22 at 16:27 +0200, ext Arik Nemtsov wrote: >> Change structures in a non-destructive manner. This means no >> changes in size or location of existing members used by STA. >> >> Signed-off-by: Arik Nemtsov <arik@xxxxxxxxxx> >> --- > > [...] > >> diff --git a/drivers/net/wireless/wl12xx/rx.h b/drivers/net/wireless/wl12xx/rx.h >> index 3abb26f..19fd591 100644 >> --- a/drivers/net/wireless/wl12xx/rx.h >> +++ b/drivers/net/wireless/wl12xx/rx.h >> @@ -86,10 +86,11 @@ >> /* >> * RX Descriptor status >> * >> - * Bits 0-2 - status >> - * Bits 3-7 - reserved >> + * Bits 0-2 - error code >> + * Bits 3-5 - process_id tag (AP mode FW) >> + * Bits 6-7 - reserved >> */ >> -#define WL1271_RX_DESC_STATUS_MASK 0x07 >> +#define WL1271_RX_DESC_STATUS_MASK 0x03 > > What happened with bit 2? You're right. Actually ported over a bug from the original TI driver. Since only errors 1-3 are defined this still works. But its a bug anyway. Will revert. Regards, Arik -- 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