Hi Roger, >> +void __init omap5xxx_check_revision(void) >> +{ >> + u32 idcode; >> + u16 hawkeye; >> + u8 rev; >> + >> + idcode = read_tap_reg(OMAP_TAP_IDCODE); >> + hawkeye = (idcode >> 12) & 0xffff; >> + rev = (idcode >> 28) & 0xff; >> + switch (hawkeye) { >> + case 0xb942: >> + switch (rev) { >> + case 0: >> + omap_revision = OMAP5430_REV_ES1_0; >> + break; >> + case 1: >> + omap_revision = OMAP5430_REV_ES2_0; >> + break; >> + default: >> + omap_revision = OMAP5430_REV_ES1_0; > > Default should always be the latest version supported. i.e. in this case > it should be OMAP5430_REV_ES2_0 > ok thanks, will correct it. Thanks, Sricharan -- 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