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

Wolfgang Denk wd at denx.de
Sun Feb 17 22:02:56 CET 2013


Dear Otavio Salvador,

In message <CAP9ODKr3xaU_Vg7YgjYOsntD03YvLaygRdSZC-Y4jQgDi+D7sw at 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        -1
> >> include/configs/scb9328.h:#define CONFIG_BOOTDELAY   -1
> >> include/configs/sh7763rdp.h:#define CONFIG_BOOTDELAY        -1
> >>
> >> So maybe those could have CONFIG_BOOTDELAY undefined keeping them
> >> working as before?
> >
> > Yes, they could.  But it is a change of a long-standing behaviour, and
> > we try to avoid breaking existing boards.  At minimum this needs to be
> > announced, the respective board maintainers need to be modified, and
> > given sufficient time to react.
> 
> Yes; I agree but breaking out of tree boards ... well, it is the price
> of not working upstream so I think it cannot be a blocker for U-Boot
> improvements and cleanups.

You also change behaviour (and thus potentially break) at least 5
boards in mainline.  See also [1] about breaking user space in Linux
(which is pretty much equivalent to what would happen here).

[1] http://article.gmane.org/gmane.linux.kernel/1414106

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
"'Tis true, 'tis pity, and pity 'tis 'tis true."
    - Poloniouius, in Willie the Shake's _Hamlet, Prince of Darkness_


More information about the U-Boot mailing list