> -----Original Message----- > From: Tony Lindgren [mailto:t...@atomide.com] > Sent: Monday, May 25, 2009 9:59 PM > To: Shilimkar, Santosh > Cc: Russell King - ARM Linux; > linux-arm-ker...@lists.arm.linux.org.uk; linux-omap@vger.kernel.org > Subject: Re: [PATCH 07/10] ARM: OMAP2/3: sDMA: Correct > omap_request_dma_chain(), v2 > > * Shilimkar, Santosh <santosh.shilim...@ti.com> [090525 02:40]: > > > -----Original Message----- > > > From: Russell King - ARM Linux [mailto:li...@arm.linux.org.uk] > > > Sent: Monday, May 25, 2009 3:04 PM > > > To: Tony Lindgren > > > Cc: linux-arm-ker...@lists.arm.linux.org.uk; > > > linux-omap@vger.kernel.org; Shilimkar, Santosh > > > Subject: Re: [PATCH 07/10] ARM: OMAP2/3: sDMA: Correct > > > omap_request_dma_chain() > > > > > > On Tue, May 19, 2009 at 04:40:07PM -0700, Tony Lindgren wrote: > > > > From: Santosh Shilimkar <santosh.shilim...@ti.com> > > > > > > > > Original OMAP DMA chaining design has chain_id as one of > > > the callback > > > > parameters. Patch 538528de0cb256f65716ab2e9613d9e920f97fe2 broke > > > > the desing. > > > > > > > > Change the callback to return the logical channel number and not > > > > chain_id. > > > > > > This patch does not make a material change. It merely > > > changes the name > > > used in a couple function pointer prototype. This has no > > > impact on the > > > generated code - iow, it's cosmetic. > > Yes it does look cosmetic but if you see the API signature, > it corrects the callback parameters which actually the DMA lib sends. > > > > The current API signature is misleading for chained DMA > transfers where the actual driver passes 'channel number' as > one of the callback parameter instead of chain_id which was > the original design. > > Here's this one with more descriptive comments.
Thanks Tony !! Regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
