On Thu, May 01, 2014 at 12:48:57AM +0200, Laurent Pinchart wrote: > Hi Sakari, > > On Thursday 01 May 2014 01:45:47 Sakari Ailus wrote: > > Hi Laurent, > > > > Thanks for the set! I've been looking forward to see this! :) > > > > On Mon, Apr 21, 2014 at 02:28:47PM +0200, Laurent Pinchart wrote: > > > The macro is meant to test whether the statistics engine uses an > > > external DMA engine to transfer data or supports DMA directly. As both > > > cases will be supported by DMA coherent buffers rename the macro to > > > ISP_STAT_USES_DMAENGINE for improved clarity. > > > > Both use DMA, but the ISP just implements its own. How about calling the > > macro ISP_STAT_USES_SYSTEM_DMA instead? Up to you. > > DMA engine is the Linux name for the system DMA engine API. It might indeed be The documentation does not use that term at least. It speaks of DMA mapping instead. The DMA being used for the transfers in that case is OMAP system DMA, not ISP DMA. > slightly generic, but I'm not too sure whether ISP_STAT_USES_SYSTEM_DMA would > be more descriptive. I suppose it depends on the background of the reader :-) > If you insist I can change it. I think ISP_STAT_USES_SYSTEM_DMA is simply better. Up to you. -- Regards, Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
