[PATCH 2/3] ARM: OMAP: Move omap2_init_common_devices out of init_early

Tony Lindgren tony at atomide.com
Wed Aug 24 08:20:50 EDT 2011


* Cousson, Benoit <b-cousson at ti.com> [110824 04:38]:
> Hi Tony,
> 
> On 8/24/2011 1:54 PM, Tony Lindgren wrote:
> >There's no need to call omap2_init_common_devices from init_early.
> >It reprograms the memory timings for some boards, so it's better to
> >do it later so we have a chance to get console messages if something
> >goes wrong.
> 
> BTW, I did some comment on that function in my OMAP4 device-tree
> series, because the name is misleading.
> That function is just initializing the SDRC controller present in
> OMAP2 & 3. OMAP4 platform should not even call it.
> In fact, I'm now wondering were the OMAP4 EMIF is configured?

Probably nowhere currently?
 
> void __init omap2_init_common_devices(struct omap_sdrc_params *sdrc_cs0,
>                                       struct omap_sdrc_params *sdrc_cs1)
> {
>         if (cpu_is_omap24xx() || omap3_has_sdrc()) {
>                 omap2_sdrc_init(sdrc_cs0, sdrc_cs1);
>                 _omap2_init_reprogram_sdrc();
>         }
> }
> 
> I guess it used to do much more than that in the past.
> 
> Shouldn't we rename it to clarify its purpose?

Yeah we might as well rename it to omap_init_sdrc since we're
patching it. Will post an updated patch.

Tony



More information about the linux-arm-kernel mailing list