Re: [PATCH 0/7] omap: iommu migration, relocation and cleanups

2011-08-23 Thread Roedel, Joerg
On Wed, Aug 17, 2011 at 07:10:01PM -0400, Ohad Ben-Cohen wrote:
 1. Migrate OMAP's iommu driver to the generic IOMMU API, and move it
 to the dedicated iommu folder.
 
 2. Fix omap3isp appropriately so it doesn't break.
 
 3. Adapt OMAP's iovmm appropriately as well, because omap3isp still relies
 on it. The plan is eventually to remove iovmm and replace it with the
 upcoming IOMMU-based generic DMA-API. Move iovmm to the iommu folder too;
 it belongs there more than it belongs in the OMAP folders.
 
 4. Start cleaning up OMAP's iommu components towards the end goal, where
 1) omap-iommu no longer exports public API (and instead provides its
entire functionality via the generic IOMMU API).
 2) omap-iovmm is removed.

Besides the locking issue these patches look good to me. Please repost
when this is solved and I will put it in my tree then.

Joerg

-- 
AMD Operating System Research Center

Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach
General Managers: Alberto Bozzo, Andrew Bowd
Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632

--
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


Re: [PATCH 0/7] omap: iommu migration, relocation and cleanups

2011-08-23 Thread Ohad Ben-Cohen
On Tue, Aug 23, 2011 at 5:26 PM, Roedel, Joerg joerg.roe...@amd.com wrote:
 Besides the locking issue these patches look good to me. Please repost
 when this is solved and I will put it in my tree then.

Great, thanks !

I'll do a quick re-spin with the collected comments and ACKs and repost.

At least for the first few patches, which still change stuff in the OMAP
folders, we will also need Tony's ACK.

Tony can you please take a look ?

Thanks!
Ohad.
--
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


Re: [PATCH 0/7] omap: iommu migration, relocation and cleanups

2011-08-18 Thread Laurent Pinchart
Hi Ohad,

On Thursday 18 August 2011 01:10:01 Ohad Ben-Cohen wrote:
 1. Migrate OMAP's iommu driver to the generic IOMMU API, and move it
 to the dedicated iommu folder.
 
 2. Fix omap3isp appropriately so it doesn't break.
 
 3. Adapt OMAP's iovmm appropriately as well, because omap3isp still relies
 on it. The plan is eventually to remove iovmm and replace it with the
 upcoming IOMMU-based generic DMA-API. Move iovmm to the iommu folder too;
 it belongs there more than it belongs in the OMAP folders.
 
 4. Start cleaning up OMAP's iommu components towards the end goal, where
 1) omap-iommu no longer exports public API (and instead provides its
entire functionality via the generic IOMMU API).
 2) omap-iovmm is removed.
 
 Changes:
 1. Migrate iommu, iovmm and omap3isp in a single patch for bisectibility
 sake.
 2. Apply Laurent Pinchart's comments (thanks Laurent!)

You're welcome.

 3. Rebase to 3.1-rc2

For the OMAP3 ISP driver changes,

Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

-- 
Regards,

Laurent Pinchart
--
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


Re: [PATCH 0/7] omap: iommu migration, relocation and cleanups

2011-08-18 Thread Hiroshi DOYU
Hi Ohad,

From: ext Laurent Pinchart laurent.pinch...@ideasonboard.com
Subject: Re: [PATCH 0/7] omap: iommu migration, relocation and cleanups
Date: Thu, 18 Aug 2011 11:12:55 +0200

 Hi Ohad,
 
 On Thursday 18 August 2011 01:10:01 Ohad Ben-Cohen wrote:
 1. Migrate OMAP's iommu driver to the generic IOMMU API, and move it
 to the dedicated iommu folder.
 
 2. Fix omap3isp appropriately so it doesn't break.
 
 3. Adapt OMAP's iovmm appropriately as well, because omap3isp still relies
 on it. The plan is eventually to remove iovmm and replace it with the
 upcoming IOMMU-based generic DMA-API. Move iovmm to the iommu folder too;
 it belongs there more than it belongs in the OMAP folders.
 
 4. Start cleaning up OMAP's iommu components towards the end goal, where
 1) omap-iommu no longer exports public API (and instead provides its
entire functionality via the generic IOMMU API).
 2) omap-iovmm is removed.

Good migration path!

 Changes:
 1. Migrate iommu, iovmm and omap3isp in a single patch for bisectibility
 sake.
 2. Apply Laurent Pinchart's comments (thanks Laurent!)
 
 You're welcome.
 
 3. Rebase to 3.1-rc2
 
 For the OMAP3 ISP driver changes,
 
 Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

For the rest of omap iommu/iovmm patches, they look good:

Acked-by: Hiroshi DOYU hiroshi.d...@nokia.com
--
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