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: check that G_DV_TIMINGS matches S_DV_TIMINGS
Author:  Hans Verkuil <hans.verk...@cisco.com>
Date:    Thu Feb 8 15:31:30 2018 +0100

When enumerating valid timings and setting them in turn, call G_DV_TIMINGS
to verify that the driver actually set the new timings.

Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>

 utils/v4l2-compliance/v4l2-test-io-config.cpp | 12 ++++++++++++
 1 file changed, 12 insertions(+)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=d77a3b80c20f5239712a517bc08303954bbc0c6b
diff --git a/utils/v4l2-compliance/v4l2-test-io-config.cpp 
b/utils/v4l2-compliance/v4l2-test-io-config.cpp
index f10c8b82204a..404955478b73 100644
--- a/utils/v4l2-compliance/v4l2-test-io-config.cpp
+++ b/utils/v4l2-compliance/v4l2-test-io-config.cpp
@@ -199,6 +199,12 @@ static int checkTimings(struct node *node, bool 
has_timings, bool is_input)
                fail_on_test(doioctl(node, VIDIOC_S_DV_TIMINGS, 
&enumtimings.timings));
                fail_on_test(check_0(enumtimings.timings.bt.reserved,
                                     sizeof(enumtimings.timings.bt.reserved)));
+               
+               struct v4l2_dv_timings g_timings;
+               fail_on_test(doioctl(node, VIDIOC_G_DV_TIMINGS, &g_timings));
+               fail_on_test(g_timings.bt.width != 
enumtimings.timings.bt.width);
+               fail_on_test(g_timings.bt.height != 
enumtimings.timings.bt.height);
+
                if (node->is_vbi)
                        continue;
                fmt.type = type;
@@ -276,6 +282,12 @@ static int checkSubDevEnumTimings(struct node *node, __u32 
pad)
                fail_on_test(doioctl(node, VIDIOC_S_DV_TIMINGS, 
&enumtimings.timings));
                fail_on_test(check_0(enumtimings.timings.bt.reserved,
                                     sizeof(enumtimings.timings.bt.reserved)));
+
+               struct v4l2_dv_timings g_timings;
+
+               fail_on_test(doioctl(node, VIDIOC_G_DV_TIMINGS, &g_timings));
+               fail_on_test(g_timings.bt.width != 
enumtimings.timings.bt.width);
+               fail_on_test(g_timings.bt.height != 
enumtimings.timings.bt.height);
        }
        enumtimings.pad = node->entity.pads;
        enumtimings.index = 0;

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

Reply via email to