Commit 996bc370 authored by Larry Finger's avatar Larry Finger Committed by John W. Linville

b43: Relax requirement for descriptors to be in the DMA zone

When 64-bit DMA was first used, there were problems with the
BCM4311 (14e4:4311). The problem was "fixed" by using the GFP_DMA
flag in the allocation of coherent ring descriptor memory.

The original problem is now believed to have been due to bugs in
the 64-bit DMA implementation in the rest of the kernel, and that
those bugs have been fixed. Accordingly, the requirement for the
descriptors to be in the DMA zone is relaxed.

Bounce buffers are left in the DMA zone.
Signed-off-by: default avatarLarry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: default avatarMichael Buesch <m@bues.ch>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 1eb85d63
...@@ -423,14 +423,7 @@ static int alloc_ringmemory(struct b43_dmaring *ring) ...@@ -423,14 +423,7 @@ static int alloc_ringmemory(struct b43_dmaring *ring)
* has shown that 4K is sufficient for the latter as long as the buffer * has shown that 4K is sufficient for the latter as long as the buffer
* does not cross an 8K boundary. * does not cross an 8K boundary.
* *
* For unknown reasons - possibly a hardware error - the BCM4311 rev
* 02, which uses 64-bit DMA, needs the ring buffer in very low memory,
* which accounts for the GFP_DMA flag below.
*
* The flags here must match the flags in free_ringmemory below!
*/ */
if (ring->type == B43_DMA_64BIT)
flags |= GFP_DMA;
ring->descbase = dma_alloc_coherent(ring->dev->dev->dma_dev, ring->descbase = dma_alloc_coherent(ring->dev->dev->dma_dev,
B43_DMA_RINGMEMSIZE, B43_DMA_RINGMEMSIZE,
&(ring->dmabase), flags); &(ring->dmabase), flags);
......
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