* Shilimkar, Santosh <[email protected]> [090516 13:04]: > Thanks Russell for scanning all the patches minutely !! > > > -----Original Message----- > > From: Russell King - ARM Linux [mailto:[email protected]] > > Sent: Saturday, May 16, 2009 3:24 PM > > To: Shilimkar, Santosh > > Cc: [email protected]; > > [email protected] > > Subject: Re: [RESUBMIT][PATCH 4/7] OMAP4: Update common omap > > platform common sources. > > > > On Thu, May 07, 2009 at 11:59:13AM +0530, Santosh Shilimkar wrote: > > > @@ -309,3 +313,26 @@ void __init omap2_set_globals_343x(void) > > > } > > > #endif > > > > > > +#if defined(CONFIG_ARCH_OMAP4) > > > +static struct omap_globals *omap4_globals; > > > + > > > +static void __init __omap4_set_globals(void) > > > +{ > > > + omap2_set_globals_tap(omap4_globals); > > > + omap2_set_globals_control(omap4_globals); > > > +} > > > +static struct omap_globals omap443x_globals = { > > > + .class = OMAP443X_CLASS, > > > + .tap = OMAP2_IO_ADDRESS(0x4830A000), > > > + .ctrl = OMAP2_IO_ADDRESS(OMAP443X_CTRL_BASE), > > > + .prm = OMAP2_IO_ADDRESS(OMAP4430_PRM_BASE), > > > + .cm = OMAP2_IO_ADDRESS(OMAP4430_CM_BASE), > > > +}; > > > + > > > +void __init omap2_set_globals_443x(void) > > > +{ > > > + omap4_globals = &omap443x_globals; > > > + __omap4_set_globals(); > > > > Hmm, confused. omap4_globals is a static variable, and > > __omap4_set_globals > > is a static function. The only user of omap4_globals is > > __omap4_set_globals. > > It looks to me like the only purpose of omap4_globals is to pass a > > structure to __omap4_set_globals. Why not use a function > > argument instead? > Indeed. Actually I just more or less followed what is done for OMAP2/OMAP3 > here. I will clean this for OMAP4. > > Tony, > We may want to clean up this for OMAP2/OMAP3 as well. Can we add this to the > clean up patches planned if any. I can create the patch.
Sure, please post that patch for review. Regards, Tony -- 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
