On Wed, Jul 28, 2010 at 5:40 PM, Ernesto Ramos <ernesto@xxxxxx> wrote: > Replace simple_strtoul by strict_strtoul in atoi function. In general the question what is the purpose here to do the change? > @@ -1023,7 +1025,9 @@ static s32 atoi(char *psz_buf) > base = 16; > } > > - return simple_strtoul(pch, NULL, base); > + ret_val = strict_strtoul(pch, base, &res); > + > + return ret_val ? : res; May be better to use explicit values in condition? -- With Best Regards, Andy Shevchenko -- 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