Hi Igor, On 8 October 2015 at 20:10, Igor Grinberg <grinb...@compulab.co.il> wrote: > Add CONFIG_BOOTDELAY to the Kconfig. > Default it to 3 seconds according to the majority of configs. > > Signed-off-by: Igor Grinberg <grinb...@compulab.co.il> > Cc: Simon Glass <s...@chromium.org> > Cc: Masahiro Yamada <yamada.masah...@socionext.com> > --- > common/Kconfig | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/common/Kconfig b/common/Kconfig > index d98eb19..e13d255 100644 > --- a/common/Kconfig > +++ b/common/Kconfig > @@ -100,6 +100,16 @@ config AUTOBOOT_STOP_STR_SHA256 > string / password matches a values that is encypted via > a SHA256 hash and saved in the environment. > > +config BOOTDELAY > + int "Seconds to delay before autobooting" > + default 3 > + help > + Delay before automatically booting the default image; > + set to -1 to disable autoboot. > + set to -2 to autoboot with no delay and not check for abort > + (even when CONFIG_ZERO_BOOTDELAY_CHECK is defined). > + See doc/README.autoboot for additional information. > + > endmenu > > comment "Commands" > -- > 2.4.9 >
It's great that you are taking on these difficult ones. IMO we should split this config into a few parts: - enabling the boot-delay feature - setting the boot-delay time - whether to allow abort - whether to allow abort even when boot delay is 0 The way it is written -2 sounds like a weird case that would be better merged with the last one above. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
