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?
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
