On 05/01/2015 01:33 PM, Hans Verkuil wrote: > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > This patch series adds the VIDIOC_SUBDEV_QUERYCAP ioctl for v4l-subdev devices > as discussed during the ELC in San Jose and as discussed here: > > http://www.spinics.net/lists/linux-media/msg88009.html > > It also adds the entity_id to v4l2_capability. Question: why do we have CONFIG_VIDEO_V4L2_SUBDEV_API? I don't really see the point of this and I would propose to remove this config option and instead use CONFIG_MEDIA_CONTROLLER. I don't see the use-case of having MEDIA_CONTROLLER defined but not VIDEO_V4L2_SUBDEV_API. Comments? Hans > > > Hans Verkuil (3): > v4l2-subdev: add VIDIOC_SUBDEV_QUERYCAP ioctl > DocBook/media: document VIDIOC_SUBDEV_QUERYCAP > videodev2.h: add entity_id to struct v4l2_capability > > Documentation/DocBook/media/v4l/v4l2.xml | 1 + > .../DocBook/media/v4l/vidioc-querycap.xml | 18 ++- > .../DocBook/media/v4l/vidioc-subdev-querycap.xml | 140 +++++++++++++++++++++ > drivers/media/v4l2-core/v4l2-ioctl.c | 7 ++ > drivers/media/v4l2-core/v4l2-subdev.c | 19 +++ > include/uapi/linux/v4l2-subdev.h | 12 ++ > include/uapi/linux/videodev2.h | 5 +- > 7 files changed, 199 insertions(+), 3 deletions(-) > create mode 100644 Documentation/DocBook/media/v4l/vidioc-subdev-querycap.xml > -- 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
