cvTony Lindgren wrote on Thursday, February 10, 2011 7:09 AM:

> Hi,
> 
> * Hemant Pedanekar <[email protected]> [110209 08:54]:
>> 
>>  1) Multi-OMAP build with CONFIG_SOC_OMAPTI816X: The kernel built only for
>>  TI816X OMAP3 SoCs along with any other OMAP2/4 (e.g., OMAP4s). May not
>>  boot on other OMAP3 SoCs.
> 
> This option does not sound right. For the multi-omap builds we need to have
> the system working for all the processors, there should not be any reason
> not to have things working.
> 
> Do you have some example in mind where other omaps break? If so, we should
> just fix that issue.
> 
> Regards,
> 
> Tony

I will take care of this using separate map_io as you mentioned on
the other mail on patch 2 comment.

Do you think the 2nd case restriction mentioned in this patch comment is OK?

" 2) OMAP3 only build with CONFIG_SOC_OMAPTI816X: This will build TI816X
 optimized kernel. May not boot on other OMAP3 SoCs."

As I had mentioned on follow up on Paul's comment on v4, some OMAP3 specific
code may get skipped on non-multi-omap builds with CONFIG_SOC_OMAPTI816X
(cpu_is_ti816x() is true) such as:

-       if (omap_rev() >= OMAP3430_REV_ES2_0)     
+       if (!cpu_is_ti816x() && (omap_rev() >= OMAP3430_REV_ES2_0)) 
                omap3_clk_lock_dpll5();    

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