cron job: media_tree daily build: ERRORS

2018-09-01 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sun Sep 2 05:00:10 CEST 2018 media-tree git hash:d842a7cf938b6e0f8a1aa9f1aec0476c9a599310 media_build

[GIT PULL FOR v4.20] Various fixes/enhancements

2018-09-01 Thread Hans Verkuil
The following changes since commit d842a7cf938b6e0f8a1aa9f1aec0476c9a599310: media: adv7842: enable reduced fps detection (2018-08-31 10:03:51 -0400) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v4.20a for you to fetch changes up to

[PATCH] vicodec: fix sparse warning

2018-09-01 Thread Hans Verkuil
drivers/media/platform/vicodec/vicodec-core.c:160:25: warning: variable 'q_out' set but not used [-Wunused-but-set-variable] It's indeed not used, and it can be removed. Signed-off-by: Hans Verkuil --- drivers/media/platform/vicodec/vicodec-core.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH] videodev2.h.rst.exceptions: add V4L2_DV_FL_CAN_DETECT_REDUCED_FPS

2018-09-01 Thread Hans Verkuil
This fixes a documentation warning: Documentation/output/videodev2.h.rst:6: WARNING: undefined label: v4l2-dv-fl-can-detect-reduced-fps (if the link has no caption the label must precede a section header) Signed-off-by: Hans Verkuil --- Documentation/media/videodev2.h.rst.exceptions | 1 + 1

[PATCHv3 09/10] media-request: EPERM -> EACCES/EBUSY

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil If requests are not supported by the driver, then return EACCES, not EPERM. If you attempt to mix queueing buffers directly and using requests, then EBUSY is returned instead of EPERM: once a specific queueing mode has been chosen the queue is 'busy' if you attempt the other

[PATCHv3 06/10] media-request: add media_request_(un)lock_for_access

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil Add helper functions to prevent a completed request from being re-inited while it is being accessed. Signed-off-by: Hans Verkuil Reviewed-by: Tomasz Figa --- drivers/media/media-request.c | 10 +++ include/media/media-request.h | 56 +++

[PATCHv3 03/10] buffer.rst: only set V4L2_BUF_FLAG_REQUEST_FD for QBUF

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil Document that V4L2_BUF_FLAG_REQUEST_FD should only be used with VIDIOC_QBUF and cleared otherwise. Signed-off-by: Hans Verkuil Reviewed-by: Tomasz Figa --- Documentation/media/uapi/v4l/buffer.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCHv3 05/10] vb2: set reqbufs/create_bufs capabilities

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil Set the capabilities field of v4l2_requestbuffers and v4l2_create_buffers. The various mapping modes were easy, but for signaling the request capability a new 'supports_requests' bitfield was added to videobuf2-core.h (and set in vim2m and vivid). Drivers have to set this

[PATCHv3 00/10] Post-v18: Request API updates

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil Hi all, This patch series sits on top of the request_api topic branch in the media_tree. It makes some final (?) changes as discussed in: https://www.mail-archive.com/linux-media@vger.kernel.org/msg134419.html and: https://www.spinics.net/lists/linux-media/msg138596.html

[PATCHv3 10/10] media-request: update documentation

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil Various clarifications and readability improvements based on Laurent Pinchart's review of the documentation. Signed-off-by: Hans Verkuil Reviewed-by: Tomasz Figa --- .../uapi/mediactl/media-ioc-request-alloc.rst | 3 +- .../uapi/mediactl/media-request-ioc-queue.rst | 7

[PATCHv3 04/10] videodev2.h: add new capabilities for buffer types

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil VIDIOC_REQBUFS and VIDIOC_CREATE_BUFFERS will return capabilities telling userspace what the given buffer type is capable of. Signed-off-by: Hans Verkuil Reviewed-by: Tomasz Figa --- .../media/uapi/v4l/vidioc-create-bufs.rst | 14 ++-

[PATCH v2 1/2] media: ov2680: don't register the v4l2 subdevice before checking chip ID

2018-09-01 Thread Javier Martinez Canillas
The driver registers the v4l2 subdevice before attempting to power on the chip and checking its ID. This means that a media device driver that it's waiting for this subdevice to be bound, will prematurely expose its media device node to userspace because if something goes wrong the media entity

[PATCHv3 01/10] media-request: return -EINVAL for invalid request_fds

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil Instead of returning -ENOENT when a request_fd was not found (VIDIOC_QBUF and VIDIOC_G/S/TRY_EXT_CTRLS), we now return -EINVAL. This is in line with what we do when invalid dmabuf fds are passed to e.g. VIDIOC_QBUF. Also document that EINVAL is returned for invalid m.fd

[PATCHv3 08/10] v4l2-ctrls: improve media_request_(un)lock_for_update

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil The request reference count was decreased again once a reference to the request object was taken. Postpone this until we finished using the object. In theory I think it is possible that the request_fd can be closed by the application from another thread. In that case when

[PATCHv3 02/10] v4l2-ctrls: return -EACCES if request wasn't completed

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil For now (this might be relaxed in the future) we do not allow getting controls from a request that isn't completed. In that case we return -EACCES. Update the documentation accordingly. Signed-off-by: Hans Verkuil Reviewed-by: Tomasz Figa ---

[PATCHv3 07/10] v4l2-ctrls: use media_request_(un)lock_for_access

2018-09-01 Thread Hans Verkuil
From: Hans Verkuil When getting control values from a completed request, we have to protect the request against being re-inited when it is being accessed by calling media_request_(un)lock_for_access. Signed-off-by: Hans Verkuil Reviewed-by: Tomasz Figa ---

[PATCH] vicodec: fix wrong sizeimage

2018-09-01 Thread Hans Verkuil
The initial sizeimage for the compressed decoder output was wrong. The size of the output was incorrectly used to calculate the image size, that should have been the size of the capture. Rework the code to fix this. Signed-off-by: Hans Verkuil --- diff --git

Re: [PATCH] media: ov2680: register the v4l2 subdev async at the end of probe

2018-09-01 Thread Javier Martinez Canillas
Hi Sakari, Thanks for the feedback. On 09/01/2018 01:46 PM, Sakari Ailus wrote: > Hi Javier, > > On Fri, Aug 31, 2018 at 05:19:06PM +0200, Javier Martinez Canillas wrote: >> The driver registers the subdev async in the middle of the probe function >> but this has to be done at the very end of

Re: [PATCH v2] media: ov5640: do not change mode if format or frame interval is unchanged

2018-09-01 Thread Sakari Ailus
On Thu, Aug 16, 2018 at 09:56:13AM +, Hugues FRUCHET wrote: > Hi all, > > Please ignore this v2, the v1 was merged. > I've just pushed a new patch which fixes the regression observed, see: > https://www.mail-archive.com/linux-media@vger.kernel.org/msg134413.html > > Sorry for inconvenience.