From: "Lad, Prabhakar" <prabhakar.cse...@gmail.com>

this patch adds support to return -ENODATA for *dv_timings calls
if the current output does not support it.

Signed-off-by: Lad, Prabhakar <prabhakar.cse...@gmail.com>
Acked-by: Scott Jiang <scott.jiang.li...@gmail.com>
Tested-by: Scott Jiang <scott.jiang.li...@gmail.com>
---
 drivers/media/platform/blackfin/bfin_capture.c | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/drivers/media/platform/blackfin/bfin_capture.c 
b/drivers/media/platform/blackfin/bfin_capture.c
index f97d94d..2dead84 100644
--- a/drivers/media/platform/blackfin/bfin_capture.c
+++ b/drivers/media/platform/blackfin/bfin_capture.c
@@ -487,6 +487,11 @@ static int bcap_enum_dv_timings(struct file *file, void 
*priv,
                                struct v4l2_enum_dv_timings *timings)
 {
        struct bcap_device *bcap_dev = video_drvdata(file);
+       struct v4l2_input input;
+
+       input = bcap_dev->cfg->inputs[bcap_dev->cur_input];
+       if (!(input.capabilities & V4L2_IN_CAP_DV_TIMINGS))
+               return -ENODATA;
 
        timings->pad = 0;
 
@@ -498,6 +503,11 @@ static int bcap_query_dv_timings(struct file *file, void 
*priv,
                                struct v4l2_dv_timings *timings)
 {
        struct bcap_device *bcap_dev = video_drvdata(file);
+       struct v4l2_input input;
+
+       input = bcap_dev->cfg->inputs[bcap_dev->cur_input];
+       if (!(input.capabilities & V4L2_IN_CAP_DV_TIMINGS))
+               return -ENODATA;
 
        return v4l2_subdev_call(bcap_dev->sd, video,
                                query_dv_timings, timings);
@@ -507,6 +517,11 @@ static int bcap_g_dv_timings(struct file *file, void *priv,
                                struct v4l2_dv_timings *timings)
 {
        struct bcap_device *bcap_dev = video_drvdata(file);
+       struct v4l2_input input;
+
+       input = bcap_dev->cfg->inputs[bcap_dev->cur_input];
+       if (!(input.capabilities & V4L2_IN_CAP_DV_TIMINGS))
+               return -ENODATA;
 
        *timings = bcap_dev->dv_timings;
        return 0;
@@ -516,7 +531,13 @@ static int bcap_s_dv_timings(struct file *file, void *priv,
                                struct v4l2_dv_timings *timings)
 {
        struct bcap_device *bcap_dev = video_drvdata(file);
+       struct v4l2_input input;
        int ret;
+
+       input = bcap_dev->cfg->inputs[bcap_dev->cur_input];
+       if (!(input.capabilities & V4L2_IN_CAP_DV_TIMINGS))
+               return -ENODATA;
+
        if (vb2_is_busy(&bcap_dev->buffer_queue))
                return -EBUSY;
 
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to