[libcamera-devel] [PATCH 15/15] v4l2: v4l2_camera_proxy: Fix v4l2-compliance streaming tests
Jacopo Mondi
jacopo at jmondi.org
Wed Jun 17 17:43:55 CEST 2020
Hi Paul,
On Tue, Jun 16, 2020 at 10:12:44PM +0900, Paul Elder wrote:
> Fix v4l2-compliance streaming tests, for reqbufs, qbuf, dqbuf, and
> streamon.
This seems a bit of a collection of random fixes. I'm not opposed to
that in principle, but I think the single issues should at least be
listed in the commit message.
>
> Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
> ---
> src/v4l2/v4l2_camera_proxy.cpp | 15 +++++++++++++--
> 1 file changed, 13 insertions(+), 2 deletions(-)
>
> diff --git a/src/v4l2/v4l2_camera_proxy.cpp b/src/v4l2/v4l2_camera_proxy.cpp
> index e4d534a..fca728a 100644
> --- a/src/v4l2/v4l2_camera_proxy.cpp
> +++ b/src/v4l2/v4l2_camera_proxy.cpp
> @@ -232,7 +232,6 @@ void V4L2CameraProxy::updateBuffers()
> buf.timestamp.tv_usec = fmd.timestamp % 1000000;
> buf.sequence = fmd.sequence;
>
> - buf.flags |= V4L2_BUF_FLAG_DONE;
> break;
> case FrameMetadata::FrameError:
> buf.flags |= V4L2_BUF_FLAG_ERROR;
> @@ -517,6 +516,9 @@ int V4L2CameraProxy::vidioc_reqbufs(int fd, struct v4l2_requestbuffers *arg)
> return ret;
> }
>
> + if (!mmaps_.empty())
> + return -EBUSY;
> +
> unlock(fd);
> return freeBuffers();
> }
> @@ -616,6 +618,9 @@ int V4L2CameraProxy::vidioc_qbuf(int fd, struct v4l2_buffer *arg)
> if (arg->index >= bufferCount_)
> return -EINVAL;
>
> + if (buffers_[arg->index].flags & V4L2_BUF_FLAG_QUEUED)
> + return -EINVAL;
> +
> int ret = lock(fd);
> if (ret < 0)
> return ret;
> @@ -629,9 +634,12 @@ int V4L2CameraProxy::vidioc_qbuf(int fd, struct v4l2_buffer *arg)
> if (ret < 0)
> return ret;
>
> - arg->flags |= V4L2_BUF_FLAG_QUEUED;
> + arg->flags |= V4L2_BUF_FLAG_QUEUED | V4L2_BUF_FLAG_MAPPED;
> arg->flags &= ~V4L2_BUF_FLAG_DONE;
>
> + buffers_[arg->index].flags |= V4L2_BUF_FLAG_QUEUED | V4L2_BUF_FLAG_MAPPED;
> + buffers_[arg->index].flags &= ~V4L2_BUF_FLAG_DONE;
> +
> return ret;
> }
>
> @@ -694,6 +702,9 @@ int V4L2CameraProxy::vidioc_streamon(int fd, int *arg)
> if (arg == nullptr)
> return -EFAULT;
>
> + if (bufferCount_ == 0)
> + return -EINVAL;
> +
It's a bit hard to review them without much context :/
> if (streaming_)
> return 0;
>
> --
> 2.27.0
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
More information about the libcamera-devel
mailing list