[U-Boot] [PATCH] [v2] powerpc: 'monitor' environment variable contains full video configuration

Wolfgang Denk
2011-03-14T23:33:07Z

Dear Timur Tabi,

In message <1300133949-1115-1-git-send-email-timur@freescale.com> you wrote:
> Update the "monitor" environment variable (for Freescale chips that have a
> DIU display controller) to designate the full video configuration, instead
> of just the output monitor.
>
> The old definition of the "monitor" environment variable only determines
> which video port to use for output. This variable is set to a number (0,
> 1, or sometimes 2) to specify a DVI, LVDS, or Dual-LVDS port. The
> resolution is hard-coded into board-specific code. The Linux command-line
> arguments need to be hard-coded to the proper video definition string.
>
> Instead, the "monitor" variable is now set to a string that indicates not
> only the part name, but also the resolution, color depth, and refresh rate.
> This string is then parsed for all needed information, and can be passed to
> the kernel via the 'diubootargs' environment variable.

Looks as if this was a Freescale specific implementation?

Can this not be turned into generic code, usable for other boards /
systems as well?

Please see the discussion we had about video modes for the i.MX
systems.

I want to see handled most of this with generic code, not with
processor / architecture specific one.

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: wd at denx.de
Even historians fail to learn from history -- they repeat the same
mistakes.
-- John Gill, "Patterns of Force", stardate 2534.7