From: Russell King <rmk+ker...@armlinux.org.uk>

Setting and getting frame rates is part of the negotiation mechanism
between subdevs.  The lack of support means that a frame rate at the
sensor can't be negotiated through the subdev path.

Add support at MIPI CSI2 level for handling this part of the
negotiation.

Signed-off-by: Russell King <rmk+ker...@armlinux.org.uk>
Signed-off-by: Steve Longerbeam <steve_longerb...@mentor.com>
---
 drivers/staging/media/imx/imx6-mipi-csi2.c | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/drivers/staging/media/imx/imx6-mipi-csi2.c 
b/drivers/staging/media/imx/imx6-mipi-csi2.c
index 23dca80..c62f14e 100644
--- a/drivers/staging/media/imx/imx6-mipi-csi2.c
+++ b/drivers/staging/media/imx/imx6-mipi-csi2.c
@@ -34,6 +34,7 @@ struct csi2_dev {
        struct v4l2_subdev      sd;
        struct media_pad       pad[CSI2_NUM_PADS];
        struct v4l2_mbus_framefmt format_mbus;
+       struct v4l2_fract      frame_interval;
        struct clk             *dphy_clk;
        struct clk             *cfg_clk;
        struct clk             *pix_clk; /* what is this? */
@@ -397,6 +398,30 @@ static int csi2_set_fmt(struct v4l2_subdev *sd,
        return 0;
 }
 
+static int csi2_g_frame_interval(struct v4l2_subdev *sd,
+                                struct v4l2_subdev_frame_interval *fi)
+{
+       struct csi2_dev *csi2 = sd_to_dev(sd);
+
+       fi->interval = csi2->frame_interval;
+
+       return 0;
+}
+
+static int csi2_s_frame_interval(struct v4l2_subdev *sd,
+                                struct v4l2_subdev_frame_interval *fi)
+{
+       struct csi2_dev *csi2 = sd_to_dev(sd);
+
+       /* Output pads mirror active input pad, no limits on input pads */
+       if (fi->pad != CSI2_SINK_PAD)
+               fi->interval = csi2->frame_interval;
+
+       csi2->frame_interval = fi->interval;
+
+       return 0;
+}
+
 /*
  * retrieve our pads parsed from the OF graph by the media device
  */
@@ -430,6 +455,8 @@ static struct v4l2_subdev_core_ops csi2_core_ops = {
 
 static struct v4l2_subdev_video_ops csi2_video_ops = {
        .s_stream = csi2_s_stream,
+       .g_frame_interval = csi2_g_frame_interval,
+       .s_frame_interval = csi2_s_frame_interval,
 };
 
 static struct v4l2_subdev_pad_ops csi2_pad_ops = {
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to