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

Wolfgang Denk wd at denx.de
Sat Feb 9 07:54:20 CET 2013


Dear Joe Hershberger,

In message <1360355280-1197-1-git-send-email-joe.hershberger at 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.
> 
> Change the source to always check if any default is given so that other
> values can be selected and used at runtime.
> 
> Signed-off-by: Joe Hershberger <joe.hershberger at ni.com>
> ---
>  common/main.c | 14 ++++++--------
>  1 file changed, 6 insertions(+), 8 deletions(-)
> 
> diff --git a/common/main.c b/common/main.c
> index e2d2e09..0973c59 100644
> --- a/common/main.c
> +++ b/common/main.c
> @@ -95,7 +95,7 @@ extern void mdm_init(void); /* defined in board.c */
>   * Watch for 'delay' seconds for autoboot stop or autoboot delay string.
>   * returns: 0 -  no key string, allow autoboot 1 - got key string, abort
>   */
> -#if defined(CONFIG_BOOTDELAY) && (CONFIG_BOOTDELAY >= 0)
> +#if defined(CONFIG_BOOTDELAY)

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 grace period (eventually using
doc/feature-removal-schedule.txt ?)

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
                  Nail here --X-- for new monitor.


More information about the U-Boot mailing list