Tony, On Mon, May 7, 2012 at 11:04 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote: > * R, Sricharan <r.sricharan@xxxxxx> [120506 00:42]: >> Hi Tony, >> >> >> - if (!(cpu_is_omap44xx())) >> >> + if ((!(cpu_is_omap44xx())) && (!cpu_is_omap54xx())) >> >> return -ENODEV; >> >> >> >> for (i = 0; i < L3_MODULES; i++) { >> > >> > Isn't there some unnecessary parens here? >> >> You mean in this above for loop?. >> There are multiple statements . > > No, both the old (!(cpu_is_omap44xx())) and the added one. > oh ok, will remove those extra ones. Thanks. 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