Hi Hans, On Thu, Apr 19, 2012 at 05:48:51PM +0200, Hans Verkuil wrote: > While I was cleaning up some older drivers I came across a few bugs that are > fixed here. The fixes are all trivial one-liners. > > Regards, > > Hans > > The following changes since commit f4d4e7656b26a6013bc5072c946920d2e2c44e8e: > > [media] em28xx: Make em28xx-input.c a separate module (2012-04-10 20:45:41 -0300) > > are available in the git repository at: > > git://linuxtv.org/hverkuil/media_tree.git fixes > > for you to fetch changes up to f85e735051e71410bfd695536a25c1013bceeabc: > > vivi: fix duplicate line. (2012-04-19 17:38:52 +0200) > > ---------------------------------------------------------------- > Hans Verkuil (4): > V4L: fix incorrect refcounting. > V4L2: drivers implementing vidioc_default should also return -ENOTTY > v4l2-ctrls.c: zero min/max/step/def values for 64 bit integers. Thanks for the patches. I'd put the setting of the min/max/step values for 64-bit controls to v4l2_ctrl_new() instead for two reasons: - All 64-bit controls require this for the time being, independent of the purpose, at least until min/max/step are implemented for them and - that's where part of the string control initialisation is done as well. What do you think? I think that in the long run we'll need to add capability to get min/max/step for 64-bit controls also to user space, but the information could be added to the controls in the kernel already earlier. Kind regards, -- Sakari Ailus e-mail: sakari.ailus@xxxxxx jabber/XMPP/Gmail: sailus@xxxxxxxxxxxxxx -- 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
