Hilman, Kevin wrote on Tuesday, February 15, 2011 4:58 AM:
Hemant Pedanekar [off-list ref] writes:
[...]
quoted
@@ -402,7 +422,7 @@ void __init
omap2_init_common_devices(struct omap_sdrc_params *sdrc_cs0,
quoted
omap_hwmod_late_init();
- if (cpu_is_omap24xx() || cpu_is_omap34xx()) {
+ if (omap3_has_sdrc()) {
omap2_sdrc_init(sdrc_cs0, sdrc_cs1);
_omap2_init_reprogram_sdrc();
}
At first glance, this one doesn't look quite right. Will
omap3_has_sdrc() be true on 24xx? Please confirm.
Kevin
You are right, this is incorrect as there is no 'features' set for
omap24xx. I will send new version with cpu_is_omap24xx() check retained.
Thanks.
Hemant