>-----Original Message----- >From: Hiroshi DOYU [mailto:[email protected]] >Sent: Thursday, March 25, 2010 11:26 PM >To: Guzman Lugo, Fernando >Cc: Kanigeri, Hari; [email protected]; [email protected]; >[email protected] >Subject: Re: PATCH] OMAP3: add mailbox initialization for 3630 > >From: "ext Guzman Lugo, Fernando" <[email protected]> >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; [email protected] >>>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.
Ok, I will send the patch again with the change. Regards, Fernando. -- 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
