rt2x00: Align ieee80211 header to 4-byte boundary for PCI devices
authorIvo van Doorn <ivdoorn@gmail.com>
Sat, 8 Aug 2009 21:53:47 +0000 (23:53 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 14 Aug 2009 13:13:52 +0000 (09:13 -0400)
Some hardware require the ieee80211 header to be aligned to a
4-byte boundary before mapping it to the DMA. Otherwise some
frames (like beacons) will not be send out correctly by the
device.

Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rt2x00/rt2x00queue.c

index 65435c9fe4bcee7a39fa3d1ad7576a5c49c8f848..e67e339d9dfdf75c692fc9aa4e42659fa0bbbf4d 100644 (file)
@@ -453,9 +453,21 @@ int rt2x00queue_write_tx_frame(struct data_queue *queue, struct sk_buff *skb)
                        rt2x00crypto_tx_remove_iv(skb, &txdesc);
        }
 
+       /*
+        * When DMA allocation is required we should guarentee to the
+        * driver that the DMA is aligned to a 4-byte boundary.
+        * Aligning the header to this boundary can be done by calling
+        * rt2x00queue_payload_align with the header length of 0.
+        * However some drivers require L2 padding to pad the payload
+        * rather then the header. This could be a requirement for
+        * PCI and USB devices, while header alignment only is valid
+        * for PCI devices.
+        */
        if (test_bit(DRIVER_REQUIRE_L2PAD, &queue->rt2x00dev->flags))
                rt2x00queue_payload_align(entry->skb, true,
                                          txdesc.header_length);
+       else if (test_bit(DRIVER_REQUIRE_DMA, &queue->rt2x00dev->flags))
+               rt2x00queue_payload_align(entry->skb, false, 0);
 
        /*
         * It could be possible that the queue was corrupted and this