* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
@ 2009-09-25 18:49 Dirk Behme
2009-09-25 20:47 ` Tom
0 siblings, 1 reply; 10+ messages in thread
From: Dirk Behme @ 2009-09-25 18:49 UTC (permalink / raw)
To: u-boot
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 at 60,vxres=1024,vyres=768\0" \
- "videospec=omapfb:vram:2M,vram:4M\0" \
+ "vram=12M\0" \
+ "dvimode=1024x768MR-16 at 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 " \
+ "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} " \
+ "root=${nandroot} " \
+ "rootfstype=${nandrootfstype}\0" \
"loadbootscript=fatload mmc 0 ${loadaddr} boot.scr\0" \
"bootscript=echo Running bootscript from mmc ...; " \
"source ${loadaddr}\0" \
Index: u-boot-ti/include/configs/omap3_beagle.h
===================================================================
--- u-boot-ti.orig/include/configs/omap3_beagle.h
+++ u-boot-ti/include/configs/omap3_beagle.h
@@ -164,16 +164,27 @@
#define CONFIG_EXTRA_ENV_SETTINGS \
"loadaddr=0x82000000\0" \
"console=ttyS2,115200n8\0" \
- "videomode=1024x768 at 60,vxres=1024,vyres=768\0" \
- "videospec=omapfb:vram:2M,vram:4M\0" \
+ "vram=12M\0" \
+ "dvimode=1024x768MR-16 at 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 " \
+ "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} " \
+ "root=${nandroot} " \
+ "rootfstype=${nandrootfstype}\0" \
"loadbootscript=fatload mmc 0 ${loadaddr} boot.scr\0" \
"bootscript=echo Running bootscript from mmc ...; " \
"source ${loadaddr}\0" \
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-09-25 18:49 [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment Dirk Behme
@ 2009-09-25 20:47 ` Tom
2009-09-26 5:20 ` Dirk Behme
2009-09-26 13:41 ` Steve Sakoman
0 siblings, 2 replies; 10+ messages in thread
From: Tom @ 2009-09-25 20:47 UTC (permalink / raw)
To: u-boot
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 at 60,vxres=1024,vyres=768\0" \
> - "videospec=omapfb:vram:2M,vram:4M\0" \
> + "vram=12M\0" \
> + "dvimode=1024x768MR-16 at 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
> + "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.
Tom
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-09-25 20:47 ` Tom
@ 2009-09-26 5:20 ` Dirk Behme
2009-09-26 13:41 ` Steve Sakoman
1 sibling, 0 replies; 10+ messages in thread
From: Dirk Behme @ 2009-09-26 5:20 UTC (permalink / raw)
To: u-boot
Steve,
Tom 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 at 60,vxres=1024,vyres=768\0" \
>> - "videospec=omapfb:vram:2M,vram:4M\0" \
>> + "vram=12M\0" \
>> + "dvimode=1024x768MR-16 at 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
>
>> + "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.
Any opinion on this and above (omapfb.debug)?
Regards
Dirk
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-09-25 20:47 ` Tom
2009-09-26 5:20 ` Dirk Behme
@ 2009-09-26 13:41 ` Steve Sakoman
2009-09-27 11:52 ` Tom
1 sibling, 1 reply; 10+ messages in thread
From: Steve Sakoman @ 2009-09-26 13:41 UTC (permalink / raw)
To: u-boot
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 at 60,vxres=1024,vyres=768\0" \
>> - ? ? "videospec=omapfb:vram:2M,vram:4M\0" \
>> + ? ? "vram=12M\0" \
>> + ? ? "dvimode=1024x768MR-16 at 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.
>
>> + ? ? ? ? ? ? "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
Or if they want to change the DVI resolution to 640 x 480:
dvimode=640x480MR-16 at 60
saveenv
Perhaps I'm jut not understanding your suggestion. Could you give more detail?
Steve
>
> Tom
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-09-26 13:41 ` Steve Sakoman
@ 2009-09-27 11:52 ` Tom
2009-09-27 14:02 ` Steve Sakoman
0 siblings, 1 reply; 10+ messages in thread
From: Tom @ 2009-09-27 11:52 UTC (permalink / raw)
To: u-boot
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 at 60,vxres=1024,vyres=768\0" \
>>> - "videospec=omapfb:vram:2M,vram:4M\0" \
>>> + "vram=12M\0" \
>>> + "dvimode=1024x768MR-16 at 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
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-09-27 11:52 ` Tom
@ 2009-09-27 14:02 ` Steve Sakoman
2009-09-27 14:21 ` Tom
0 siblings, 1 reply; 10+ messages in thread
From: Steve Sakoman @ 2009-09-27 14:02 UTC (permalink / raw)
To: u-boot
On Sun, Sep 27, 2009 at 4:52 AM, Tom <Tom.Rix@windriver.com> wrote:
> 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 at 60,vxres=1024,vyres=768\0" \
>>>> - ? ? "videospec=omapfb:vram:2M,vram:4M\0" \
>>>> + ? ? "vram=12M\0" \
>>>> + ? ? "dvimode=1024x768MR-16 at 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.
Heh, like it or not we are all beta testing DSS2! It is so much
better than the old DSS code that I want to do everything possible to
help Tomi get it accepted upstream as quickly as possible :-)
>>>> + ? ? ? ? ? ? "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} ?.. ?"
I agree that would be nice, and in fact that is the way I did it
initially. But for some reason it didn't work. I didn't spend a lot
of time investigating, but it seemed that there was a limit to how
many "layers" of variable definitions you could have. I won't rule
out operator error though!
Steve
>
> Tom
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-09-27 14:02 ` Steve Sakoman
@ 2009-09-27 14:21 ` Tom
2009-10-10 18:03 ` Steve Sakoman
0 siblings, 1 reply; 10+ messages in thread
From: Tom @ 2009-09-27 14:21 UTC (permalink / raw)
To: u-boot
Steve Sakoman wrote:
> On Sun, Sep 27, 2009 at 4:52 AM, Tom <Tom.Rix@windriver.com> wrote:
>> 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 at 60,vxres=1024,vyres=768\0" \
>>>>> - "videospec=omapfb:vram:2M,vram:4M\0" \
>>>>> + "vram=12M\0" \
>>>>> + "dvimode=1024x768MR-16 at 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.
>
> Heh, like it or not we are all beta testing DSS2! It is so much
> better than the old DSS code that I want to do everything possible to
> help Tomi get it accepted upstream as quickly as possible :-)
>
>>>>> + "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} .. "
>
> I agree that would be nice, and in fact that is the way I did it
> initially. But for some reason it didn't work. I didn't spend a lot
> of time investigating, but it seemed that there was a limit to how
> many "layers" of variable definitions you could have. I won't rule
> out operator error though!
>
> Steve
>
Sounds like you have thought this stuff through.
The video_args is just a nice-to-have.
Ack
Tom
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-09-27 14:21 ` Tom
@ 2009-10-10 18:03 ` Steve Sakoman
2009-10-10 18:13 ` Dirk Behme
0 siblings, 1 reply; 10+ messages in thread
From: Steve Sakoman @ 2009-10-10 18:03 UTC (permalink / raw)
To: u-boot
On Sun, Sep 27, 2009 at 7:21 AM, Tom <Tom.Rix@windriver.com> wrote:
> Steve Sakoman wrote:
>>
>> On Sun, Sep 27, 2009 at 4:52 AM, Tom <Tom.Rix@windriver.com> wrote:
>>>
>>> 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 at 60,vxres=1024,vyres=768\0" \
>>>>>> - ? ? "videospec=omapfb:vram:2M,vram:4M\0" \
>>>>>> + ? ? "vram=12M\0" \
>>>>>> + ? ? "dvimode=1024x768MR-16 at 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.
>>
>> Heh, like it or not we are all beta testing DSS2! ?It is so much
>> better than the old DSS code that I want to do everything possible to
>> help Tomi get it accepted upstream as quickly as possible :-)
>>
>>>>>> + ? ? ? ? ? ? "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} ?.. ?"
>>
>> I agree that would be nice, and in fact that is the way I did it
>> initially. ?But for some reason it didn't work. ?I didn't spend a lot
>> of time investigating, but it seemed that there was a limit to how
>> many "layers" of variable definitions you could have. ?I won't rule
>> out operator error though!
>>
>> Steve
>>
>
> Sounds like you have thought this stuff through.
> The video_args is just a nice-to-have.
>
> Ack
I'm a bit confused!
This patch made it into the u-boot-ti tree and then disappeared during
one of the rebasings.
Does this mean it is rejected, or perhaps just lost in the shuffle?
Steve
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-10-10 18:03 ` Steve Sakoman
@ 2009-10-10 18:13 ` Dirk Behme
2009-10-10 18:32 ` Paulraj, Sandeep
0 siblings, 1 reply; 10+ messages in thread
From: Dirk Behme @ 2009-10-10 18:13 UTC (permalink / raw)
To: u-boot
Steve Sakoman wrote:
> On Sun, Sep 27, 2009 at 7:21 AM, Tom <Tom.Rix@windriver.com> wrote:
>> Steve Sakoman wrote:
>>> On Sun, Sep 27, 2009 at 4:52 AM, Tom <Tom.Rix@windriver.com> wrote:
>>>> 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 at 60,vxres=1024,vyres=768\0" \
>>>>>>> - "videospec=omapfb:vram:2M,vram:4M\0" \
>>>>>>> + "vram=12M\0" \
>>>>>>> + "dvimode=1024x768MR-16 at 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.
>>> Heh, like it or not we are all beta testing DSS2! It is so much
>>> better than the old DSS code that I want to do everything possible to
>>> help Tomi get it accepted upstream as quickly as possible :-)
>>>
>>>>>>> + "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} .. "
>>> I agree that would be nice, and in fact that is the way I did it
>>> initially. But for some reason it didn't work. I didn't spend a lot
>>> of time investigating, but it seemed that there was a limit to how
>>> many "layers" of variable definitions you could have. I won't rule
>>> out operator error though!
>>>
>>> Steve
>>>
>> Sounds like you have thought this stuff through.
>> The video_args is just a nice-to-have.
>>
>> Ack
>
> I'm a bit confused!
>
> This patch made it into the u-boot-ti tree and then disappeared during
> one of the rebasings.
>
> Does this mean it is rejected,
I don't think so.
> or perhaps just lost in the shuffle?
Hopefully yes.
Sandeep: Could you check what happened with
http://lists.denx.de/pipermail/u-boot/2009-September/061564.html
?
Thanks and best regards
Dirk
^ permalink raw reply [flat|nested] 10+ messages in thread* [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
2009-10-10 18:13 ` Dirk Behme
@ 2009-10-10 18:32 ` Paulraj, Sandeep
0 siblings, 0 replies; 10+ messages in thread
From: Paulraj, Sandeep @ 2009-10-10 18:32 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Dirk Behme [mailto:dirk.behme at googlemail.com]
> Sent: Saturday, October 10, 2009 2:13 PM
> To: Paulraj, Sandeep
> Cc: Steve Sakoman; u-boot at lists.denx.de; Tom
> Subject: Re: [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment
>
> Steve Sakoman wrote:
> > On Sun, Sep 27, 2009 at 7:21 AM, Tom <Tom.Rix@windriver.com> wrote:
> >> Steve Sakoman wrote:
> >>> On Sun, Sep 27, 2009 at 4:52 AM, Tom <Tom.Rix@windriver.com> wrote:
> >>>> 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 at 60,vxres=1024,vyres=768\0" \
> >>>>>>> - "videospec=omapfb:vram:2M,vram:4M\0" \
> >>>>>>> + "vram=12M\0" \
> >>>>>>> + "dvimode=1024x768MR-16 at 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.
> >>> Heh, like it or not we are all beta testing DSS2! It is so much
> >>> better than the old DSS code that I want to do everything possible to
> >>> help Tomi get it accepted upstream as quickly as possible :-)
> >>>
> >>>>>>> + "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} .. "
> >>> I agree that would be nice, and in fact that is the way I did it
> >>> initially. But for some reason it didn't work. I didn't spend a lot
> >>> of time investigating, but it seemed that there was a limit to how
> >>> many "layers" of variable definitions you could have. I won't rule
> >>> out operator error though!
> >>>
> >>> Steve
> >>>
> >> Sounds like you have thought this stuff through.
> >> The video_args is just a nice-to-have.
> >>
> >> Ack
> >
> > I'm a bit confused!
> >
> > This patch made it into the u-boot-ti tree and then disappeared during
> > one of the rebasings.
> >
> > Does this mean it is rejected,
>
> I don't think so.
>
> > or perhaps just lost in the shuffle?
>
> Hopefully yes.
>
> Sandeep: Could you check what happened with
>
> http://lists.denx.de/pipermail/u-boot/2009-September/061564.html
>
> ?
There were a few e-mails exchanges on this but this finally culminated in ACK from TOM which I missed and thus I failed to apply.
>
> Thanks and best regards
>
> Dirk
>
>
Pushed to u-boot-ti next
http://git.denx.de/?p=u-boot/u-boot-ti.git;a=commit;h=83fb7d7bb237cc7adae37e0e3328c203fd1b9ad1
Thanks,
Sandeep
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2009-10-10 18:32 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-09-25 18:49 [U-Boot] [PATCH] OMAP3: Update Overo and Beagle environment Dirk Behme
2009-09-25 20:47 ` Tom
2009-09-26 5:20 ` Dirk Behme
2009-09-26 13:41 ` Steve Sakoman
2009-09-27 11:52 ` Tom
2009-09-27 14:02 ` Steve Sakoman
2009-09-27 14:21 ` Tom
2009-10-10 18:03 ` Steve Sakoman
2009-10-10 18:13 ` Dirk Behme
2009-10-10 18:32 ` Paulraj, Sandeep
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox