[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1285370032-16937-1-git-send-email-iws@ovro.caltech.edu>
Date: Fri, 24 Sep 2010 16:13:50 -0700
From: "Ira W. Snyder" <iws@...o.caltech.edu>
To: linux-kernel@...r.kernel.org
Cc: Dan Williams <dan.j.williams@...el.com>,
linuxppc-dev@...ts.ozlabs.org
Subject: [PATCH RFCv2 0/2] dma: add support for sg-to-sg transfers
This series adds support for scatterlist to scatterlist transfers to the
generic DMAEngine API. I have hidden it behind a configuration option to
allow specific drivers that need this functionality to enable it.
This series is intended to lay the groundwork for further changes to the
series titled "CARMA Board Support". That series will be updated when I
have time and hardware to test with.
This series has not been runtime tested yet. I am posting it only to
gain comments before I spend the effort to update the driver that
depends on this.
To help reviewers, I'd like to comment on the architecture of
dma_async_memcpy_sg_to_sg(). It explicitly avoids using descriptor
chaining due to the way that feature interacts with the fsldma
controller's external start feature. To use the external start feature
properly, the in-memory descriptor chain must not be fragmented into
multiple smaller chains. This is what is achieved by submitting all
descriptors without using chaining.
An alternative implementation would create a device_prep_sg_to_sg()
function, and use that to allocate all descriptors in one shot. That
implementation would be safer against allocation failures than this one.
I would recommend against committing this until I've tested it on real
hardware.
Ira W. Snyder (2):
dmaengine: add support for scatterlist to scatterlist transfers
fsldma: use generic support for scatterlist to scatterlist transfers
arch/powerpc/include/asm/fsldma.h | 115 ++------------------
drivers/dma/Kconfig | 4 +
drivers/dma/dmaengine.c | 119 ++++++++++++++++++++
drivers/dma/fsldma.c | 219 +++++++------------------------------
include/linux/dmaengine.h | 10 ++
5 files changed, 181 insertions(+), 286 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists