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-tracer: replace buftype2s with val2s
Author:  Deborah Brouwer <deborah.brou...@collabora.com>
Date:    Mon Nov 13 12:06:13 2023 -0800

While buftype2s gives a written description of the buffer type, val2s will
reproduce the type exactly as it appears in videodev2.h which is easier to
follow for tracing.

Signed-off-by: Deborah Brouwer <deborah.brou...@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 utils/v4l2-tracer/retrace.cpp      | 19 ++++++++++++-------
 utils/v4l2-tracer/trace-helper.cpp |  6 +++---
 utils/v4l2-tracer/trace.cpp        |  4 ++--
 3 files changed, 17 insertions(+), 12 deletions(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=7e03ea268ad3833eeeb089c90bd087a3db3e8cf2
diff --git a/utils/v4l2-tracer/retrace.cpp b/utils/v4l2-tracer/retrace.cpp
index 14c42568e40f..1bec635de760 100644
--- a/utils/v4l2-tracer/retrace.cpp
+++ b/utils/v4l2-tracer/retrace.cpp
@@ -336,7 +336,8 @@ void retrace_vidioc_querybuf(int fd_retrace, json_object 
*ioctl_args_user)
 
        if (is_verbose() || (errno != 0)) {
                fprintf(stderr, "%s, index: %d, fd: %d, ",
-                       buftype2s((int) buf->type).c_str(), buf->index, 
fd_retrace);
+                       val2s(buf->type, v4l2_buf_type_val_def).c_str(),
+                       buf->index, fd_retrace);
                perror("VIDIOC_QUERYBUF");
                debug_line_info();
                print_context();
@@ -360,7 +361,8 @@ void retrace_vidioc_qbuf(int fd_retrace, json_object 
*ioctl_args_user)
 
        if (is_verbose() || (errno != 0)) {
                fprintf(stderr, "%s, index: %d, fd: %d, ",
-                       buftype2s((int) ptr->type).c_str(), ptr->index, 
fd_retrace);
+                       val2s(ptr->type, v4l2_buf_type_val_def).c_str(),
+                       ptr->index, fd_retrace);
                perror("VIDIOC_QBUF");
                debug_line_info();
                print_context();
@@ -395,7 +397,8 @@ void retrace_vidioc_dqbuf(int fd_retrace, json_object 
*ioctl_args_user)
 
        if (is_verbose() || (errno != 0)) {
                fprintf(stderr, "%s, index: %d, fd: %d, ",
-                       buftype2s((int) buf->type).c_str(), buf->index, 
fd_retrace);
+                       val2s(buf->type, v4l2_buf_type_val_def).c_str(),
+                       buf->index, fd_retrace);
                perror("VIDIOC_DQBUF");
                debug_line_info();
                print_context();
@@ -416,7 +419,8 @@ void retrace_vidioc_prepare_buf(int fd_retrace, json_object 
*ioctl_args_user)
 
        if (is_verbose() || (errno != 0)) {
                fprintf(stderr, "%s, index: %d, fd: %d, ",
-                       buftype2s((int) buf->type).c_str(), buf->index, 
fd_retrace);
+                       val2s(buf->type, v4l2_buf_type_val_def).c_str(),
+                       buf->index, fd_retrace);
                perror("VIDIOC_PREPARE_BUF");
                debug_line_info();
                print_context();
@@ -482,7 +486,7 @@ void retrace_vidioc_streamon(int fd_retrace, json_object 
*ioctl_args)
        ioctl(fd_retrace, VIDIOC_STREAMON, &buf_type);
 
        if (is_verbose() || (errno != 0)) {
-               fprintf(stderr, "%s, ", buftype2s(buf_type).c_str());
+               fprintf(stderr, "%s, ", val2s(buf_type, 
v4l2_buf_type_val_def).c_str());
                perror("VIDIOC_STREAMON");
        }
 }
@@ -497,7 +501,7 @@ void retrace_vidioc_streamoff(int fd_retrace, json_object 
*ioctl_args)
        ioctl(fd_retrace, VIDIOC_STREAMOFF, &buf_type);
 
        if (is_verbose() || (errno != 0)) {
-               fprintf(stderr, "%s, ", buftype2s(buf_type).c_str());
+               fprintf(stderr, "%s, ", val2s(buf_type, 
v4l2_buf_type_val_def).c_str());
                perror("VIDIOC_STREAMOFF");
        }
 }
