Re: [RFC PATCH 0/3] Add VIDIOC_SUBDEV_QUERYCAP
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 >>> >>> 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [RFC PATCH 0/3] Add VIDIOC_SUBDEV_QUERYCAP
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 > > > > 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
Re: [RFC PATCH 0/3] Add VIDIOC_SUBDEV_QUERYCAP
On 05/01/2015 01:33 PM, Hans Verkuil wrote: > From: Hans Verkuil > > 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[RFC PATCH 0/3] Add VIDIOC_SUBDEV_QUERYCAP
From: Hans Verkuil 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. 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 -- 2.1.4 -- 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