diff options
author | Liu Ying <Ying.Liu@freescale.com> | 2013-03-08 03:44:41 -0500 |
---|---|---|
committer | Nitin Garg <nitin.garg@freescale.com> | 2014-04-16 09:57:59 -0400 |
commit | b58c82ce8e5094299951f12844b705efde597806 (patch) | |
tree | f05c9aca0c9114f1f138bcd1ec9f898dbb1bcdb4 | |
parent | a0a1533563bc16052c93936ec832e5796a63c613 (diff) |
ENGR00243315-3 MXC V4L2 Capture:Remove unnecessary mclk setting
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)
-rwxr-xr-x | drivers/media/platform/mxc/capture/mxc_v4l2_capture.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/drivers/media/platform/mxc/capture/mxc_v4l2_capture.c b/drivers/media/platform/mxc/capture/mxc_v4l2_capture.c index 6f1920a97ea2..5f4db9582aba 100755 --- a/drivers/media/platform/mxc/capture/mxc_v4l2_capture.c +++ b/drivers/media/platform/mxc/capture/mxc_v4l2_capture.c | |||
@@ -1354,19 +1354,11 @@ static int mxc_v4l2_s_param(cam_data *cam, struct v4l2_streamparm *parm) | |||
1354 | csi_param.csi = cam->csi; | 1354 | csi_param.csi = cam->csi; |
1355 | csi_param.mclk = 0; | 1355 | csi_param.mclk = 0; |
1356 | 1356 | ||
1357 | /*This may not work on other platforms. Check when adding a new one.*/ | ||
1358 | /*The mclk clock was never set correclty in the ipu register*/ | ||
1359 | /*for now we are going to use this mclk as pixel clock*/ | ||
1360 | /*to set csi0_data_dest register.*/ | ||
1361 | /*This is a workaround which should be fixed*/ | ||
1362 | pr_debug(" clock_curr=mclk=%d\n", ifparm.u.bt656.clock_curr); | 1357 | pr_debug(" clock_curr=mclk=%d\n", ifparm.u.bt656.clock_curr); |
1363 | if (ifparm.u.bt656.clock_curr == 0) { | 1358 | if (ifparm.u.bt656.clock_curr == 0) |
1364 | csi_param.clk_mode = IPU_CSI_CLK_MODE_CCIR656_INTERLACED; | 1359 | csi_param.clk_mode = IPU_CSI_CLK_MODE_CCIR656_INTERLACED; |
1365 | /*protocol bt656 use 27Mhz pixel clock */ | 1360 | else |
1366 | csi_param.mclk = 27000000; | ||
1367 | } else { | ||
1368 | csi_param.clk_mode = IPU_CSI_CLK_MODE_GATED_CLK; | 1361 | csi_param.clk_mode = IPU_CSI_CLK_MODE_GATED_CLK; |
1369 | } | ||
1370 | 1362 | ||
1371 | csi_param.pixclk_pol = ifparm.u.bt656.latch_clk_inv; | 1363 | csi_param.pixclk_pol = ifparm.u.bt656.latch_clk_inv; |
1372 | 1364 | ||