On Tue, Oct 11, 2011 at 05:59:25PM -0300, Mauro Carvalho Chehab wrote:
> > - if (i > 2) {
> > + if (i >= 2) {
>
> It would be better to add a NUM_INPUT macro (or something like that, defined
> together
> with the INPUT macro) that would do an ARRAY_SIZE(cx25821_boards) and use it
> here,
> instead of a "2" magic number.
You're right. The hard coded 2 is not helpful... Why not just an:
if (i >= ARRAY_SIZE(cx25821_boards)) {...
I'll send a patch to do that.
regards,
dan carpenter
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
