From: "ext Guzman Lugo, Fernando" <x0095840@xxxxxx> Subject: RE: PATCH] OMAP3: add mailbox initialization for 3630 Date: Fri, 26 Mar 2010 00:44:38 +0100 > > >>-----Original Message----- >>From: Kanigeri, Hari >>Sent: Thursday, March 25, 2010 4:56 PM >>To: Guzman Lugo, Fernando; linux-omap@xxxxxxxxxxxxxxx >>Cc: Doyu Hiroshi (Nokia-D/Helsinki); Palande Ameya (Nokia-D/Helsinki); >>Felipe Contreras >>Subject: RE: PATCH] OMAP3: add mailbox initialization for 3630 >> >>Fernando, >> >>> + if (cpu_is_omap2420() || cpu_is_omap3430() || >>> + cpu_is_omap3630() || cpu_is_omap44xx()) >> >>looks like this check is applied to all OMAPS. Can we just remove this >>check ? > > There is more defines for more versions like: > > cpu_is_omap2422 > cpu_is_omap3503 > ... etc, > > But I suppose that the mailbox is valid for all of them 24XX, 34XX, 36XX, 44XX. But then why it was only checking for cpu_is_omap2420 or cpu_is_omap3430 or cpu_is_omap44xx? > > Maybe someone knows the reason. Otherwise I agree with you. When mailbox driver is introduced for the first time, "dspgateway" only supported omap2420 since that's the one to have C55x dsp. Now the type of dsp doesn't matter for mailbox anymore. I think that removing the this type check would make sense now. -- 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