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