On Friday, September 12, 2014 4:20 AM, Ian Abbott wrote: > "addi_apci_3120" allocates more pages of DMA buffer than it uses, may > allocate half a double-buffer it does not use because it's the "wrong" > half that it managed to allocate (unlikely), and relies on virt_to_bus() > to treat generic kernel memory from get_free_pages() as coherent DMA > memory. Correct the issues, using dma_alloc_coherent() to allocate the > DMA buffers. > > 1) staging: comedi: addi_apci_3120: don't allocate 2nd DMA buffer on failure > 2) staging: comedi: addi_apci_3120: don't overallocate DMA buffer > 3) staging: comedi: addi_apci_3120: use dma_alloc_coherent() > 4) staging: comedi: addi_apci_3120: simplify setting of devpriv->us_UseDma > > drivers/staging/comedi/Kconfig | 2 +- > .../staging/comedi/drivers/addi-data/addi_common.h | 3 +- > drivers/staging/comedi/drivers/addi_apci_3120.c | 44 > ++++++++++------------ > 3 files changed, 22 insertions(+), 27 deletions(-)
Reviewed-by: H Hartley Sweeten <hswee...@visionengravers.com> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
