On 05/04/2015 12:33 AM, Laurent Pinchart wrote: > 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.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? > > The idea is to compile the subdev userspace API code out when not needed. Not > all media controller drivers need that API. Sure, but it is not a lot of code, and I think it is somewhat confusing. Just my opinion, though. 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
