Hilman, Kevin wrote on Tuesday, February 15, 2011 4:58 AM:

> Hemant Pedanekar <[email protected]> writes:
> 
> [...]
> 
>> @@ -402,7 +422,7 @@ void __init
> omap2_init_common_devices(struct omap_sdrc_params *sdrc_cs0,
>> 
>>      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--
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

Reply via email to