Re: [PATCH v2] ath6kl: iw dev wlan0 link implementation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Greg KH <gregkh@xxxxxxx> 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
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux