From: Hans Verkuil <hverkuil@xs4all.nl>
To: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: [GIT PULL FOR v3.17] Add support for compound controls, use in solo/go7007
Date: Fri, 27 Jun 2014 09:40:51 +0200 [thread overview]
Message-ID: <53AD2003.7020503@xs4all.nl> (raw)
Hi Mauro,
This is the pull request for the compound control patch series. The only
change made since the REVIEWv4 series (1) is that the v4l2_query_ext_ctrl
reserved[] field has been increased from 16 to 32 elements based on feedback
from Sakari.
Note that I did not go for variable number of dimensions for multi-dimensional
arrays. Instead I set the maximum at 8. Based on the feedback Sakari (2) gave
me and my own feelings on this topic I did not want to replace the
dims[V4L2_CTRL_MAX_DIMS] array with a dims pointer. If changing to a u32 *dims
is the only way to get this in, then I will do so but I am not ready to give
up on this. We're dealing with video hardware, not with 11-dimensional string
theory and introducing a pointer makes both kernel and (more importantly)
userspace unnecessarily complex.
I have added support for getting/setting multidimensional arrays to
v4l2-ctl here:
http://git.linuxtv.org/cgit.cgi/hverkuil/v4l-utils.git/log/?h=compound
I've also tested this with the solo and go7007 driver.
Regards,
Hans
(1) https://www.mail-archive.com/linux-media@vger.kernel.org/msg75929.html
(2) http://www.spinics.net/lists/linux-media/msg77233.html
The following changes since commit b5b620584b9c4644b85e932895a742e0c192d66c:
[media] technisat-sub2: Fix stream curruption on high bitrate (2014-06-26 09:20:18 -0300)
are available in the git repository at:
git://linuxtv.org/hverkuil/media_tree.git propapi-part4
for you to fetch changes up to 1ac3c9ba06f6c08ced9ef18520b05fea1deb1253:
go7007: add motion detection support. (2014-06-27 09:22:47 +0200)
----------------------------------------------------------------
Hans Verkuil (34):
v4l2-ctrls: increase internal min/max/step/def to 64 bit
v4l2-ctrls: use pr_info/cont instead of printk.
videodev2.h: add initial support for compound controls.
videodev2.h: add struct v4l2_query_ext_ctrl and VIDIOC_QUERY_EXT_CTRL.
v4l2-ctrls: add support for compound types.
v4l2: integrate support for VIDIOC_QUERY_EXT_CTRL.
v4l2-ctrls: create type_ops.
v4l2-ctrls: rewrite copy routines to operate on union v4l2_ctrl_ptr.
v4l2-ctrls: compare values only once.
v4l2-ctrls: use ptrs for all but the s32 type.
v4l2-ctrls: prepare for array support
v4l2-ctrls: prepare for array support.
v4l2-ctrls: type_ops can handle array elements.
v4l2-ctrls: add array support.
v4l2-ctrls: return elem_size instead of strlen
v4l2-ctrl: fix error return of copy_to/from_user.
DocBook media: document VIDIOC_QUERY_EXT_CTRL.
DocBook media: update VIDIOC_G/S/TRY_EXT_CTRLS.
DocBook media: fix coding style in the control example code
DocBook media: improve control section.
DocBook media: update control section.
v4l2-controls.txt: update to the new way of accessing controls.
v4l2-ctrls/videodev2.h: add u8 and u16 types.
DocBook media: document new u8 and u16 control types.
v4l2-ctrls: fix comments
v4l2-ctrls/v4l2-controls.h: add MD controls
DocBook media: document new motion detection controls.
v4l2: add a motion detection event.
DocBook: document new v4l motion detection event.
solo6x10: implement the new motion detection controls.
solo6x10: implement the motion detection event.
solo6x10: fix 'dma from stack' warning.
solo6x10: check dma_map_sg() return value
go7007: add motion detection support.
Documentation/DocBook/media/v4l/controls.xml | 278 ++++++++++++++++------
Documentation/DocBook/media/v4l/vidioc-dqevent.xml | 44 ++++
Documentation/DocBook/media/v4l/vidioc-g-ext-ctrls.xml | 51 +++-
Documentation/DocBook/media/v4l/vidioc-queryctrl.xml | 232 ++++++++++++++++---
Documentation/DocBook/media/v4l/vidioc-subscribe-event.xml | 8 +
Documentation/video4linux/v4l2-controls.txt | 63 +++--
drivers/media/i2c/mt9v032.c | 4 +-
drivers/media/i2c/smiapp/smiapp-core.c | 8 +-
drivers/media/pci/ivtv/ivtv-controls.c | 4 +-
drivers/media/platform/vivi.c | 9 +-
drivers/media/radio/si4713/si4713.c | 4 +-
drivers/media/v4l2-core/v4l2-common.c | 6 +-
drivers/media/v4l2-core/v4l2-ctrls.c | 847 ++++++++++++++++++++++++++++++++++++++++++++++---------------------
drivers/media/v4l2-core/v4l2-dev.c | 2 +
drivers/media/v4l2-core/v4l2-ioctl.c | 33 +++
drivers/media/v4l2-core/v4l2-subdev.c | 3 +
drivers/staging/media/go7007/go7007-driver.c | 127 +++++++---
drivers/staging/media/go7007/go7007-fw.c | 28 ++-
drivers/staging/media/go7007/go7007-priv.h | 16 ++
drivers/staging/media/go7007/go7007-v4l2.c | 317 +++++++++++++++++--------
drivers/staging/media/go7007/go7007.h | 40 ----
drivers/staging/media/go7007/saa7134-go7007.c | 1 -
drivers/staging/media/msi3101/msi001.c | 2 +-
drivers/staging/media/rtl2832u_sdr/rtl2832_sdr.c | 4 +-
drivers/staging/media/solo6x10/solo6x10-disp.c | 14 +-
drivers/staging/media/solo6x10/solo6x10-v4l2-enc.c | 183 ++++++++-------
drivers/staging/media/solo6x10/solo6x10.h | 26 +--
include/media/v4l2-ctrls.h | 123 +++++++---
include/media/v4l2-ioctl.h | 2 +
include/uapi/linux/v4l2-controls.h | 17 ++
include/uapi/linux/videodev2.h | 51 +++-
31 files changed, 1804 insertions(+), 743 deletions(-)
delete mode 100644 drivers/staging/media/go7007/go7007.h
next reply other threads:[~2014-06-27 7:41 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-27 7:40 Hans Verkuil [this message]
-- strict thread matches above, loose matches on Subject: below --
2014-07-16 14:53 [GIT PULL FOR v3.17] Add support for compound controls, use in solo/go7007 Hans Verkuil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53AD2003.7020503@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.