From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jeroen Hofstee <jeroen@myspectrum.nl>
Date: Wed, 22 Oct 2014 10:47:49 +0200
Subject: [U-Boot] [PATCH] mx6sabresd: Add Seiko WVGA panel support
In-Reply-To: <1413933293-23139-1-git-send-email-festevam@gmail.com>
References: <1413933293-23139-1-git-send-email-festevam@gmail.com>
Message-ID: <54476F35.70406@myspectrum.nl>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Fabio,

On 22-10-14 01:14, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Add support for the 4.3'' Seiko WVGA parallel display.
>
> In order to direct the splash screen to the Seiko display:
>
> => setenv panel SEIKO-WVGA
> => save
> => reset
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
[snip]
> +} }, {
> +	.bus	= 0,
> +	.addr	= 0,
> +	.pixfmt	= IPU_PIX_FMT_RGB24,
> +	.detect	= NULL,
> +	.enable	= enable_rgb,
> +	.mode	= {
> +		.name           = "SEIKO-WVGA",
> +		.refresh        = 60,
> +		.xres           = 800,
> +		.yres           = 480,
> +		.pixclock       = 29850,
> +		.left_margin    = 89,
> +		.right_margin   = 164,
> +		.upper_margin   = 23,
> +		.lower_margin   = 10,
> +		.hsync_len      = 10,
> +		.vsync_len      = 10,
> +		.sync           = 0,
> +		.vmode          = FB_VMODE_NONINTERLACED
>   } } };
>   size_t display_count = ARRAY_SIZE(displays);
>   

If [1] is a datasheet for this lcd, you likely want to add

#include <../drivers/video/mxcfb.h>
  .sync           = FB_SYNC_CLK_LAT_FALL,

Or something similar, since the data is sampled on the falling edge
of the pixel clock / the pixel clock is inverted.

Regards,
Jeroen

[1] http://www.glyn.de/data/glyn/media/doc/43wvf1g-0.pdf