Greg KH <[email protected]> writes:

>> +    static const s32 legacy[] = { 1000, 2000, 5500, 11000,
>> +                                  6000, 9000, 12000, 18000, 24000,
>> +                                  36000, 48000, 54000 };
>> +    u8 i;
>> +
>> +    for (i = 0; i < 12; i++) {
>
> Please use ARRAY_SIZE for this, and your other loop variables to ensure
> that you got it right.

Good point, I missed that. I'll send v3.

Thanks.

-- 
Kalle Valo
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to