Search Linux Wireless

Re: [PATCH v2 08/18] wl1271: Add AP related definitions to HOST-FW interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux