* [PATCH] omap3: Remove non-existent config option
@ 2010-07-22 9:13 Sanjeev Premi
2010-08-03 8:09 ` Marathe, Yogesh
0 siblings, 1 reply; 8+ messages in thread
From: Sanjeev Premi @ 2010-07-22 9:13 UTC (permalink / raw)
To: linux-omap; +Cc: Yogesh Marathe, Sanjeev Premi
From: Yogesh Marathe <yogesh_marathe@ti.com>
The definition of "iva2" device in iommu_device
is wrapped inside CONFIG_MPU_BRIDGE_IOMMU, but
this option is not defined in KConfig.
This patch removes the wrapper and makes "iva2"
available as another iommu_device.
Signed-off-by: Yogesh Marathe <yogesh_marathe@ti.com>
Signed-off-by: Sanjeev Premi <premi@ti.com>
---
arch/arm/mach-omap2/omap-iommu.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/arch/arm/mach-omap2/omap-iommu.c b/arch/arm/mach-omap2/omap-iommu.c
index f5a1aad..bb8c01d 100644
--- a/arch/arm/mach-omap2/omap-iommu.c
+++ b/arch/arm/mach-omap2/omap-iommu.c
@@ -35,7 +35,6 @@ static struct iommu_device omap3_devices[] = {
.clk_name = "cam_ick",
},
},
-#if defined(CONFIG_MPU_BRIDGE_IOMMU)
{
.base = 0x5d000000,
.irq = 28,
@@ -45,7 +44,6 @@ static struct iommu_device omap3_devices[] = {
.clk_name = "iva2_ck",
},
},
-#endif
};
#define NR_OMAP3_IOMMU_DEVICES ARRAY_SIZE(omap3_devices)
static struct platform_device *omap3_iommu_pdev[NR_OMAP3_IOMMU_DEVICES];
--
1.6.6.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* RE: [PATCH] omap3: Remove non-existent config option
2010-07-22 9:13 [PATCH] omap3: Remove non-existent config option Sanjeev Premi
@ 2010-08-03 8:09 ` Marathe, Yogesh
2010-08-04 11:11 ` Tony Lindgren
0 siblings, 1 reply; 8+ messages in thread
From: Marathe, Yogesh @ 2010-08-03 8:09 UTC (permalink / raw)
To: linux-omap@vger.kernel.org; +Cc: Premi, Sanjeev
ping..
> -----Original Message-----
> From: Premi, Sanjeev
> Sent: Thursday, July 22, 2010 2:44 PM
> To: linux-omap@vger.kernel.org
> Cc: Marathe, Yogesh; Premi, Sanjeev
> Subject: [PATCH] omap3: Remove non-existent config option
>
> From: Yogesh Marathe <yogesh_marathe@ti.com>
>
> The definition of "iva2" device in iommu_device
> is wrapped inside CONFIG_MPU_BRIDGE_IOMMU, but
> this option is not defined in KConfig.
>
> This patch removes the wrapper and makes "iva2"
> available as another iommu_device.
>
> Signed-off-by: Yogesh Marathe <yogesh_marathe@ti.com>
> Signed-off-by: Sanjeev Premi <premi@ti.com>
> ---
> arch/arm/mach-omap2/omap-iommu.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/omap-iommu.c b/arch/arm/mach-omap2/omap-
> iommu.c
> index f5a1aad..bb8c01d 100644
> --- a/arch/arm/mach-omap2/omap-iommu.c
> +++ b/arch/arm/mach-omap2/omap-iommu.c
> @@ -35,7 +35,6 @@ static struct iommu_device omap3_devices[] = {
> .clk_name = "cam_ick",
> },
> },
> -#if defined(CONFIG_MPU_BRIDGE_IOMMU)
> {
> .base = 0x5d000000,
> .irq = 28,
> @@ -45,7 +44,6 @@ static struct iommu_device omap3_devices[] = {
> .clk_name = "iva2_ck",
> },
> },
> -#endif
> };
> #define NR_OMAP3_IOMMU_DEVICES ARRAY_SIZE(omap3_devices)
> static struct platform_device
> *omap3_iommu_pdev[NR_OMAP3_IOMMU_DEVICES];
> --
> 1.6.6.1
Regards,
Yogesh.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] omap3: Remove non-existent config option
2010-08-03 8:09 ` Marathe, Yogesh
@ 2010-08-04 11:11 ` Tony Lindgren
2010-08-04 11:20 ` Hiroshi DOYU
0 siblings, 1 reply; 8+ messages in thread
From: Tony Lindgren @ 2010-08-04 11:11 UTC (permalink / raw)
To: Marathe, Yogesh; +Cc: linux-omap@vger.kernel.org, Premi, Sanjeev, Hiroshi DOYU
* Marathe, Yogesh <yogesh_marathe@ti.com> [100803 11:03]:
> ping..
Hiroshi ack/nak?
Tony
> > -----Original Message-----
> > From: Premi, Sanjeev
> > Sent: Thursday, July 22, 2010 2:44 PM
> > To: linux-omap@vger.kernel.org
> > Cc: Marathe, Yogesh; Premi, Sanjeev
> > Subject: [PATCH] omap3: Remove non-existent config option
> >
> > From: Yogesh Marathe <yogesh_marathe@ti.com>
> >
> > The definition of "iva2" device in iommu_device
> > is wrapped inside CONFIG_MPU_BRIDGE_IOMMU, but
> > this option is not defined in KConfig.
> >
> > This patch removes the wrapper and makes "iva2"
> > available as another iommu_device.
> >
> > Signed-off-by: Yogesh Marathe <yogesh_marathe@ti.com>
> > Signed-off-by: Sanjeev Premi <premi@ti.com>
> > ---
> > arch/arm/mach-omap2/omap-iommu.c | 2 --
> > 1 files changed, 0 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/mach-omap2/omap-iommu.c b/arch/arm/mach-omap2/omap-
> > iommu.c
> > index f5a1aad..bb8c01d 100644
> > --- a/arch/arm/mach-omap2/omap-iommu.c
> > +++ b/arch/arm/mach-omap2/omap-iommu.c
> > @@ -35,7 +35,6 @@ static struct iommu_device omap3_devices[] = {
> > .clk_name = "cam_ick",
> > },
> > },
> > -#if defined(CONFIG_MPU_BRIDGE_IOMMU)
> > {
> > .base = 0x5d000000,
> > .irq = 28,
> > @@ -45,7 +44,6 @@ static struct iommu_device omap3_devices[] = {
> > .clk_name = "iva2_ck",
> > },
> > },
> > -#endif
> > };
> > #define NR_OMAP3_IOMMU_DEVICES ARRAY_SIZE(omap3_devices)
> > static struct platform_device
> > *omap3_iommu_pdev[NR_OMAP3_IOMMU_DEVICES];
> > --
> > 1.6.6.1
>
> Regards,
> Yogesh.
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] omap3: Remove non-existent config option
2010-08-04 11:11 ` Tony Lindgren
@ 2010-08-04 11:20 ` Hiroshi DOYU
2010-08-05 15:12 ` Kanigeri, Hari
0 siblings, 1 reply; 8+ messages in thread
From: Hiroshi DOYU @ 2010-08-04 11:20 UTC (permalink / raw)
To: tony; +Cc: yogesh_marathe, linux-omap, premi
From: ext Tony Lindgren <tony@atomide.com>
Subject: Re: [PATCH] omap3: Remove non-existent config option
Date: Wed, 4 Aug 2010 13:11:47 +0200
> * Marathe, Yogesh <yogesh_marathe@ti.com> [100803 11:03]:
>> ping..
>
> Hiroshi ack/nak?
Nak.
http://www.spinics.net/lists/linux-omap/msg32869.html
"tidspbridge" is in staging now.
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH] omap3: Remove non-existent config option
2010-08-04 11:20 ` Hiroshi DOYU
@ 2010-08-05 15:12 ` Kanigeri, Hari
2010-08-06 7:32 ` Hiroshi DOYU
0 siblings, 1 reply; 8+ messages in thread
From: Kanigeri, Hari @ 2010-08-05 15:12 UTC (permalink / raw)
To: Hiroshi DOYU, tony@atomide.com
Cc: Marathe, Yogesh, linux-omap@vger.kernel.org, Premi, Sanjeev
Hiroshi,
> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org
> [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Hiroshi DOYU
> Sent: Wednesday, August 04, 2010 6:20 AM
> To: tony@atomide.com
> Cc: Marathe, Yogesh; linux-omap@vger.kernel.org; Premi, Sanjeev
> Subject: Re: [PATCH] omap3: Remove non-existent config option
>
> From: ext Tony Lindgren <tony@atomide.com>
> Subject: Re: [PATCH] omap3: Remove non-existent config option
> Date: Wed, 4 Aug 2010 13:11:47 +0200
>
> > * Marathe, Yogesh <yogesh_marathe@ti.com> [100803 11:03]:
> >> ping..
> >
> > Hiroshi ack/nak?
>
> Nak.
>
> http://www.spinics.net/lists/linux-omap/msg32869.html
>
> "tidspbridge" is in staging now.
Can you please elaborate what this means ?
Yogesh patch enables the IOMMU for BRIDGE by default and we need this as IOMMU is going to get use in 3430.
> --
> To unsubscribe from this list: send the line "unsubscribe
> linux-omap" in the body of a message to
> majordomo@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] omap3: Remove non-existent config option
2010-08-05 15:12 ` Kanigeri, Hari
@ 2010-08-06 7:32 ` Hiroshi DOYU
2010-08-10 12:19 ` Marathe, Yogesh
0 siblings, 1 reply; 8+ messages in thread
From: Hiroshi DOYU @ 2010-08-06 7:32 UTC (permalink / raw)
To: tony@atomide.com, ext Kanigeri, Hari
Cc: Marathe, Yogesh, linux-omap@vger.kernel.org, Premi, Sanjeev
Hi Hari,
On Thu, 5 Aug 2010 17:12:11 +0200
"ext Kanigeri, Hari" <h-kanigeri2@ti.com> wrote:
> Hiroshi,
>
>
> > -----Original Message-----
> > From: linux-omap-owner@vger.kernel.org
> > [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Hiroshi DOYU
> > Sent: Wednesday, August 04, 2010 6:20 AM
> > To: tony@atomide.com
> > Cc: Marathe, Yogesh; linux-omap@vger.kernel.org; Premi, Sanjeev
> > Subject: Re: [PATCH] omap3: Remove non-existent config option
> >
> > From: ext Tony Lindgren <tony@atomide.com>
> > Subject: Re: [PATCH] omap3: Remove non-existent config option
> > Date: Wed, 4 Aug 2010 13:11:47 +0200
> >
> > > * Marathe, Yogesh <yogesh_marathe@ti.com> [100803 11:03]:
> > >> ping..
> > >
> > > Hiroshi ack/nak?
> >
> > Nak.
> >
> > http://www.spinics.net/lists/linux-omap/msg32869.html
> >
> > "tidspbridge" is in staging now.
>
> Can you please elaborate what this means ?
> Yogesh patch enables the IOMMU for BRIDGE by default and we need this as IOMMU is going to get use in 3430.
Ok, I misunderstood this intention, sorry.
Tony,
please put this into your queue for next merge.
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH] omap3: Remove non-existent config option
2010-08-06 7:32 ` Hiroshi DOYU
@ 2010-08-10 12:19 ` Marathe, Yogesh
2010-08-10 12:39 ` Tony Lindgren
0 siblings, 1 reply; 8+ messages in thread
From: Marathe, Yogesh @ 2010-08-10 12:19 UTC (permalink / raw)
To: Hiroshi DOYU, tony@atomide.com, Kanigeri, Hari
Cc: linux-omap@vger.kernel.org, Premi, Sanjeev
Tony,
> -----Original Message-----
> From: Hiroshi DOYU [mailto:Hiroshi.DOYU@nokia.com]
> Sent: Friday, August 06, 2010 1:03 PM
> To: tony@atomide.com; Kanigeri, Hari
> Cc: Marathe, Yogesh; linux-omap@vger.kernel.org; Premi, Sanjeev
> Subject: Re: [PATCH] omap3: Remove non-existent config option
>
> Hi Hari,
>
> On Thu, 5 Aug 2010 17:12:11 +0200
> "ext Kanigeri, Hari" <h-kanigeri2@ti.com> wrote:
>
> > Hiroshi,
> >
> >
> > > -----Original Message-----
> > > From: linux-omap-owner@vger.kernel.org
> > > [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Hiroshi DOYU
> > > Sent: Wednesday, August 04, 2010 6:20 AM
> > > To: tony@atomide.com
> > > Cc: Marathe, Yogesh; linux-omap@vger.kernel.org; Premi, Sanjeev
> > > Subject: Re: [PATCH] omap3: Remove non-existent config option
> > >
> > > From: ext Tony Lindgren <tony@atomide.com>
> > > Subject: Re: [PATCH] omap3: Remove non-existent config option
> > > Date: Wed, 4 Aug 2010 13:11:47 +0200
> > >
> > > > * Marathe, Yogesh <yogesh_marathe@ti.com> [100803 11:03]:
> > > >> ping..
> > > >
> > > > Hiroshi ack/nak?
> > >
> > > Nak.
> > >
> > > http://www.spinics.net/lists/linux-omap/msg32869.html
> > >
> > > "tidspbridge" is in staging now.
> >
> > Can you please elaborate what this means ?
> > Yogesh patch enables the IOMMU for BRIDGE by default and we need this as
> IOMMU is going to get use in 3430.
>
> Ok, I misunderstood this intention, sorry.
>
> Tony,
> please put this into your queue for next merge.
Is this patch taken?
Regards,
Yogesh.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2010-08-10 12:39 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-22 9:13 [PATCH] omap3: Remove non-existent config option Sanjeev Premi
2010-08-03 8:09 ` Marathe, Yogesh
2010-08-04 11:11 ` Tony Lindgren
2010-08-04 11:20 ` Hiroshi DOYU
2010-08-05 15:12 ` Kanigeri, Hari
2010-08-06 7:32 ` Hiroshi DOYU
2010-08-10 12:19 ` Marathe, Yogesh
2010-08-10 12:39 ` Tony Lindgren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).