[media] v4l2-dv-timings: add condition checks for reduced fps

Added a helper function to check necessary conditions required for
reduced fps. The reduced fps is supported for CVT and CEA861 timings.
CVT supports reduced fps only if reduced blanking v2 (indicated by
vsync = 8) is true. Whereas CEA861 supports reduced fps if
V4L2_DV_FL_CAN_REDUCE_FPS flag is true.

Signed-off-by: Prashant Laddha <prladdha@cisco.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

diff --git a/drivers/media/v4l2-core/v4l2-dv-timings.c b/drivers/media/v4l2-core/v4l2-dv-timings.c
index edb4125d1018fa3846b352de9611a729c9fe8d54..9926d521ea888973ba1dc57324c24ee2e9e50eed 100644 (file)
--- a/drivers/media/v4l2-core/v4l2-dv-timings.c
+++ b/drivers/media/v4l2-core/v4l2-dv-timings.c
@@ -210,7 +210,12 @@ bool v4l2_find_dv_timings_cap(struct v4l2_dv_timings *t,
                                          fnc, fnc_handle) &&
                    v4l2_match_dv_timings(t, v4l2_dv_timings_presets + i,
                                          pclock_delta)) {
+                       u32 flags = t->bt.flags & V4L2_DV_FL_REDUCED_FPS;
+
                        *t = v4l2_dv_timings_presets[i];
+                       if (can_reduce_fps(&t->bt))
+                               t->bt.flags |= flags;
+
                        return true;
                }
        }

diff --git a/include/media/v4l2-dv-timings.h b/include/media/v4l2-dv-timings.h
index a209526b6014b11fe15fae9b39565e34996980fb..69829a5a13685500d8d757be6d6aeecbafb26d11 100644 (file)
--- a/include/media/v4l2-dv-timings.h
+++ b/include/media/v4l2-dv-timings.h
@@ -183,4 +183,25 @@ bool v4l2_detect_gtf(unsigned frame_height, unsigned hfreq, unsigned vsync,
  */
 struct v4l2_fract v4l2_calc_aspect_ratio(u8 hor_landscape, u8 vert_portrait);
 
+/*
+ * reduce_fps - check if conditions for reduced fps are true.
+ * bt - v4l2 timing structure
+ * For different timings reduced fps is allowed if following conditions
+ * are met -
+ * For CVT timings: if reduced blanking v2 (vsync == 8) is true.
+ * For CEA861 timings: if V4L2_DV_FL_CAN_REDUCE_FPS flag is true.
+ */
+static inline  bool can_reduce_fps(struct v4l2_bt_timings *bt)
+{
+       if ((bt->standards & V4L2_DV_BT_STD_CVT) && (bt->vsync == 8))
+               return true;
+
+       if ((bt->standards & V4L2_DV_BT_STD_CEA861) &&
+           (bt->flags & V4L2_DV_FL_CAN_REDUCE_FPS))
+               return true;
+
+       return false;
+}
+
+
 #endif