Hi Hans, On Friday 01 May 2015 13:41:03 Hans Verkuil wrote: > On 05/01/2015 01:33 PM, Hans Verkuil wrote: > > From: Hans Verkuil <hans.verk...@cisco.com> > > > > 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?
The idea is to compile the subdev userspace API code out when not needed. Not all media controller drivers need that API. > > 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 -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
