
Steve Sakoman wrote:
On Fri, Sep 25, 2009 at 1:47 PM, Tom Tom.Rix@windriver.com wrote:
Dirk Behme wrote:
From: Steve Sakoman sakoman@gmail.com
Update default environment to support new kernel DSS2 subsystem and simplify rootfs type and location changes.
Signed-off-by: Steve Sakoman sakoman@gmail.com Signed-off-by: Dirk Behme dirk.behme@googlemail.com
include/configs/omap3_beagle.h | 27 +++++++++++++++++++-------- include/configs/omap3_overo.h | 27 +++++++++++++++++++-------- 2 files changed, 38 insertions(+), 16 deletions(-)
Index: u-boot-ti/include/configs/omap3_overo.h
--- u-boot-ti.orig/include/configs/omap3_overo.h +++ u-boot-ti/include/configs/omap3_overo.h @@ -155,16 +155,27 @@ #define CONFIG_EXTRA_ENV_SETTINGS \ "loadaddr=0x82000000\0" \ "console=ttyS2,115200n8\0" \
"videomode=1024x768@60,vxres=1024,vyres=768\0" \
"videospec=omapfb:vram:2M,vram:4M\0" \
"vram=12M\0" \
"dvimode=1024x768MR-16@60\0" \
"defaultdisplay=dvi\0" \
"mmcroot=/dev/mmcblk0p2 rw\0" \
"mmcrootfstype=ext3 rootwait\0" \
"nandroot=/dev/mtdblock4 rw\0" \
"nandrootfstype=jffs2\0" \ "mmcargs=setenv bootargs console=${console} " \
"video=${videospec},mode:${videomode} " \
"root=/dev/mmcblk0p2 rw " \
"rootfstype=ext3 rootwait\0" \
"vram=${vram} " \
"omapfb.mode=dvi:${dvimode} " \
"omapfb.debug=y " \
Is setting the debug option needed ? This would seem useful (from the name) only to developers
DSS2 is still under active development. This setting makes user support easier since boot logs contain needed debug info.
Or it turns them all into beta-testers. It is a toss up if this is a good thing I will go with it. OK.
"omapdss.def_disp=${defaultdisplay} " \
"root=${mmcroot} " \
"rootfstype=${mmcrootfstype}\0" \ "nandargs=setenv bootargs console=${console} " \
"video=${videospec},mode:${videomode} " \
"root=/dev/mtdblock4 rw " \
"rootfstype=jffs2\0" \
"vram=${vram} " \
"omapfb.mode=dvi:${dvimode} " \
"omapfb.debug=y " \
"omapdss.def_disp=${defaultdisplay} " \
From vram to omapdss.def_disp are common for nand and mmc. These should be changed to something like
"videoargs= .... "
Similar for beagle.
I'm not sure exactly what you are suggesting. The current setup is tailored to make user support easier.
If someone wants to use the 4.3" LCD display option, you tell them to simply type:
setenv defaultdisplay=lcd43 saveenv
Yes I see how just a single video_args= would not give you this flexiblity. A lesser way to do this would be to that you may want to do is video_args="${defaultdisplay} ${dvimode} .. "
Tom