* [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
@ 2010-02-18 21:07 Anna, Suman
2010-02-20 18:46 ` Shilimkar, Santosh
0 siblings, 1 reply; 5+ messages in thread
From: Anna, Suman @ 2010-02-18 21:07 UTC (permalink / raw)
To: linux-omap@vger.kernel.org
Cc: Hiroshi.DOYU@nokia.com, Clark, Rob, Kanigeri, Hari,
C.A, Subramaniam
>From 78d00b562548e1d95ad12a4d8121be67950daf68 Mon Sep 17 00:00:00 2001
From: Suman Anna <s-anna@ti.com>
Date: Mon, 25 Jan 2010 18:27:21 -0600
Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
Remove a compiler warning in device-specific
mailbox module.
Signed-off-by: Suman Anna <s-anna@ti.com>
---
arch/arm/mach-omap2/mailbox.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c
index 2c9fd1c..c970cf6 100644
--- a/arch/arm/mach-omap2/mailbox.c
+++ b/arch/arm/mach-omap2/mailbox.c
@@ -419,8 +419,10 @@ static int __devinit omap2_mbox_probe(struct platform_device *pdev)
#endif
return 0;
+#if defined(CONFIG_ARCH_OMAP2420) /* IVA */
err_iva1:
omap_mbox_unregister(&mbox_dsp_info);
+#endif
err_dsp:
iounmap(mbox_base);
--
1.6.6.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
2010-02-18 21:07 [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning Anna, Suman
@ 2010-02-20 18:46 ` Shilimkar, Santosh
[not found] ` <2C1563023E232C49B9F505633D57C2DA2640E025F9@dlee04.ent.ti.com>
0 siblings, 1 reply; 5+ messages in thread
From: Shilimkar, Santosh @ 2010-02-20 18:46 UTC (permalink / raw)
To: Anna, Suman, linux-omap@vger.kernel.org
Cc: Hiroshi.DOYU@nokia.com, Clark, Rob, Kanigeri, Hari,
C.A, Subramaniam
> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Anna,
> Suman
> Sent: Friday, February 19, 2010 2:37 AM
> To: linux-omap@vger.kernel.org
> Cc: Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A, Subramaniam
> Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
>
> From 78d00b562548e1d95ad12a4d8121be67950daf68 Mon Sep 17 00:00:00 2001
> From: Suman Anna <s-anna@ti.com>
> Date: Mon, 25 Jan 2010 18:27:21 -0600
> Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
>
> Remove a compiler warning in device-specific
> mailbox module.
>
> Signed-off-by: Suman Anna <s-anna@ti.com>
> ---
> arch/arm/mach-omap2/mailbox.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c
> index 2c9fd1c..c970cf6 100644
> --- a/arch/arm/mach-omap2/mailbox.c
> +++ b/arch/arm/mach-omap2/mailbox.c
> @@ -419,8 +419,10 @@ static int __devinit omap2_mbox_probe(struct platform_device *pdev)
> #endif
> return 0;
>
> +#if defined(CONFIG_ARCH_OMAP2420) /* IVA */
> err_iva1:
> omap_mbox_unregister(&mbox_dsp_info);
> +#endif
Can't we avoid the #ifdef here? Is this "omap_mbox_unregister" omap2 specific?
Regards,
Santosh
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
[not found] ` <2C1563023E232C49B9F505633D57C2DA2640E025F9@dlee04.ent.ti.com>
@ 2010-02-22 21:37 ` Tony Lindgren
2010-02-22 21:44 ` Anna, Suman
0 siblings, 1 reply; 5+ messages in thread
From: Tony Lindgren @ 2010-02-22 21:37 UTC (permalink / raw)
To: Anna, Suman
Cc: Shilimkar, Santosh, linux-omap@vger.kernel.org,
Hiroshi.DOYU@nokia.com, Clark, Rob, Kanigeri, Hari,
C.A, Subramaniam
* Anna, Suman <s-anna@ti.com> [100222 11:53]:
> Hi Santosh,
>
> > -----Original Message-----
> > From: Shilimkar, Santosh
> > Sent: Saturday, February 20, 2010 12:47 PM
> > To: Anna, Suman; linux-omap@vger.kernel.org
> > Cc: Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A, Subramaniam
> > Subject: RE: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> >
> > > -----Original Message-----
> > > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> > owner@vger.kernel.org] On Behalf Of Anna,
> > > Suman
> > > Sent: Friday, February 19, 2010 2:37 AM
> > > To: linux-omap@vger.kernel.org
> > > Cc: Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A, Subramaniam
> > > Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > >
> > > From 78d00b562548e1d95ad12a4d8121be67950daf68 Mon Sep 17 00:00:00 2001
> > > From: Suman Anna <s-anna@ti.com>
> > > Date: Mon, 25 Jan 2010 18:27:21 -0600
> > > Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > >
> > > Remove a compiler warning in device-specific
> > > mailbox module.
> > >
> > > Signed-off-by: Suman Anna <s-anna@ti.com>
> > > ---
> > > arch/arm/mach-omap2/mailbox.c | 2 ++
> > > 1 files changed, 2 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-
> > omap2/mailbox.c
> > > index 2c9fd1c..c970cf6 100644
> > > --- a/arch/arm/mach-omap2/mailbox.c
> > > +++ b/arch/arm/mach-omap2/mailbox.c
> > > @@ -419,8 +419,10 @@ static int __devinit omap2_mbox_probe(struct
> > platform_device *pdev)
> > > #endif
> > > return 0;
> > >
> > > +#if defined(CONFIG_ARCH_OMAP2420) /* IVA */
> > > err_iva1:
> > > omap_mbox_unregister(&mbox_dsp_info);
> > > +#endif
> > Can't we avoid the #ifdef here? Is this "omap_mbox_unregister" omap2
> > specific?
>
> Yes, this #ifdef is specific to 2420. There are other places in the mailbox code which also utilize a similar #ifdef. I have merely added it to remove the compile warning. I think it needs a separate patch to remove all the 2420 specific #ifdefs.
Let's drop this patch for now. We should fix this with something like
this in mailbox.h:
#ifdef CONFIG_ARCH_OMAP2420
int omap_mbox_unregister(struct omap_mbox *);
#else
static inline int omap_mbox_unregister(struct omap_mbox *mbox)
{
return 0;
}
#endif
That way we can get rid of the ifdefs in other places too.
Tony
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
2010-02-22 21:37 ` Tony Lindgren
@ 2010-02-22 21:44 ` Anna, Suman
2010-02-22 23:25 ` Tony Lindgren
0 siblings, 1 reply; 5+ messages in thread
From: Anna, Suman @ 2010-02-22 21:44 UTC (permalink / raw)
To: Tony Lindgren
Cc: Shilimkar, Santosh, linux-omap@vger.kernel.org,
Hiroshi.DOYU@nokia.com, Clark, Rob, Kanigeri, Hari,
C.A, Subramaniam
Tony,
> -----Original Message-----
> From: Tony Lindgren [mailto:tony@atomide.com]
> Sent: Monday, February 22, 2010 3:37 PM
> To: Anna, Suman
> Cc: Shilimkar, Santosh; linux-omap@vger.kernel.org;
> Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A, Subramaniam
> Subject: Re: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
>
> * Anna, Suman <s-anna@ti.com> [100222 11:53]:
> > Hi Santosh,
> >
> > > -----Original Message-----
> > > From: Shilimkar, Santosh
> > > Sent: Saturday, February 20, 2010 12:47 PM
> > > To: Anna, Suman; linux-omap@vger.kernel.org
> > > Cc: Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A,
> Subramaniam
> > > Subject: RE: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > >
> > > > -----Original Message-----
> > > > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> > > owner@vger.kernel.org] On Behalf Of Anna,
> > > > Suman
> > > > Sent: Friday, February 19, 2010 2:37 AM
> > > > To: linux-omap@vger.kernel.org
> > > > Cc: Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A,
> Subramaniam
> > > > Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > > >
> > > > From 78d00b562548e1d95ad12a4d8121be67950daf68 Mon Sep 17 00:00:00
> 2001
> > > > From: Suman Anna <s-anna@ti.com>
> > > > Date: Mon, 25 Jan 2010 18:27:21 -0600
> > > > Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > > >
> > > > Remove a compiler warning in device-specific
> > > > mailbox module.
> > > >
> > > > Signed-off-by: Suman Anna <s-anna@ti.com>
> > > > ---
> > > > arch/arm/mach-omap2/mailbox.c | 2 ++
> > > > 1 files changed, 2 insertions(+), 0 deletions(-)
> > > >
> > > > diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-
> > > omap2/mailbox.c
> > > > index 2c9fd1c..c970cf6 100644
> > > > --- a/arch/arm/mach-omap2/mailbox.c
> > > > +++ b/arch/arm/mach-omap2/mailbox.c
> > > > @@ -419,8 +419,10 @@ static int __devinit omap2_mbox_probe(struct
> > > platform_device *pdev)
> > > > #endif
> > > > return 0;
> > > >
> > > > +#if defined(CONFIG_ARCH_OMAP2420) /* IVA */
> > > > err_iva1:
> > > > omap_mbox_unregister(&mbox_dsp_info);
> > > > +#endif
> > > Can't we avoid the #ifdef here? Is this "omap_mbox_unregister" omap2
> > > specific?
> >
> > Yes, this #ifdef is specific to 2420. There are other places in the
> mailbox code which also utilize a similar #ifdef. I have merely added it
> to remove the compile warning. I think it needs a separate patch to remove
> all the 2420 specific #ifdefs.
>
> Let's drop this patch for now. We should fix this with something like
> this in mailbox.h:
>
> #ifdef CONFIG_ARCH_OMAP2420
> int omap_mbox_unregister(struct omap_mbox *);
> #else
> static inline int omap_mbox_unregister(struct omap_mbox *mbox)
> {
> return 0;
> }
> #endif
>
> That way we can get rid of the ifdefs in other places too.
>
> Tony
Sorry, my statement is a little bit misleading. The omap_mbox_unregister function itself is not specific to 2420, but rather the call is. It should be called only for 2420, and hence the #ifdef.
Regards
Suman
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
2010-02-22 21:44 ` Anna, Suman
@ 2010-02-22 23:25 ` Tony Lindgren
0 siblings, 0 replies; 5+ messages in thread
From: Tony Lindgren @ 2010-02-22 23:25 UTC (permalink / raw)
To: Anna, Suman
Cc: Shilimkar, Santosh, linux-omap@vger.kernel.org,
Hiroshi.DOYU@nokia.com, Clark, Rob, Kanigeri, Hari,
C.A, Subramaniam
* Anna, Suman <s-anna@ti.com> [100222 13:41]:
> Tony,
>
>
> > -----Original Message-----
> > From: Tony Lindgren [mailto:tony@atomide.com]
> > Sent: Monday, February 22, 2010 3:37 PM
> > To: Anna, Suman
> > Cc: Shilimkar, Santosh; linux-omap@vger.kernel.org;
> > Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A, Subramaniam
> > Subject: Re: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> >
> > * Anna, Suman <s-anna@ti.com> [100222 11:53]:
> > > Hi Santosh,
> > >
> > > > -----Original Message-----
> > > > From: Shilimkar, Santosh
> > > > Sent: Saturday, February 20, 2010 12:47 PM
> > > > To: Anna, Suman; linux-omap@vger.kernel.org
> > > > Cc: Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A,
> > Subramaniam
> > > > Subject: RE: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > > >
> > > > > -----Original Message-----
> > > > > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> > > > owner@vger.kernel.org] On Behalf Of Anna,
> > > > > Suman
> > > > > Sent: Friday, February 19, 2010 2:37 AM
> > > > > To: linux-omap@vger.kernel.org
> > > > > Cc: Hiroshi.DOYU@nokia.com; Clark, Rob; Kanigeri, Hari; C.A,
> > Subramaniam
> > > > > Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > > > >
> > > > > From 78d00b562548e1d95ad12a4d8121be67950daf68 Mon Sep 17 00:00:00
> > 2001
> > > > > From: Suman Anna <s-anna@ti.com>
> > > > > Date: Mon, 25 Jan 2010 18:27:21 -0600
> > > > > Subject: [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning
> > > > >
> > > > > Remove a compiler warning in device-specific
> > > > > mailbox module.
> > > > >
> > > > > Signed-off-by: Suman Anna <s-anna@ti.com>
> > > > > ---
> > > > > arch/arm/mach-omap2/mailbox.c | 2 ++
> > > > > 1 files changed, 2 insertions(+), 0 deletions(-)
> > > > >
> > > > > diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-
> > > > omap2/mailbox.c
> > > > > index 2c9fd1c..c970cf6 100644
> > > > > --- a/arch/arm/mach-omap2/mailbox.c
> > > > > +++ b/arch/arm/mach-omap2/mailbox.c
> > > > > @@ -419,8 +419,10 @@ static int __devinit omap2_mbox_probe(struct
> > > > platform_device *pdev)
> > > > > #endif
> > > > > return 0;
> > > > >
> > > > > +#if defined(CONFIG_ARCH_OMAP2420) /* IVA */
> > > > > err_iva1:
> > > > > omap_mbox_unregister(&mbox_dsp_info);
> > > > > +#endif
> > > > Can't we avoid the #ifdef here? Is this "omap_mbox_unregister" omap2
> > > > specific?
> > >
> > > Yes, this #ifdef is specific to 2420. There are other places in the
> > mailbox code which also utilize a similar #ifdef. I have merely added it
> > to remove the compile warning. I think it needs a separate patch to remove
> > all the 2420 specific #ifdefs.
> >
>
>
> > Let's drop this patch for now. We should fix this with something like
> > this in mailbox.h:
> >
> > #ifdef CONFIG_ARCH_OMAP2420
> > int omap_mbox_unregister(struct omap_mbox *);
> > #else
> > static inline int omap_mbox_unregister(struct omap_mbox *mbox)
> > {
> > return 0;
> > }
> > #endif
> >
> > That way we can get rid of the ifdefs in other places too.
> >
> > Tony
>
> Sorry, my statement is a little bit misleading. The omap_mbox_unregister function itself is not specific to 2420, but rather the call is. It should be called only for 2420, and hence the #ifdef.
OK, maybe you can update to just test for if (cpu_is_omap2420())
there?
Tony
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-02-22 23:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-02-18 21:07 [PATCH 1/5] omap2/3/4: mailbox: remove compiler warning Anna, Suman
2010-02-20 18:46 ` Shilimkar, Santosh
[not found] ` <2C1563023E232C49B9F505633D57C2DA2640E025F9@dlee04.ent.ti.com>
2010-02-22 21:37 ` Tony Lindgren
2010-02-22 21:44 ` Anna, Suman
2010-02-22 23:25 ` Tony Lindgren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox