diff options
author | Jason Chen <b02280@freescale.com> | 2011-11-15 17:57:46 +0800 |
---|---|---|
committer | Eric Miao <eric.miao@linaro.org> | 2011-12-19 11:25:22 +0800 |
commit | b57540ac92bc41a91cc7fff50bac94f7b636f547 (patch) | |
tree | 59f27f0d2744509f8d61b41ba56e3139abde5943 | |
parent | 48aee4660c0d7996fdc29da3178a0c085cca1a85 (diff) |
ENGR00162218 mxc_vout: put set position function before set var
fb set var will check the window position with window size, so the
correct way is setting position before fb_set_var.
Signed-off-by: Jason Chen <b02280@freescale.com>
-rw-r--r-- | drivers/media/video/mxc/output/mxc_vout.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/media/video/mxc/output/mxc_vout.c b/drivers/media/video/mxc/output/mxc_vout.c index b1730bb2c7b..2d110032032 100644 --- a/drivers/media/video/mxc/output/mxc_vout.c +++ b/drivers/media/video/mxc/output/mxc_vout.c @@ -1256,6 +1256,10 @@ static int config_disp_output(struct mxc_vout_output *vout) "set display fb to %d %d\n", var.xres, var.yres); + ret = set_window_position(vout); + if (ret < 0) + return ret; + /* Init display channel through fb API */ var.yoffset = 0; var.activate |= FB_ACTIVATE_FORCE; @@ -1271,10 +1275,6 @@ static int config_disp_output(struct mxc_vout_output *vout) for (i = 0; i < fb_num; i++) vout->disp_bufs[i] = fbi->fix.smem_start + i * display_buf_size; - ret = set_window_position(vout); - if (ret < 0) - return ret; - console_lock(); ret = fb_blank(fbi, FB_BLANK_UNBLANK); console_unlock(); |