Dear Timur Tabi,

In message <1297811359-28389-1-git-send-email-ti...@freescale.com> you wrote:
> Clean up the macro defintions used to enable DIU (video) support on the
> MPC8610HPCD and the MPC5121ADS so that they look more like the P1022DS,
> which is newer.  Add software cursor support to all three boards.

Software cursor support should be split off into a separate patch.
And CONFIG_VIDEO_SW_CURSOR needs to be documented.

> Also document the CONFIG_FSL_DIU_FB in the README.
...
> +             The DIU driver will look for the 'monitor' environment variable,
> +             and if defined, enable the DIU as a console during boot.  This
> +             variable should be set to one of these values:
> +
> +                     '0'     Output video to the DVI connector
> +                     '1'     Output video to the LVDS connector
> +                     '2'     Output video to the Dual-Link LVDS connector

Maybe it would be more user-friendly if we used easy to remember names
instead of some magic numbers, like "dvi", "lvds" resp. "dlvds" (or
whatever) ?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
I used to be indecisive, now I'm not sure.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to