[PATCH] ARM: video: mxs: Fix mxsfb misconfiguring VDCTRL0
Shawn Guo
shawn.guo at linaro.org
Mon Mar 18 08:44:24 EDT 2013
On Sat, Mar 16, 2013 at 11:06:21PM +0100, Marek Vasut wrote:
> The issue fixed by this patch manifests only then using X11
> with mxsfb driver. The X11 will display either shifted image
> or otherwise distorted image on the LCD.
>
> The problem is that the X11 tries to reconfigure the framebuffer
> and along the way call fb_ops.fb_set_par() with it's configuration
> values. The field of particular interest is fb_info->var.sync which
> contains non-standard values if configured by kernel. These are
> FB_SYNC_DATA_ENABLE_HIGH_ACT and FB_SYNC_DOTCLK_FAILING_ACT defined
> in include/linux/mxsfb.h . The driver interprets those and configures
> the LCD controller accordingly. Yet X11 only has access to standard
> values for this field defined in include/uapi/linux/fb.h and thus
> omits these special values. This results in distorted image on the
> LCD.
>
> This patch moves these non-standard values into new field of the
> mxsfb_platform_data structure so the driver can in turn check this
> field instead of the video mode field for these specific portions.
>
> Signed-off-by: Marek Vasut <marex at denx.de>
> Cc: Fabio Estevam <fabio.estavem at freescale.com>
> Cc: Linux ARM <linux-arm-kernel at lists.infradead.org>
> Cc: Linux FBDEV <linux-fbdev at vger.kernel.org>
> Cc: Lothar Waßmann <LW at karo-electronics.de>
> Cc: Sascha Hauer <kernel at pengutronix.de>
> Cc: Shawn Guo <shawn.guo at linaro.org>
> ---
> arch/arm/mach-mxs/mach-mxs.c | 22 +++++++++++-----------
> drivers/video/mxsfb.c | 7 +++++--
> include/linux/mxsfb.h | 7 +++++--
> 3 files changed, 21 insertions(+), 15 deletions(-)
Since it touches mach-mxs as well, let me send it through arm-soc tree
as fix for 3.9.
Shawn
More information about the linux-arm-kernel
mailing list