On Tue, Jul 26, 2011 at 3:05 PM, Vinod Koul [off-list ref] wrote:

On Tue, 2011-07-26 at 08:57 +0100, Russell King - ARM Linux wrote:

quoted

Here's an updated patch.

8<----------
From: Russell King <redacted>
DMAEngine: Improve slave/cyclic documentation

Improve the documentation for the slave and cyclic DMA engine support
reformatting it for easier reading, adding further APIs, splitting it
into five steps, and including references to the documentation in
dmaengine.h.

Signed-off-by: Russell King <redacted>
---
?Documentation/dmaengine.txt | ?211 ++++++++++++++++++++++++++++++-------------
?1 files changed, 146 insertions(+), 65 deletions(-)

diff --git a/Documentation/dmaengine.txt b/Documentation/dmaengine.txt
index 5a0cb1e..8c2e888 100644
--- a/Documentation/dmaengine.txt
+++ b/Documentation/dmaengine.txt

@@ -10,87 +10,168 @@

?Below is a guide to device driver writers on how to use the Slave-DMA API of the
?DMA Engine. This is applicable only for slave DMA usage only.

-The slave DMA usage consists of following steps
+The slave DMA usage consists of following steps:
?1. Allocate a DMA slave channel
?2. Set slave and controller specific parameters
?3. Get a descriptor for transaction
?4. Submit the transaction and wait for callback notification
+5. Issue pending requests

Thanks Russell,

Applied with change to 4 above. Moved "and wait for callback
notification" to 5.

Dear Vinod,

 Since it came from the RMK, most probably it'll be the best.

But applying patches upon personal timeout seems very dangerous.

People not responding doesn't mean only either people agree completely
or they don't care. Some might be interested but too busy with current tasks
that they need time to check... please make some policy for such cases.

It already happened with the patch from Rob, which you probably have to
revert.

IMHO, if nobody replied, maybe you could first ack the patch and wait
for, say a week, before applying?
That way people will know they have to hurry if they care otherwise
the patch is going upstream as such.

Thanks
-Jassi