On Mon, Jun 27, 2016 at 04:23:01PM +0900, Masahiro Yamada wrote:

> As the help message of CONFIG_BOOTDELAY says, CONFIG_BOOTDELAY=-2
> means the autoboot with no delay, with no abort check even if
> CONFIG_ZERO_BOOTDELAY_CHECK is defined.
> 
> To sum up, the autoboot behaves as follows:
> 
>  [1] CONFIG_BOOTDELAY=0 && CONFIG_ZERO_BOOTDELAY_CHECK=y
>     autoboot with no delay, but you can abort it by key input
> 
>  [2] CONFIG_BOOTDELAY=0 && CONFIG_ZERO_BOOTDELAY_CHECK=n
>     autoboot with no delay, with no check for abort
> 
>  [3] CONFIG_BOOTDELAY=-1
>     disable autoboot
> 
>  [4] CONFIG_BOOTDELAY=-2
>     autoboot with no delay, with no check for abort
> 
> As you notice, [2] and [4] come to the same result, which means we
> do not need CONFIG_ZERO_BOOTDELAY_CHECK.  We can control all the
> cases only by CONFIG_BOOTDELAY, like this:
> 
>  [1] CONFIG_BOOTDELAY=0
>     autoboot with no delay, but you can abort it by key input
> 
>  [2] CONFIG_BOOTDELAY=-1
>     disable autoboot
> 
>  [3] CONFIG_BOOTDELAY=-2
>     autoboot with no delay, with no check for abort
> 
> This commit converts the logic as follow:
>   CONFIG_BOOTDELAY=0 && CONFIG_ZERO_BOOTDELAY_CHECK=n
>     --> CONFIG_BOOTDELAY=-2
> 
> Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com>
> Reviewed-by: Stefan Roese <s...@denx.de>
> Acked-by: Igor Grinberg <grinb...@compulab.co.il>
> Reviewed-by: Simon Glass <s...@chromium.org>
> Acked-by: Vladimir Zapolskiy <v...@mleia.com>
> Reviewed-by: Heiko Schocher <h...@denx.de>
> Acked-by: Christian Riesch <christian.rie...@omicronenergy.com>
> Acked-by: Hannes Schmelzer <hannes.schmel...@br-automation.com>

Applied to u-boot/master, thanks!

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to