Commit f126cba4 authored by Jochen Friedrich's avatar Jochen Friedrich Committed by John W. Linville

rt2x00: Fix txdone_entry_desc_flags

txdone_entry_desc_flags is used with __set_bit and test_bit which
bit-shift the values, so don't bit-shift the flags in the enum.
Also make sure flags are initialized before being used.
Signed-off-by: default avatarJochen Friedrich <jochen@scram.de>
Signed-off-by: default avatarIvo van Doorn <IvDoorn@gmail.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 430cd47f
...@@ -173,10 +173,10 @@ struct rxdone_entry_desc { ...@@ -173,10 +173,10 @@ struct rxdone_entry_desc {
* frame transmission failed due to excessive retries. * frame transmission failed due to excessive retries.
*/ */
enum txdone_entry_desc_flags { enum txdone_entry_desc_flags {
TXDONE_UNKNOWN = 1 << 0, TXDONE_UNKNOWN,
TXDONE_SUCCESS = 1 << 1, TXDONE_SUCCESS,
TXDONE_FAILURE = 1 << 2, TXDONE_FAILURE,
TXDONE_EXCESSIVE_RETRY = 1 << 3, TXDONE_EXCESSIVE_RETRY,
}; };
/** /**
......
...@@ -181,6 +181,7 @@ static void rt2x00usb_interrupt_txdone(struct urb *urb) ...@@ -181,6 +181,7 @@ static void rt2x00usb_interrupt_txdone(struct urb *urb)
* (Only indirectly by looking at the failed TX counters * (Only indirectly by looking at the failed TX counters
* in the register). * in the register).
*/ */
txdesc.flags = 0;
if (!urb->status) if (!urb->status)
__set_bit(TXDONE_UNKNOWN, &txdesc.flags); __set_bit(TXDONE_UNKNOWN, &txdesc.flags);
else else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment