Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-03-04 Thread Joe Hershberger
Hi Tom, On Fri, Mar 1, 2013 at 2:28 PM, Tom Rini tr...@ti.com wrote: On Wed, Feb 27, 2013 at 02:11:31PM -0600, Joe Hershberger wrote: Hi Tom, On Mon, Feb 18, 2013 at 11:20 AM, Tom Rini tr...@ti.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/09/2013 11:21 AM, Otavio

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-03-01 Thread Tom Rini
On Wed, Feb 27, 2013 at 02:11:31PM -0600, Joe Hershberger wrote: Hi Tom, On Mon, Feb 18, 2013 at 11:20 AM, Tom Rini tr...@ti.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/09/2013 11:21 AM, Otavio Salvador wrote: Hello Wolfgang, On Sat, Feb 9, 2013 at 4:54 AM,

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-27 Thread Joe Hershberger
Hi Tom, On Mon, Feb 18, 2013 at 11:20 AM, Tom Rini tr...@ti.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/09/2013 11:21 AM, Otavio Salvador wrote: Hello Wolfgang, On Sat, Feb 9, 2013 at 4:54 AM, Wolfgang Denk w...@denx.de wrote: Dear Joe Hershberger, In message

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-18 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/09/2013 11:21 AM, Otavio Salvador wrote: Hello Wolfgang, On Sat, Feb 9, 2013 at 4:54 AM, Wolfgang Denk w...@denx.de wrote: Dear Joe Hershberger, In message 1360355280-1197-1-git-send-email-joe.hershber...@ni.com you wrote: Because the

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-17 Thread Wolfgang Denk
Dear Otavio Salvador, In message cap9odkqs5fsw+nvqznvm6+4vvk6jfc1vd6ezqfumdkvxzcq...@mail.gmail.com you wrote: Careful!! This is probably changing behaviour of a number of boards significantly. we have to check if we really want this, and if yes, we have to announce it and provide a

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-17 Thread Otavio Salvador
On Sun, Feb 17, 2013 at 5:18 PM, Wolfgang Denk w...@denx.de wrote: Dear Otavio Salvador, In message cap9odkqs5fsw+nvqznvm6+4vvk6jfc1vd6ezqfumdkvxzcq...@mail.gmail.com you wrote: Careful!! This is probably changing behaviour of a number of boards significantly. we have to check if

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-17 Thread Wolfgang Denk
Dear Otavio Salvador, In message cap9odkr3xau_vg7ygjyosntd03yvlaygrdszc-y4jqgdi+d...@mail.gmail.com you wrote: include/configs/RPXsuper.h:#define CONFIG_BOOTDELAY-1 include/configs/ep8260.h:#define CONFIG_BOOTDELAY-1 include/configs/espt.h:#define CONFIG_BOOTDELAY

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-17 Thread Otavio Salvador
On Sun, Feb 17, 2013 at 6:02 PM, Wolfgang Denk w...@denx.de wrote: Dear Otavio Salvador, In message cap9odkr3xau_vg7ygjyosntd03yvlaygrdszc-y4jqgdi+d...@mail.gmail.com you wrote: include/configs/RPXsuper.h:#define CONFIG_BOOTDELAY-1 include/configs/ep8260.h:#define

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-09 Thread Otavio Salvador
Hello Wolfgang, On Sat, Feb 9, 2013 at 4:54 AM, Wolfgang Denk w...@denx.de wrote: Dear Joe Hershberger, In message 1360355280-1197-1-git-send-email-joe.hershber...@ni.com you wrote: Because the code that handles bootdelay is compiled in conditionally based on the default value, you are

[U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-08 Thread Joe Hershberger
Because the code that handles bootdelay is compiled in conditionally based on the default value, you are restricted in the default, regardless of what you want the runtime options to be. Change the source to always check if any default is given so that other values can be selected and used at

Re: [U-Boot] [PATCH] CONFIG_BOOTDELAY default should not affect runtime

2013-02-08 Thread Wolfgang Denk
Dear Joe Hershberger, In message 1360355280-1197-1-git-send-email-joe.hershber...@ni.com you wrote: Because the code that handles bootdelay is compiled in conditionally based on the default value, you are restricted in the default, regardless of what you want the runtime options to be.