ENGR00243315-3 MXC V4L2 Capture:Remove unnecessary mclk setting
authorLiu Ying <Ying.Liu@freescale.com>
Fri, 8 Mar 2013 08:44:41 +0000 (16:44 +0800)
committerNitin Garg <nitin.garg@freescale.com>
Wed, 16 Apr 2014 13:57:59 +0000 (08:57 -0500)
commit f8e1a3bb62eecf93a31a51c4dbe08a0214fa1d57 added a hard
coding for csi_parma.mclk setting to 27MHz. The comment added by
that commit is totally wrong by telling that csi_param.mclk
would be a kind of 'pixel clock' set in 'csi_data_dest' register.
This patch removes the unnecessary mclk setting for csi_param.mclk
variable, since it is only valid for CSI test mode.

Conflicts:

drivers/media/video/mxc/capture/mxc_v4l2_capture.c

Signed-off-by: Liu Ying <Ying.Liu@freescale.com>
(cherry picked from commit 5fed1e3fde2d63c80f414f204734d35ceecef561)

drivers/media/platform/mxc/capture/mxc_v4l2_capture.c

index 6f1920a97ea267c954b8f9781347ae4a59e588d1..5f4db9582abab7b48c8c4a29561fda7ce1a45a7c 100755 (executable)
@@ -1354,19 +1354,11 @@ static int mxc_v4l2_s_param(cam_data *cam, struct v4l2_streamparm *parm)
        csi_param.csi = cam->csi;
        csi_param.mclk = 0;
 
-       /*This may not work on other platforms. Check when adding a new one.*/
-       /*The mclk clock was never set correclty in the ipu register*/
-       /*for now we are going to use this mclk as pixel clock*/
-       /*to set csi0_data_dest register.*/
-       /*This is a workaround which should be fixed*/
        pr_debug("   clock_curr=mclk=%d\n", ifparm.u.bt656.clock_curr);
-       if (ifparm.u.bt656.clock_curr == 0) {
+       if (ifparm.u.bt656.clock_curr == 0)
                csi_param.clk_mode = IPU_CSI_CLK_MODE_CCIR656_INTERLACED;
-               /*protocol bt656 use 27Mhz pixel clock */
-               csi_param.mclk = 27000000;
-       } else {
+       else
                csi_param.clk_mode = IPU_CSI_CLK_MODE_GATED_CLK;
-       }
 
        csi_param.pixclk_pol = ifparm.u.bt656.latch_clk_inv;