On Mon, Mar 08, 2010 at 10:50:50PM +0530, Thara Gopinath wrote: > +static inline int sr_type(void) > +{ > + if (cpu_is_omap3630()) > + return SR_TYPE_V2; > + else if (cpu_is_omap343x()) > + return SR_TYPE_V1; > + else { > + pr_err("Trying to enable SR for Chip not support SR! \n"); > + return 0; > + } if one branch has curly braces, all of them should have. Check Documentation/CodingStyle. -- balbi -- 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