v4l2-compliance: missing check on the frameinterval

A wrong discrete frameinterval as returned by ENUM_FRAMEINTERVALS
wasn't detected.

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

diff --git a/utils/v4l2-compliance/v4l2-test-formats.cpp b/utils/v4l2-compliance/v4l2-test-formats.cpp
index a4f73c8..4e26020 100644 (file)
--- a/utils/v4l2-compliance/v4l2-test-formats.cpp
+++ b/utils/v4l2-compliance/v4l2-test-formats.cpp
@@ -82,6 +82,10 @@ static int testEnumFrameIntervals(struct node *node, __u32 pixfmt, __u32 w, __u3
                switch (frmival.type) {
                case V4L2_FRMIVAL_TYPE_DISCRETE:
                        ret = check_fract(&frmival.discrete);
+                       if (ret)
+                               return fail("invalid frameinterval %d (%d/%d)\n", f,
+                                               frmival.discrete.numerator,
+                                               frmival.discrete.denominator);
                        if (found_stepwise)
                                return fail("mixing discrete and stepwise is not allowed\n");
                        break;