Readability fixes for rtw_recv.c

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Some readability fixes for rtw_ieee80211.c - more is needed

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused recvframe_push() function

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Do not maintain phead/ptail/pend/pdata/len in struct recv_buf

There is no point maintaining these pointers and len since we don't
use them at all. After all Linux uses skbuffs and not silly mbufs.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Eliminate unnecessary pbuf entry from struct recv_buf

This pointer was simply a mapping to skb->data

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused dma_transfer_addr

We don't need to track the dma addresses for a USB driver

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
irp_pending is always set to false

No point checking if irp_pending == false

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused struct intf_priv definition

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused alloc_sz from struct recv_buf

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused spin lock from struct recv_buf

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Don't bother explicitly aligning the first recv_buf

There is no point explicitly aligning the first recv_buf and not the
rest of them. In particular as these are Linux kernel structs with no
DMA alignment requirements.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused fragcnt entry from struct recv_frame

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused pre_recv_entry() function

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
CONFIG_80211N_HT is always defined

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Strip out DBX_RX_DROP_FRAME code

This is debug code that isn't used and it simply makes it a lot harder
to read the code, so strip it out.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
struct recv_frame->rx_end isn't used for anything real

Get rid of it

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Fold recv_func() into rtw_recv_entry()

rtw_recv_entry() was really just a wrapper for calling recv_func(). No
point in keeping them as two functions.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
recvframe_defrag(): pfhdr and prframe are the same pointer

Lets not obfuscate the code more than necessary

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Don't typecast recv_frame * to u8 * and back when calling _decrypt functions

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
More cleaning up of rx buffer handling

Note this is still work in progress

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>