> Basically the enum is not required. > Everything works fine without replacing VIDEO_MODE_xxx by > AV7110_VIDEO_MODE_xxx. (VIDEO_MODE_xxx is defined in videodev.h.) > > On the other hand, I like the enum because it defines the interface > between firmware and driver in a clean way. video_tuner->mode defines > should not be used here because anything except VIDEO_MODE_PAL or > VIDEO_MODE_NTSC are not valid for the firmware. In the future the enum > might be extended to display NTSC content on a PAL TV... For me, having the enum is overkill, but this doesn't hurt. So, let's then keep the enum if you wish. > > Also, please review the patch with scripts/checkpatch.pl. > > Will do so. Ok, thanks. -- Cheers, Mauro _______________________________________________ linux-dvb mailing list linux-dvb@xxxxxxxxxxx http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb