On Thu, 19 Jul 2012 21:48:33 +0200 Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > 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. Oh, didn't know that. Interesting. -- 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
