Hi Laurent, On 04/03/2014 03:01 PM, Laurent Pinchart wrote: > Hello, > > This patch set fixes miscellaneous issues with the OMAP IOMMU driver, found > when trying to port the OMAP3 ISP away from omap-iovmm to the ARM DMA API. The > biggest issue is fixed by patch 6/6, while the other patches fix smaller > problems that I've noticed when reading the code, without experiencing them at > runtime. > > I believe I've addressed all comments received for v1. Suman, could you please > have a quick look at ack the patches you haven't acked yet ? I'll then send a > pull request.
Thanks for the revised series. Just acked Patch 4/6. Patch 3/6 is a simple comment removal, and I am good with that change. First 3 are already are from me, so I guess you need to add your sign-off before you send out the pull request. regards Suman > > Changes since v1: > > - Dropped the "iommu/omap: Use the cache cleaning API" and "iommu/omap: Flush > the TLB only after updating translation table entries" patches > - Reworked "iommu/omap: Fix 'no page for' debug message in flush_iotlb_page()" > to just break out of the loop > - Added patches 1/6 to 3/6 > > Laurent Pinchart (3): > iommu/omap: Fix 'no page for' debug message in flush_iotlb_page() > iommu/omap: Remove comment about supporting single page mappings only > iommu/omap: Fix map protection value handling > > Suman Anna (3): > iommu/omap: Correct init value of iotlb_entry valid field > iommu/omap: Remove omap_iommu_domain_has_cap() function > iommu/omap: Move to_iommu definition from omap-iopgtable.h > > drivers/iommu/omap-iommu.c | 31 ++++++++++++------------------- > drivers/iommu/omap-iopgtable.h | 3 --- > 2 files changed, 12 insertions(+), 22 deletions(-) > -- 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
