cvTony Lindgren wrote on Thursday, February 10, 2011 7:09 AM: > Hi, > > * Hemant Pedanekar <hemantp@xxxxxx> [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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html