From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932408Ab1LEPSL (ORCPT ); Mon, 5 Dec 2011 10:18:11 -0500 Received: from ch1ehsobe004.messaging.microsoft.com ([216.32.181.184]:29210 "EHLO ch1outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932351Ab1LEPSH convert rfc822-to-8bit (ORCPT ); Mon, 5 Dec 2011 10:18:07 -0500 X-SpamScore: -28 X-BigFish: VPS-28(zz9371K936eK146fK1432N98dKzz1202hzz8275ch15d4R8275bh8275dhz2dh668h839h) X-Forefront-Antispam-Report: CIP:163.181.249.109;KIP:(null);UIP:(null);IPV:NLI;H:ausb3twp02.amd.com;RD:none;EFVD:NLI X-FB-SS: 13, X-WSS-ID: 0LVQL61-02-6R5-02 X-M-MSG: Date: Mon, 5 Dec 2011 16:18:00 +0100 From: "Roedel, Joerg" To: Ohad Ben-Cohen CC: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Hiroshi DOYU , Laurent Pinchart , "iommu@lists.linux-foundation.org" , Arnd Bergmann , "linux-kernel@vger.kernel.org" , Cousson Benoit , Kevin Hilman , Tony Lindgren , Grant Likely Subject: Re: [RFC 0/5] OMAP groundwork for IOMMU-based DMA API Message-ID: <20111205151800.GD25178@amd.com> References: <1316948337-7924-1-git-send-email-ohad@wizery.com> <20111115103926.GA5627@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Content-Transfer-Encoding: 8BIT X-OriginatorOrg: amd.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Dec 05, 2011 at 04:21:21PM +0200, Ohad Ben-Cohen wrote: > Hi Joerg, > > On Tue, Nov 15, 2011 at 12:39 PM, Roedel, Joerg wrote: > > On Sun, Sep 25, 2011 at 06:58:52AM -0400, Ohad Ben-Cohen wrote: > >> Ohad Ben-Cohen (5): > >>   ARM: dev_archdata: add private iommu extension > >>   ARM: OMAP: omap_device: add a method to set iommu private archdata > >>   ARM: OMAP: iommu: declare a private iommu binding struct > >>   ARM: OMAP3: bind omap3isp_device to its iommu device > >>   iommu/omap: eliminate the public omap_find_iommu_device() method > > > > It doesn't apply cleanly, can you please rebase the code, collect the > > Acked-by's and resend? > > I have collected Tony's and Laurent's Acked-by's and amended them to > the commit logs. > > I have also made sure it applies cleanly to a recent 3.2, and tested > it with OMAP4/remoteproc. I couldn't test it with omap3isp though, > because I'm gated by (what looks like a) 3.2 omap3isp regression (see > http://www.spinics.net/lists/linux-omap/msg60510.html). I did test it > of course with omap3isp and 3.1, so I don't expect any omap3isp/iommu > issues with these patches. Anyway, when the omap3isp issue will be > resolved I'll make sure there isn't any iommu regression. > > Please pull, thanks a lot! > > The following changes since commit cfcfc9eca2bcbd26a8e206baeb005b055dbf8e37: > > Linux 3.2-rc2 (2011-11-15 15:02:59 -0200) > > are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/ohad/linux.git for-joerg > > Ohad Ben-Cohen (3): > ARM: OMAP: iommu: declare a private iommu binding struct > ARM: OMAP3: bind omap3isp_device to its iommu device > iommu/omap: eliminate the public omap_find_iommu_device() method > > arch/arm/mach-omap2/devices.c | 7 ++++ > arch/arm/plat-omap/include/plat/iommu.h | 31 +++++++++++++++-- > arch/arm/plat-omap/include/plat/iovmm.h | 12 +++--- > drivers/iommu/omap-iommu.c | 58 ++++++++++++++----------------- > drivers/iommu/omap-iovmm.c | 31 +++++++++++------ > drivers/media/video/omap3isp/isp.c | 30 +++------------- > drivers/media/video/omap3isp/isp.h | 2 - > drivers/media/video/omap3isp/ispccdc.c | 18 +++++----- > drivers/media/video/omap3isp/ispstat.c | 8 ++-- > drivers/media/video/omap3isp/ispvideo.c | 4 +- > 10 files changed, 107 insertions(+), 94 deletions(-) Pulled into arm/omap, thanks Ohad. -- 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Roedel, Joerg" Subject: Re: [RFC 0/5] OMAP groundwork for IOMMU-based DMA API Date: Mon, 5 Dec 2011 16:18:00 +0100 Message-ID: <20111205151800.GD25178@amd.com> References: <1316948337-7924-1-git-send-email-ohad@wizery.com> <20111115103926.GA5627@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Ohad Ben-Cohen Cc: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Hiroshi DOYU , Laurent Pinchart , "iommu@lists.linux-foundation.org" , Arnd Bergmann , "linux-kernel@vger.kernel.org" , Cousson Benoit , Kevin Hilman , Tony Lindgren , Grant Likely List-Id: linux-omap@vger.kernel.org On Mon, Dec 05, 2011 at 04:21:21PM +0200, Ohad Ben-Cohen wrote: > Hi Joerg, >=20 > On Tue, Nov 15, 2011 at 12:39 PM, Roedel, Joerg wrote: > > On Sun, Sep 25, 2011 at 06:58:52AM -0400, Ohad Ben-Cohen wrote: > >> Ohad Ben-Cohen (5): > >> =A0 ARM: dev_archdata: add private iommu extension > >> =A0 ARM: OMAP: omap_device: add a method to set iommu private arch= data > >> =A0 ARM: OMAP: iommu: declare a private iommu binding struct > >> =A0 ARM: OMAP3: bind omap3isp_device to its iommu device > >> =A0 iommu/omap: eliminate the public omap_find_iommu_device() meth= od > > > > It doesn't apply cleanly, can you please rebase the code, collect t= he > > Acked-by's and resend? >=20 > I have collected Tony's and Laurent's Acked-by's and amended them to > the commit logs. >=20 > I have also made sure it applies cleanly to a recent 3.2, and tested > it with OMAP4/remoteproc. I couldn't test it with omap3isp though, > because I'm gated by (what looks like a) 3.2 omap3isp regression (see > http://www.spinics.net/lists/linux-omap/msg60510.html). I did test it > of course with omap3isp and 3.1, so I don't expect any omap3isp/iommu > issues with these patches. Anyway, when the omap3isp issue will be > resolved I'll make sure there isn't any iommu regression. >=20 > Please pull, thanks a lot! >=20 > The following changes since commit cfcfc9eca2bcbd26a8e206baeb005b055d= bf8e37: >=20 > Linux 3.2-rc2 (2011-11-15 15:02:59 -0200) >=20 > are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/ohad/linux.git for-jo= erg >=20 > Ohad Ben-Cohen (3): > ARM: OMAP: iommu: declare a private iommu binding struct > ARM: OMAP3: bind omap3isp_device to its iommu device > iommu/omap: eliminate the public omap_find_iommu_device() metho= d >=20 > arch/arm/mach-omap2/devices.c | 7 ++++ > arch/arm/plat-omap/include/plat/iommu.h | 31 +++++++++++++++-- > arch/arm/plat-omap/include/plat/iovmm.h | 12 +++--- > drivers/iommu/omap-iommu.c | 58 ++++++++++++++-------= ---------- > drivers/iommu/omap-iovmm.c | 31 +++++++++++------ > drivers/media/video/omap3isp/isp.c | 30 +++------------- > drivers/media/video/omap3isp/isp.h | 2 - > drivers/media/video/omap3isp/ispccdc.c | 18 +++++----- > drivers/media/video/omap3isp/ispstat.c | 8 ++-- > drivers/media/video/omap3isp/ispvideo.c | 4 +- > 10 files changed, 107 insertions(+), 94 deletions(-) Pulled into arm/omap, thanks Ohad. --=20 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg.Roedel@amd.com (Roedel, Joerg) Date: Mon, 5 Dec 2011 16:18:00 +0100 Subject: [RFC 0/5] OMAP groundwork for IOMMU-based DMA API In-Reply-To: References: <1316948337-7924-1-git-send-email-ohad@wizery.com> <20111115103926.GA5627@amd.com> Message-ID: <20111205151800.GD25178@amd.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Dec 05, 2011 at 04:21:21PM +0200, Ohad Ben-Cohen wrote: > Hi Joerg, > > On Tue, Nov 15, 2011 at 12:39 PM, Roedel, Joerg wrote: > > On Sun, Sep 25, 2011 at 06:58:52AM -0400, Ohad Ben-Cohen wrote: > >> Ohad Ben-Cohen (5): > >> ? ARM: dev_archdata: add private iommu extension > >> ? ARM: OMAP: omap_device: add a method to set iommu private archdata > >> ? ARM: OMAP: iommu: declare a private iommu binding struct > >> ? ARM: OMAP3: bind omap3isp_device to its iommu device > >> ? iommu/omap: eliminate the public omap_find_iommu_device() method > > > > It doesn't apply cleanly, can you please rebase the code, collect the > > Acked-by's and resend? > > I have collected Tony's and Laurent's Acked-by's and amended them to > the commit logs. > > I have also made sure it applies cleanly to a recent 3.2, and tested > it with OMAP4/remoteproc. I couldn't test it with omap3isp though, > because I'm gated by (what looks like a) 3.2 omap3isp regression (see > http://www.spinics.net/lists/linux-omap/msg60510.html). I did test it > of course with omap3isp and 3.1, so I don't expect any omap3isp/iommu > issues with these patches. Anyway, when the omap3isp issue will be > resolved I'll make sure there isn't any iommu regression. > > Please pull, thanks a lot! > > The following changes since commit cfcfc9eca2bcbd26a8e206baeb005b055dbf8e37: > > Linux 3.2-rc2 (2011-11-15 15:02:59 -0200) > > are available in the git repository at: > git://git.kernel.org/pub/scm/linux/kernel/git/ohad/linux.git for-joerg > > Ohad Ben-Cohen (3): > ARM: OMAP: iommu: declare a private iommu binding struct > ARM: OMAP3: bind omap3isp_device to its iommu device > iommu/omap: eliminate the public omap_find_iommu_device() method > > arch/arm/mach-omap2/devices.c | 7 ++++ > arch/arm/plat-omap/include/plat/iommu.h | 31 +++++++++++++++-- > arch/arm/plat-omap/include/plat/iovmm.h | 12 +++--- > drivers/iommu/omap-iommu.c | 58 ++++++++++++++----------------- > drivers/iommu/omap-iovmm.c | 31 +++++++++++------ > drivers/media/video/omap3isp/isp.c | 30 +++------------- > drivers/media/video/omap3isp/isp.h | 2 - > drivers/media/video/omap3isp/ispccdc.c | 18 +++++----- > drivers/media/video/omap3isp/ispstat.c | 8 ++-- > drivers/media/video/omap3isp/ispvideo.c | 4 +- > 10 files changed, 107 insertions(+), 94 deletions(-) Pulled into arm/omap, thanks Ohad. -- 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