From: Hans Verkuil <hans.verkuil@cisco.com>
Date: Sun, 10 Mar 2013 10:40:00 +0000 (+0100)
Subject: v4l2-compliance: missing check on the frameinterval
X-Git-Tag: v4l-utils-0.9.4~40
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=125e23e16fee4346729b104ed8f17dbabf27e3c1;p=platform%2Fupstream%2Fv4l-utils.git

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
--- 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;