On Thu July 19 2012 20:41:11 Ismael Luceno wrote: > On Thu, 19 Jul 2012 10:25:09 -0300 > Ezequiel Garcia <elezegarcia@xxxxxxxxx> wrote: > > On Wed, Jul 18, 2012 at 7:26 PM, Ismael Luceno > > <ismael.luceno@xxxxxxxxx> wrote: > > > On Thu, Jun 21, 2012 at 4:52 PM, Ezequiel Garcia > > > <elezegarcia@xxxxxxxxx> wrote: > > >> This patch moves video_nr module parameter to core.c > > >> and then passes that parameter as an argument to functions > > >> that need it. > > >> This way we avoid the extern declaration and parameter > > >> dependencies are better exposed. > > > <...> > > > > > > NACK. > > > > > > The changes to video_nr are supposed to be preserved. > > > > Mmm, I'm sorry but I don't see any functionality change in this patch, > > just a cleanup. > > > > What do you mean by "changes to video_nr are supposed to be > > preserved"? > > It is modified by solo_enc_alloc, which is called multiple times by > solo_enc_v4l2_init. You don't need to modify it at all. video_register_device() will start looking for a free video node number starting at video_nr and counting upwards, so increasing video_nr has no purpose. Leaving it out will give you exactly the same result. The only driver that does the same thing is vivi, and I think it should be removed from there as well. The solo driver probably copied it from vivi :-( Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html