From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Date: Tue, 15 Mar 2011 09:50:33 -0500 Subject: [U-Boot] [PATCH] [v2] powerpc: 'monitor' environment variable contains full video configuration In-Reply-To: <20110315090546.E34491518DB7@gemini.denx.de> References: <1300133949-1115-1-git-send-email-timur@freescale.com> <20110314233307.4787D1518DB6@gemini.denx.de> <4D7EA72C.9040603@freescale.com> <20110315090546.E34491518DB7@gemini.denx.de> Message-ID: <4D7F7CB9.8090005@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Wolfgang Denk wrote: > Agreed, but please not by introducing lots of new, probably later > incompatible code. Fair enough. >> > Can you point me to the thread or at least tell me the subject line? I have >> > no idea what you're talking about. > See http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/88169/focus=88240 Ok, so what I get from that thread is to change the name of the variables ("monitor" -> "video" and "diubootargs" -> something else more generic) and update the videomodes.c code to parse the same video string that the kernel uses. I also gathered that no one has chosen to update videomodes.c, so I would have to do it. Do I have that right? > OK, so please split that patch: one patch should do this cleanup, and > another one should contain the rest. I can't make any promises, since a lot of the code is intermingled. > Instead of a non-standard and undocumented 'diubootargs' environment > variable please use something (probably called "video-mode" :-) that > can be passed as "video-mode=" boot argument to Linux. > > At this point I wonder which use ther eis left for your "monitor" > variable - it should be completely redundant now? Well, it may still be necessary to have two variables. One for the mode that U-Boot parses, and one for the string that is passed to the kernel. This is because the kernel also needs to see "console=tty0" on the command line only if video is supposed to be enabled. I want to make video mode completely dynamic, so that if the 'video-mode' variable is set, then the console is switched to the video device, and the kernel is told to do the same. Otherwise, all output will go to the serial port. -- Timur Tabi Linux kernel developer at Freescale -- Timur Tabi Linux kernel developer at Freescale