> -----Original Message-----
> From: [email protected] 
> [mailto:[email protected]] On Behalf Of Anna,
> Suman
> Sent: Friday, February 19, 2010 2:37 AM
> To: [email protected]
> Cc: [email protected]; 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 <[email protected]>
> 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 <[email protected]>
> ---
>  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

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to