[U-Boot] [PATCH] mx53loco: Add support for SEIKO 4.3'' WVGA panel

Stefano Babic
2012-10-25T10:28:31Z

Am 23/10/2012 19:36, schrieb Fabio Estevam:
> Hi Stefano,
>
> On Sat, Oct 20, 2012 at 12:03 PM, Stefano Babic <sbabic@denx.de> wrote:
>
>> What about to use a u-boot environment to select the display ? Could be
>> a better solution for you ? You can then have a single u-boot image
>> managing both displays. I did in this way for the mt_ventoux board
>> (board/mt_ventoux/mt_ventoux.c), maybe can this help ?
>
> Sounds good. I tried this approach.
>
> However, when I try to manipulate the env var I am not able to boot:
>
> U-Boot 2012.10-09480-g6b08fc3-dirty (Oct 23 2012 - 15:24:03)
>
> Board: MX53 LOCO
> I2C: ready
> DRAM: 1 GiB
> ...
>
> These are my simple changes (just to show the issue I am facing):
>
> --- a/board/freescale/mx53loco/mx53loco.c
> +++ b/board/freescale/mx53loco/mx53loco.c
> @@ -471,6 +471,18 @@ void lcd_iomux(void)
> void lcd_enable(void)
> {
> int ret = ipuv3_fb_init(&claa_wvga, 0, IPU_PIX_FMT_RGB565);
> + char *e;
> +
> + e = getenv("panel");
> +
> + if (e != NULL) {
> + if (strcmp(e, "claa") == 0)
> + printf("Panel is claa\n");
> +
> + if (strcmp(e, "seiko") == 0)
> + printf("Panel is seiko\n");
> + }
> +
> if (ret)
> printf("LCD cannot be configured: %d\n", ret);
> }
>
>
> Any ideas?

Yes, I thins is due to the fact that size for the framebuffer is
allocated before relocation. This was also a reason for me to migrate to
CONFIG_VIDEO instead of CONFIG_LCD.