[U-Boot] [PATCH] MX5: Make IPU display output and pixel format configurable

[Marek Vasut]

On Tuesday, September 27, 2011 02:22:18 PM Fabio Estevam wrote:
> Hi Marek,
> 
> On Tue, Sep 27, 2011 at 9:09 AM, Marek Vasut <marek.vasut at gmail.com> wrote:
> > Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
> > Cc: Stefano Babic <sbabic at 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
> 
> What about putting a define for selecting between IPU DI0 or DI1 in
> the board config file?
> 
> There are boards that have DI0 connected as the primary display,
> others may have DI1.
> 
> This way the mxc_ipuv3_fb.c driver could handle both cases depending
> on the board config.

Sure, this is a default value for boards which just use the default (most common 
case). You can define CONFIG... in your board config file and change this.

> 
> Regards,
> 
> Fabio Estevam