@@ -1451,7 +1455,8 @@ void retrace_mem(json_object *mem_obj)
                write_to_output_buffer(buffer_pointer, bytesused, mem_obj);
 
        debug_line_info("\n\t%s, bytesused: %d, offset: %d, addr: %ld",
-                       buftype2s(type).c_str(), bytesused, offset, 
buffer_address_retrace);
+                       val2s(type, v4l2_buf_type_val_def).c_str(),
+                       bytesused, offset, buffer_address_retrace);
        print_context();
 }
 
diff --git a/utils/v4l2-tracer/trace-helper.cpp 
b/utils/v4l2-tracer/trace-helper.cpp
index e5094b675b78..a1e83a44fac2 100644
--- a/utils/v4l2-tracer/trace-helper.cpp
+++ b/utils/v4l2-tracer/trace-helper.cpp
@@ -228,7 +228,7 @@ void print_buffers_trace(void)
                return;
        for (auto &b : ctx_trace.buffers) {
                fprintf(stderr, "fd: %d, %s, index: %d, display_order: %ld, 
bytesused: %d, ",
-                       b.fd, buftype2s(b.type).c_str(), b.index, 
b.display_order, b.bytesused);
+                       b.fd, val2s(b.type, v4l2_buf_type_val_def).c_str(), 
b.index, b.display_order, b.bytesused);
                fprintf(stderr, "address: %lu, offset: %u \n",  b.address, 
b.offset);
        }
 }
@@ -320,7 +320,7 @@ void s_ext_ctrls_setup(struct v4l2_ext_controls 
*ext_controls)
 
 void qbuf_setup(struct v4l2_buffer *buf)
 {
-       debug_line_info("\n\t%s, index: %d", buftype2s((int) 
buf->type).c_str(), buf->index);
+       debug_line_info("\n\t%s, index: %d", val2s(buf->type, 
v4l2_buf_type_val_def).c_str(), buf->index);
 
        int buf_fd = get_buffer_fd_trace(buf->type, buf->index);
        __u32 buf_offset = get_buffer_offset_trace(buf->type, buf->index);
@@ -361,7 +361,7 @@ void streamoff_cleanup(v4l2_buf_type buf_type)
 {
        debug_line_info();
        if (is_verbose() || 
(getenv("V4L2_TRACER_OPTION_WRITE_DECODED_TO_YUV_FILE") != nullptr)) {
-               fprintf(stderr, "VIDIOC_STREAMOFF: %s\n", 
buftype2s(buf_type).c_str());
+               fprintf(stderr, "VIDIOC_STREAMOFF: %s\n", val2s(buf_type, 
v4l2_buf_type_val_def).c_str());
                fprintf(stderr, "%s, %s %s, width: %d, height: %d\n",
                        val2s(ctx_trace.compression_format, 
v4l2_pix_fmt_val_def).c_str(),
                        val2s(ctx_trace.pixelformat, 
v4l2_pix_fmt_val_def).c_str(),
diff --git a/utils/v4l2-tracer/trace.cpp b/utils/v4l2-tracer/trace.cpp
index 0e8531ff1cc5..f81f68d15ff2 100644
--- a/utils/v4l2-tracer/trace.cpp
+++ b/utils/v4l2-tracer/trace.cpp
@@ -181,8 +181,8 @@ void trace_mem_decoded(void)
                         */
                        if (it->bytesused < expected_length)
                                break;
-                       debug_line_info("\n\tDisplaying: %ld, %s, index: %d",
-                                       it->display_order, 
buftype2s(it->type).c_str(), it->index);
+                       debug_line_info("\n\tDisplaying: %ld, %s, index: %d", 
it->display_order,
+                                       val2s(it->type, 
v4l2_buf_type_val_def).c_str(), it->index);
                        displayed_count++;
 
                        if 
(getenv("V4L2_TRACER_OPTION_WRITE_DECODED_TO_YUV_FILE") != nullptr) {

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to