Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-10 Thread Laurent Pinchart
Hi Joerg, On Wednesday 09 April 2014 17:08:31 Joerg Roedel wrote: > On Tue, Apr 08, 2014 at 05:02:37PM +0200, Laurent Pinchart wrote: > > On Tuesday 08 April 2014 15:43:22 Joerg Roedel wrote: > > > Who is "someone" in this case? > > > > That's exactly the problem :-) The ARM DMA API implementatio

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-09 Thread Joerg Roedel
On Tue, Apr 08, 2014 at 05:02:37PM +0200, Laurent Pinchart wrote: > On Tuesday 08 April 2014 15:43:22 Joerg Roedel wrote: > > Who is "someone" in this case? > > That's exactly the problem :-) The ARM DMA API implementation doesn't care > who > that "someone" is. Existing implementations call tho

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-08 Thread Laurent Pinchart
Hi Joerg, On Tuesday 08 April 2014 15:43:22 Joerg Roedel wrote: > On Tue, Apr 08, 2014 at 02:50:38PM +0200, Laurent Pinchart wrote: > > I agree with you, the two levels are already present, but there's still > > rough edges that we need to soften. > > > > The ARM DMA API implementation requires s

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-08 Thread Joerg Roedel
Hi Laurent, On Tue, Apr 08, 2014 at 02:50:38PM +0200, Laurent Pinchart wrote: > I agree with you, the two levels are already present, but there's still rough > edges that we need to soften. > > The ARM DMA API implementation requires someone to create the VA space > mapping > by calling arm_i

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-08 Thread Laurent Pinchart
Hi Marek, On Friday 04 April 2014 14:23:57 Marek Szyprowski wrote: > Hello, > > I'm sorry for a delay, I've got back from my holidays and I'm still busy > trying to get thought all the emails. No worries. > On 2014-03-17 23:44, Laurent Pinchart wrote: > > On Monday 17 March 2014 14:58:24 Suman

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-08 Thread Laurent Pinchart
Hi Joerg, On Friday 04 April 2014 12:18:11 Joerg Roedel wrote: > On Fri, Mar 14, 2014 at 12:00:16PM +0100, Laurent Pinchart wrote: > > Right, we indeed need two levels of API, one for drivers such as > > remoteproc that need direct control of the IOMMU, and one for drivers that > > only need to ma

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-04 Thread Marek Szyprowski
Hello, I'm sorry for a delay, I've got back from my holidays and I'm still busy trying to get thought all the emails. On 2014-03-17 23:44, Laurent Pinchart wrote: Hi Suman and Sakari, On Monday 17 March 2014 14:58:24 Suman Anna wrote: > On 03/16/2014 04:54 PM, Sakari Ailus wrote: > > On Fri,

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-04-04 Thread Joerg Roedel
On Fri, Mar 14, 2014 at 12:00:16PM +0100, Laurent Pinchart wrote: > Right, we indeed need two levels of API, one for drivers such as remoteproc > that need direct control of the IOMMU, and one for drivers that only need to > map buffers without any additional requirement. In the second case the

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-17 Thread Laurent Pinchart
Hi Suman and Sakari, On Monday 17 March 2014 14:58:24 Suman Anna wrote: > On 03/16/2014 04:54 PM, Sakari Ailus wrote: > > On Fri, Mar 14, 2014 at 12:00:16PM +0100, Laurent Pinchart wrote: > >> Hi Suman, > >> > >> (CC'ing Joerg Roedel and Marek Szyprowski for the core IOMMU discussion) > >> > >>

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-17 Thread Suman Anna
Hi Laurent, Sakari, On 03/16/2014 04:54 PM, Sakari Ailus wrote: Hi Laurent and Suman, On Fri, Mar 14, 2014 at 12:00:16PM +0100, Laurent Pinchart wrote: Hi Suman, (CC'ing Joerg Roedel and Marek Szyprowski for the core IOMMU discussion) On Thursday 13 March 2014 21:33:37 Suman Anna wrote: On

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-16 Thread Sakari Ailus
Hi Laurent and Suman, On Fri, Mar 14, 2014 at 12:00:16PM +0100, Laurent Pinchart wrote: > Hi Suman, > > (CC'ing Joerg Roedel and Marek Szyprowski for the core IOMMU discussion) > > On Thursday 13 March 2014 21:33:37 Suman Anna wrote: > > On 03/07/2014 06:46 PM, Laurent Pinchart wrote: > > > Hell

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-14 Thread Laurent Pinchart
Hi Suman, (CC'ing Joerg Roedel and Marek Szyprowski for the core IOMMU discussion) On Thursday 13 March 2014 21:33:37 Suman Anna wrote: > On 03/07/2014 06:46 PM, Laurent Pinchart wrote: > > Hello, > > > > This patch set fixes miscellaneous issues with the OMAP IOMMU driver, > > found when trying

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-13 Thread Suman Anna
Hi Laurent, On 03/07/2014 06:46 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 5/5, while the other patches fix smaller

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-12 Thread Laurent Pinchart
Hello, Suman, Florian, any opinion regarding this patch series ? Your opinion on the architecture-related question below would be appreciated as well, but I'd like to get this merged first (and especially patch 5/5) in the not too distant future to push OMAP3 ISP patches that depend on it. On

Re: [PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-12 Thread Sakari Ailus
Hi Laurent, Laurent Pinchart wrote: along with a patch series that ports the OMAP3 ISP driver to the DMA API. I will submit that one for review once the IOMMU patches get accepted and after fixing a couple of remaining bugs (I'm aware that I have broken userspace PFNMAP buffers). Laurent Pincha

[PATCH 0/5] OMAP IOMMU fixes and IOMMU architecture questions

2014-03-07 Thread Laurent Pinchart
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 5/5, while the other patches fix smaller problems that I've noticed when reading the code, without exper