This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/cgit.cgi/v4l-utils.git tree:

Subject: v4l2-compliance: testRequests(): drop V4L2_BUF_FLAG_QUEUED check
Author:  Hans Verkuil <hverkuil-ci...@xs4all.nl>
Date:    Thu Aug 22 11:57:11 2024 +0200

After queueing the request it checks that the buffer is no longer
'IN_REQUEST' and it also checks that it is in state QUEUED, but that
last check is too strict since the buffer might already be marked DONE.

Just drop it.

Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 utils/v4l2-compliance/v4l2-test-buffers.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=37d02fb3e909237320b0bca6edf736870677fb1f
diff --git a/utils/v4l2-compliance/v4l2-test-buffers.cpp 
b/utils/v4l2-compliance/v4l2-test-buffers.cpp
index 2ec7d7cf5e1a..e614bd896996 100644
--- a/utils/v4l2-compliance/v4l2-test-buffers.cpp
+++ b/utils/v4l2-compliance/v4l2-test-buffers.cpp
@@ -2700,10 +2700,9 @@ int testRequests(struct node *node, bool test_streaming)
                }
                fail_on_test_val(ret, ret);
                fail_on_test(buf.querybuf(node, i));
-               // Check that the buffer is now queued up
+               // Check that the buffer is now queued up (i.e. no longer 
'IN_REQUEST')
                fail_on_test(buf.g_flags() & V4L2_BUF_FLAG_IN_REQUEST);
                fail_on_test(!(buf.g_flags() & V4L2_BUF_FLAG_REQUEST_FD));
-               fail_on_test(!(buf.g_flags() & V4L2_BUF_FLAG_QUEUED));
                // Re-initing or requeuing the request is no longer possible
                fail_on_test(doioctl_fd(buf_req_fds[i], 
MEDIA_REQUEST_IOC_REINIT, nullptr) != EBUSY);
                fail_on_test(doioctl_fd(buf_req_fds[i], 
MEDIA_REQUEST_IOC_QUEUE, nullptr) != EBUSY);

Reply via email to