[media] omap3isp: Calculate vpclk_div for CSI-2
authorSakari Ailus <sakari.ailus@iki.fi>
Wed, 25 Mar 2015 22:57:32 +0000 (19:57 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Thu, 2 Apr 2015 19:42:04 +0000 (16:42 -0300)
The video port clock is l3_ick divided by vpclk_div. This clock must be high
enough for the external pixel rate. The video port requires two clock cycles
to process a pixel.

Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/platform/omap3isp/ispcsi2.c
include/media/omap3isp.h

index 45ac90a49889e67777f4e4fea1f3e435cd8edfeb..a78338d012b4df76cea20b0ff882c62a2d28edf0 100644 (file)
@@ -548,6 +548,7 @@ int omap3isp_csi2_reset(struct isp_csi2_device *csi2)
 
 static int csi2_configure(struct isp_csi2_device *csi2)
 {
+       struct isp_pipeline *pipe = to_isp_pipeline(&csi2->subdev.entity);
        const struct isp_bus_cfg *buscfg;
        struct isp_device *isp = csi2->isp;
        struct isp_csi2_timing_cfg *timing = &csi2->timing[0];
@@ -570,7 +571,12 @@ static int csi2_configure(struct isp_csi2_device *csi2)
        csi2->frame_skip = 0;
        v4l2_subdev_call(sensor, sensor, g_skip_frames, &csi2->frame_skip);
 
-       csi2->ctrl.vp_out_ctrl = buscfg->bus.csi2.vpclk_div;
+       csi2->ctrl.vp_out_ctrl =
+               clamp_t(unsigned int, pipe->l3_ick / pipe->external_rate - 1,
+                       1, 3);
+       dev_dbg(isp->dev, "%s: l3_ick %lu, external_rate %u, vp_out_ctrl %u\n",
+               __func__, pipe->l3_ick,  pipe->external_rate,
+               csi2->ctrl.vp_out_ctrl);
        csi2->ctrl.frame_mode = ISP_CSI2_FRAME_IMMEDIATE;
        csi2->ctrl.ecc_enable = buscfg->bus.csi2.crc;
 
index 39e0748b0d316b232a304024a61da06938ccc248..0f0c08b48829527ce426f3ad99c723ed94a1e9fb 100644 (file)
@@ -129,11 +129,9 @@ struct isp_ccp2_cfg {
 /**
  * struct isp_csi2_cfg - CSI2 interface configuration
  * @crc: Enable the cyclic redundancy check
- * @vpclk_div: Video port output clock control
  */
 struct isp_csi2_cfg {
        unsigned crc:1;
-       unsigned vpclk_div:2;
        struct isp_csiphy_lanes_cfg lanecfg;
 };