Re: [PATCH v1 3/3] ARM:OMAP4 iommu:provide build support for omap4 iommu
On Wed, Aug 05, 2009 at 07:35:38AM +0300, Hiroshi DOYU wrote: From: ext Russell King - ARM Linux li...@arm.linux.org.uk Subject: Re: [PATCH v1 3/3] ARM:OMAP4 iommu:provide build support for omap4 iommu Date: Wed, 5 Aug 2009 00:42:41 +0200 On Tue, Aug 04, 2009 at 05:32:12PM -0500, Kanigeri, Hari wrote: diff --git a/arch/arm/plat-omap/Kconfig b/arch/arm/plat-omap/Kconfig index efe85d0..50aaefb 100644 --- a/arch/arm/plat-omap/Kconfig +++ b/arch/arm/plat-omap/Kconfig @@ -118,8 +118,11 @@ config OMAP_MBOX_FWK DSP, IVA1.0 and IVA2 in OMAP1/2/3. config OMAP_IOMMU - tristate - + tristate iommu Insufficiently verbose description. OMAP IOMMU Support would be better. I thought the idea here was to arrange for things to select OMAP_IOMMU when they require it rather than offering it as a separate configuration option. The former way has the advantage that you don't need to know that you need IOMMU support to (eg) use the Camera - enabling the Camera should automatically enable IOMMU support. The following is the original discussion. http://lists.arm.linux.org.uk/lurker/message/20090518.130233.3f238e72.en.html Quite. I'm repeating myself. So the question is why the original discussion didn't produce a change. -- 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 v1 3/3] ARM:OMAP4 iommu:provide build support for omap4 iommu
Hi Russell and Hiroshi, Thanks for your comments. Please see my below comments. -Original Message- From: Hiroshi DOYU [mailto:hiroshi.d...@nokia.com] Sent: Tuesday, August 04, 2009 11:36 PM To: Kanigeri, Hari Cc: linux-omap@vger.kernel.org; t...@atomide.com; li...@arm.linux.org.uk; Shilimkar, Santosh Subject: Re: [PATCH v1 3/3] ARM:OMAP4 iommu:provide build support for omap4 iommu From: ext Russell King - ARM Linux li...@arm.linux.org.uk Subject: Re: [PATCH v1 3/3] ARM:OMAP4 iommu:provide build support for omap4 iommu Date: Wed, 5 Aug 2009 00:42:41 +0200 On Tue, Aug 04, 2009 at 05:32:12PM -0500, Kanigeri, Hari wrote: diff --git a/arch/arm/plat-omap/Kconfig b/arch/arm/plat-omap/Kconfig index efe85d0..50aaefb 100644 --- a/arch/arm/plat-omap/Kconfig +++ b/arch/arm/plat-omap/Kconfig @@ -118,8 +118,11 @@ config OMAP_MBOX_FWK DSP, IVA1.0 and IVA2 in OMAP1/2/3. config OMAP_IOMMU - tristate - + tristate iommu Insufficiently verbose description. OMAP IOMMU Support would be better. I thought the idea here was to arrange for things to select OMAP_IOMMU when they require it rather than offering it as a separate configuration option. The former way has the advantage that you don't need to know that you need IOMMU support to (eg) use the Camera - enabling the Camera should automatically enable IOMMU support. The following is the original discussion. http://lists.arm.linux.org.uk/lurker/message/20090518.130233.3f238e72.en.h tml -- I agree with your comments from OMAP3 point of view. For OMAP4, I guess Camera is not going to use IOMMU module. IOMMU modules will be used only by the IPC that are communicating with 2 remote Cores (Ducati and Tesla). Having said this, there might not be any clients of IOMMU in the Kernel space as we are looking at the option of using the IOMMU as a character driver from User-space. For this reason, I think we should have the configuration option to build iommu. Please let me know your comments. Thank you, Best regards, Hari -- 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
[PATCH v1 3/3] ARM:OMAP4 iommu:provide build support for omap4 iommu
This patch provides build support to build OMAP4 IOMMU module. This also adds the changes to Kconfig file to make iommu option visible in menuconfig. Signed-off-by: Hari Kanigeri h-kanige...@ti.com --- arch/arm/mach-omap2/Makefile |2 +- arch/arm/plat-omap/Kconfig |7 +-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile index 735bae5..59e0865 100644 --- a/arch/arm/mach-omap2/Makefile +++ b/arch/arm/mach-omap2/Makefile @@ -41,7 +41,7 @@ obj-$(CONFIG_ARCH_OMAP3) += clock34xx.o iommu-y+= iommu2.o iommu-$(CONFIG_ARCH_OMAP3) += omap3-iommu.o - +iommu-$(CONFIG_ARCH_OMAP4) += omap4-iommu.o obj-$(CONFIG_OMAP_IOMMU) += $(iommu-y) # Specific board support diff --git a/arch/arm/plat-omap/Kconfig b/arch/arm/plat-omap/Kconfig index efe85d0..50aaefb 100644 --- a/arch/arm/plat-omap/Kconfig +++ b/arch/arm/plat-omap/Kconfig @@ -118,8 +118,11 @@ config OMAP_MBOX_FWK DSP, IVA1.0 and IVA2 in OMAP1/2/3. config OMAP_IOMMU - tristate - + tristate iommu + default y + help + Select this option if you want to use IOMMU support for the + Camera in OMAP3, and for Ducati processor in OMAP4. choice prompt System timer default OMAP_MPU_TIMER -- 1.5.4.3 Thank you, Best regards, Hari 0003-ARM-OMAP4-iommu-provide-build-support-for-omap4-iomm.patch Description: 0003-ARM-OMAP4-iommu-provide-build-support-for-omap4-iomm.patch
Re: [PATCH v1 3/3] ARM:OMAP4 iommu:provide build support for omap4 iommu
On Tue, Aug 04, 2009 at 05:32:12PM -0500, Kanigeri, Hari wrote: diff --git a/arch/arm/plat-omap/Kconfig b/arch/arm/plat-omap/Kconfig index efe85d0..50aaefb 100644 --- a/arch/arm/plat-omap/Kconfig +++ b/arch/arm/plat-omap/Kconfig @@ -118,8 +118,11 @@ config OMAP_MBOX_FWK DSP, IVA1.0 and IVA2 in OMAP1/2/3. config OMAP_IOMMU - tristate - + tristate iommu Insufficiently verbose description. OMAP IOMMU Support would be better. I thought the idea here was to arrange for things to select OMAP_IOMMU when they require it rather than offering it as a separate configuration option. The former way has the advantage that you don't need to know that you need IOMMU support to (eg) use the Camera - enabling the Camera should automatically enable IOMMU support. -- 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