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