From 7527582f219316119a75c69a2e71ebc8f93edf1c Mon Sep 17 00:00:00 2001 From: Sandor Yu Date: Wed, 22 Aug 2012 10:57:50 +0800 Subject: [PATCH] ENGR00220538 HDMI: Clock mismatch in suspend&resume when video playback In suspend/resume and HDMI plugin/plugout stress test, sometimes fbcon will call fb_set_par with parameter fb_var_screeninfo that xres anfd yres is zero. MX frame buffer driver can not correct handle this casue, it will cause IPU pixel clock gating/ungating mismatch. Check fb_var_screeninfo parameter in mxcfb_check_var and mxcfb_set_par function, returned if xres,yres zero. Signed-off-by: Sandor Yu --- drivers/video/mxc/mxc_ipuv3_fb.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/video/mxc/mxc_ipuv3_fb.c b/drivers/video/mxc/mxc_ipuv3_fb.c index 5e074e712548..745c485016d0 100644 --- a/drivers/video/mxc/mxc_ipuv3_fb.c +++ b/drivers/video/mxc/mxc_ipuv3_fb.c @@ -361,6 +361,9 @@ static int mxcfb_set_par(struct fb_info *fbi) dev_dbg(fbi->device, "Reconfiguring framebuffer\n"); + if (fbi->var.xres == 0 || fbi->var.yres == 0) + return 0; + if (ovfbi_enable) { ov_pos_ret = ipu_disp_get_window_pos( mxc_fbi_fg->ipu, mxc_fbi_fg->ipu_ch, @@ -709,6 +712,10 @@ static int mxcfb_check_var(struct fb_var_screeninfo *var, struct fb_info *info) u32 htotal; struct mxcfb_info *mxc_fbi = (struct mxcfb_info *)info->par; + + if (var->xres == 0 || var->yres == 0) + return 0; + /* fg should not bigger than bg */ if (mxc_fbi->ipu_ch == MEM_FG_SYNC) { struct fb_info *fbi_tmp; -- 2.39.5