[media] vpif_display: use a v4l2_subdev pointer to call a subdev

This makes it easier to have outputs without subdevs.
This needs more work. The way the outputs are configured should be identical
to how inputs are configured.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Tested-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

2 files changed, 10 insertions, 8 deletions

diff --git a/drivers/media/platform/davinci/vpif_display.c b/drivers/media/platform/davinci/vpif_display.c
index 4e2800043c9f..6229e4844bed 100644
--- a/drivers/media/platform/davinci/vpif_display.c
+++ b/drivers/media/platform/davinci/vpif_display.c
@@ -1246,6 +1246,8 @@ static int vpif_s_output(struct file *file, void *priv, unsigned int i)
                 vpif_err("Failed to set output standard\n");
 
         ch->output_idx = i;
+        if (vpif_obj.sd[i])
+                ch->sd = vpif_obj.sd[i];
         return ret;
 }
 
@@ -1317,14 +1319,13 @@ static int vpif_s_dv_timings(struct file *file, void *priv,
         }
 
         /* Configure subdevice timings, if any */
-        ret = v4l2_subdev_call(vpif_obj.sd[ch->output_idx],
-                        video, s_dv_timings, timings);
+        ret = v4l2_subdev_call(ch->sd, video, s_dv_timings, timings);
         if (ret == -ENOIOCTLCMD) {
                 vpif_dbg(2, debug, "Custom DV timings not supported by "
                                 "subdevice\n");
-                return -EINVAL;
+                return -ENODATA;
         }
-        if (ret < 0) {
+        if (ret < 0 && ret != -ENODEV) {
                 vpif_dbg(2, debug, "Error setting custom DV timings\n");
                 return ret;
         }
@@ -1449,8 +1450,7 @@ static int vpif_dbg_g_register(struct file *file, void *priv,
         struct vpif_fh *fh = priv;
         struct channel_obj *ch = fh->channel;
 
-        return v4l2_subdev_call(vpif_obj.sd[ch->output_idx], core,
-                        g_register, reg);
+        return v4l2_subdev_call(ch->sd, core, g_register, reg);
 }
 
 /*
@@ -1467,8 +1467,7 @@ static int vpif_dbg_s_register(struct file *file, void *priv,
         struct vpif_fh *fh = priv;
         struct channel_obj *ch = fh->channel;
 
-        return v4l2_subdev_call(vpif_obj.sd[ch->output_idx], core,
-                        s_register, reg);
+        return v4l2_subdev_call(ch->sd, core, s_register, reg);
 }
 #endif
 
@@ -1739,6 +1738,8 @@ static __init int vpif_probe(struct platform_device *pdev)
 
                 }
                 ch->initialized = 0;
+                if (subdev_count)
+                        ch->sd = vpif_obj.sd[0];
                 ch->channel_id = j;
                 if (j < 2)
                         ch->common[VPIF_VIDEO_INDEX].numbuffers =
diff --git a/drivers/media/platform/davinci/vpif_display.h b/drivers/media/platform/davinci/vpif_display.h
index 532ee9e8ce38..a5a18f74395c 100644
--- a/drivers/media/platform/davinci/vpif_display.h
+++ b/drivers/media/platform/davinci/vpif_display.h
@@ -125,6 +125,7 @@ struct channel_obj {
         u8 initialized;                 /* flag to indicate whether
                                          * encoder is initialized */
         u32 output_idx;                 /* Current output index */
+        struct v4l2_subdev *sd;         /* Current output subdev(may be NULL) */
 
         enum vpif_channel_id channel_id;/* Identifies channel */
         struct vpif_params vpifparams;