On 09/27/2011 02:09 PM, Marek Vasut wrote:
> Signed-off-by: Marek Vasut <marek.va...@gmail.com>
> Cc: Stefano Babic <sba...@denx.de>
> ---
> drivers/video/mxc_ipuv3_fb.c | 14 ++++++++++++--
> 1 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/video/mxc_ipuv3_fb.c b/drivers/video/mxc_ipuv3_fb.c
> index a66981c..c4443e8 100644
> --- a/drivers/video/mxc_ipuv3_fb.c
> +++ b/drivers/video/mxc_ipuv3_fb.c
> @@ -42,6 +42,16 @@
>
> DECLARE_GLOBAL_DATA_PTR;
>
> +/* Use DISP #1 by default. */
> +#ifndef CONFIG_VIDEO_MX5_OUTPUT
> +#define CONFIG_VIDEO_MX5_OUTPUT 0
> +#endif
> +
> +/* Use RGB666 pixel format by default. */
> +#ifndef CONFIG_VIDEO_MX5_PIXFMT
> +#define CONFIG_VIDEO_MX5_PIXFMT IPU_PIX_FMT_RGB666
> +#endif
> +
> void *lcd_base; /* Start of framebuffer memory */
> void *lcd_console_address; /* Start of console buffer */
>
> @@ -550,7 +560,7 @@ static int mxcfb_probe(u32 interface_pix_fmt, struct
> fb_videomode *mode)
> mxcfbi->blank = FB_BLANK_POWERDOWN;
> }
>
> - mxcfbi->ipu_di = 0;
> + mxcfbi->ipu_di = CONFIG_VIDEO_MX5_OUTPUT;
>
> ipu_disp_set_global_alpha(mxcfbi->ipu_ch, 1, 0x80);
> ipu_disp_set_color_key(mxcfbi->ipu_ch, 0, 0);
> @@ -636,7 +646,7 @@ int mx51_fb_init(struct fb_videomode *mode)
> lcd_base += 56;
>
> debug("Framebuffer at 0x%x\n", (unsigned int)lcd_base);
> - ret = mxcfb_probe(IPU_PIX_FMT_RGB666, mode);
> + ret = mxcfb_probe(CONFIG_VIDEO_MX5_PIXFMT, mode);
>
What about to not use a CONFIG_ ? at the moment, the board code calls
mx51_fb_init() to initialize the display parameter. We can add a second
parameter (display_id) to the initialization and the board code is
responsible to do that. And it is not required to recompile u-boot if
the board as the mx51evk more as one display has or can have.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de
=====================================================================
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
