On Friday 15 April 2011, Russell King - ARM Linux wrote: > On Thu, Apr 14, 2011 at 04:52:48PM -0500, Fernando Guzman Lugo wrote: > > From: Ramesh Gupta <grgupta@xxxxxx> > > > > This patch is to flush the iommu page table entries from L1 and L2 > > caches using dma_map_single. This also simplifies the implementation > > by removing the functions flush_iopgd_range/flush_iopte_range. > > No. This usage is just wrong. If you're going to use the DMA API then > unmap it, otherwise the DMA API debugging will go awol. It's also completely upside-down: The iommu support should provide interfaces using the dma-mapping API, not use that API to provide a machine specific version of the generic interface. As far as I can tell, nothing actually uses these drivers, maybe we should just remove them before we get any code in the mainline kernel that depends on it. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html