On Mon, 2007-02-05 at 13:55 +0100, Michael Buesch wrote:

> -#define BCM43xx_TXSTAT_FLAG_ACK              0x01
> -//TODO #define BCM43xx_TXSTAT_FLAG_???       0x02
> -//TODO #define BCM43xx_TXSTAT_FLAG_???       0x04
> -//TODO #define BCM43xx_TXSTAT_FLAG_???       0x08
> -//TODO #define BCM43xx_TXSTAT_FLAG_???       0x10
> -#define BCM43xx_TXSTAT_FLAG_IGNORE   0x20
> -//TODO #define BCM43xx_TXSTAT_FLAG_???       0x40
> -//TODO #define BCM43xx_TXSTAT_FLAG_???       0x80
> +#define BCM43xx_TXSTAT_FLAG_AMPDU    0x20
> +#define BCM43xx_TXSTAT_FLAG_INTER    0x40

That's wrong. You see, in the specs we say:

0x0040  intermediate status notification
0x0020 status is for an AMPDU (afterburner) 
0x001C suppression reason 
0x0002 ack received 
0x0001 valid bit 

But in the v3 code this whole stuff was shifted down by one as you can
see by the code you removed:
#define BCM43xx_TXSTAT_FLAG_ACK         0x01

johannes

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Bcm43xx-dev mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to