Re: [PATCH v2 01/26] omap3isp: stat: Rename IS_COHERENT_BUF to ISP_STAT_USES_DMAENGINE
Hi Sakari, On Thursday 01 May 2014 14:15:42 Sakari Ailus wrote: > 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. DMA mapping and DMA engine are actually two different things. The former manages DMA memory, while the latter handles the DMA engine hardware. Only the former is documented in the kernel at the moment though (but I expect that to change soon). > > 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, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 01/26] omap3isp: stat: Rename IS_COHERENT_BUF to ISP_STAT_USES_DMAENGINE
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.ai...@iki.fi XMPP: sai...@retiisi.org.uk -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 01/26] omap3isp: stat: Rename IS_COHERENT_BUF to ISP_STAT_USES_DMAENGINE
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 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. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 01/26] omap3isp: stat: Rename IS_COHERENT_BUF to ISP_STAT_USES_DMAENGINE
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. -- Regards, Sakari Ailus e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
