On Tue, Jul 26, 2011 at 3:05 PM, Vinod Koul <vkoul@xxxxxxxxxxxxx> wrote: > On Tue, 2011-07-26 at 08:57 +0100, Russell King - ARM Linux wrote: >> Here's an updated patch. >> >> 8<---------- >> From: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx> >> 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 <rmk+kernel@xxxxxxxxxxxxxxxx> >> --- >> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |