On Friday 07 August 2009 16:13:46 Mauro Carvalho Chehab wrote: > Em Thu, 6 Aug 2009 13:22:18 +0200 > "Hans Verkuil" <hverkuil@xxxxxxxxx> escreveu: > > > > > > Em Sun, 2 Aug 2009 12:10:04 +0200 > > > Hans Verkuil <hverkuil@xxxxxxxxx> escreveu: > > > > > >> Hi Mauro, > > >> > > >> Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb for the > > >> following: > > >> > > >> - v4l: introduce string control support. <snip> > > > Why do you need two routines that will always return zero? Why to create a > > > code > > > that will never be used? v4l2-compat-ioctl32.c is already complex enough > > > without adding any bogus code. > > > > When the RDS encoder driver from Eduardo is added, then he will add the > > string controls to ctrl_is_pointer() since his driver is the first to > > actually implement string controls. > > It is better to add this together with his patch, since, currently, the code > makes no sense to anyone that are analyzing it. Btw, as this kind of controls > will be used firstly on his driver, please add his patches on your tree. This > will make easier to analyze the required core changes. Eduardo, can you make a final patch series against my tree? I'll merge in your patches and make a new pull request containing everything. Regards, Hans -- Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom -- 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
