Hello,
On Monday, June 20, 2011 10:36 AM Michal Nazarewicz wrote:
On Mon, 20 Jun 2011 09:50:06 +0200, Marek Szyprowski m.szyprowski@samsung.com wrote:
+static inline void dma_sync_single_for_cpu(struct device *dev,
I wouldn't really put inline here or in the function below.
dma_addr_t handle, size_t size, enum dma_data_direction dir)+{
- BUG_ON(!valid_dma_direction(dir));
- debug_dma_sync_single_for_cpu(dev, handle, size, dir);
- if (!dmabounce_sync_for_cpu(dev, handle, size, dir))
return;- __dma_single_dev_to_cpu(dma_to_virt(dev, handle), size, dir);
I know it is just copy'n'paste but how about:
This patch is just about moving the code between the files and I wanted just to show what's being changed and how. There is a final cleanup anyway in the separate patch. All these patches are meant to start the discussion about the way the dma mapping can be redesigned for further extensions with generic iommu support.
if (dmabounce_sync_for_cpu(dev, handle, size, dir)) __dma_single_dev_to_cpu(dma_to_virt(dev, handle), size, dir);
The above lines will be removed by the next patches in this series, so I really see no point in changing this.
(snipped)
Best regards