Hi, On Fri, May 14, 2010 at 1:17 PM, Sergei Shtylyov <sshtylyov@xxxxxxxxxx> wrote: > Felipe Contreras wrote: > >> Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx> >> --- >> arch/arm/mach-omap2/mailbox.c | 3 ++- >> 1 files changed, 2 insertions(+), 1 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c >> index 252e53b..6d17da8 100644 >> --- a/arch/arm/mach-omap2/mailbox.c >> +++ b/arch/arm/mach-omap2/mailbox.c >> @@ -431,7 +431,8 @@ err_dsp: >> static int __devexit omap2_mbox_remove(struct platform_device *pdev) >> { >> #if defined(CONFIG_ARCH_OMAP2420) >> - omap_mbox_unregister(&mbox_iva_info); >> + if (cpu_is_omap2420()) >> + omap_mbox_unregister(&mbox_iva_info); >> #endif > > Shouldn't #ifdef be remove then? My first patch did that, but now mbox_iva_info is only available when ARCH_OMAP2420=y to save space when building for a specific OMAP. -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html