Re: [PATCH v2 0/6] OMAP IOMMU fixes

[Suman Anna <s-anna-l0cyMroinI0@public.gmane.org>]

On 04/03/2014 04:02 PM, Anna, Suman wrote:
> 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

Sorry, had a minor typo, I meant 5/6 w.r.t comment removal.

> 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(-)
>>
> 
> _______________________________________________
> iommu mailing list
> iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org
> https://lists.linuxfoundation.org/mailman/listinfo/iommu
>