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 . Thanks, Sricharan