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

Otavio Salvador otavio at ossystems.com.br
Sun Feb 17 22:16:57 CET 2013


On Sun, Feb 17, 2013 at 6:02 PM, Wolfgang Denk <wd at denx.de> wrote:
> 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

I see it different; I understand we cannot break current boards so a
patch should also convert the boards to the new behavior.

I also don't see a compare it to user space in Linux fair, I think
it'd be more like a driver API change in Linux and Linux breaks API
for out of tree drivers very ofthen so I think same fits here.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br


More information about the U-Boot mailing list