[U-Boot] [PATCH 5/5] autoboot: add CONFIG_AUTOBOOT to allow to not compile autoboot.c

[Soeren Moch]

On 10.06.2016 17:20, Masahiro Yamada wrote:
> Another mistake of commit bb597c0eeb7e ("common: bootdelay: move
> CONFIG_BOOTDELAY into a Kconfig option") is that CONFIG_BOOTDELAY
> is now defined for all the boards.
> 
> Before that commit, it is allowed to unset CONFIG_BOOTDELAY to not
> compile common/autoboot.c
> 
> This is apparent from the code in common/Makefile:
> 
>   # This option is not just y/n - it can have a numeric value
>   ifdef CONFIG_BOOTDELAY
>   obj-y += autoboot.o
>   endif
> 
> (and from the #ifdef in the include/autoconf.h, too.)
> It was a bit odd to enable/disable code with an integer type option,
> but anyway it was how this option worked before that commit.
> Actually several boards unset it to opt out of the autoboot feature.
> 
> This commit adds a bool option, CONFIG_AUTOBOOT, and have the
> CONFIG_BOOTDELAY to depend on it.
> 
> I chose "default y" for this option because most of boards enable the
> autoboot.  I added "# CONFIG_AUTOBOOT is not set" for the boards that
> did not set CONFIG_BOOTDELAY prior to the bad commit.
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---

[...]

> diff --git a/configs/tbs2910_defconfig b/configs/tbs2910_defconfig
> index dc69b39..5d0de21 100644
> --- a/configs/tbs2910_defconfig
> +++ b/configs/tbs2910_defconfig
> @@ -4,6 +4,7 @@ CONFIG_TARGET_TBS2910=y
>  CONFIG_FIT=y
>  CONFIG_HUSH_PARSER=y
>  CONFIG_SYS_PROMPT="Matrix U-Boot> "
> +# CONFIG_AUTOBOOT is not set
>  CONFIG_CMD_BOOTZ=y
>  # CONFIG_CMD_IMLS is not set
>  CONFIG_CMD_MEMTEST=y

"+# CONFIG_AUTOBOOT is not set" seems to be wrong for tbs2910.
Before this move to Kconfig we had
"#define CONFIG_BOOTDELAY       3"
via include/configs/mx6_common.h for tbs2910 and for a lot of other imx6
boards.

So the right fix would be to readd 3 seconds bootdelay here.

Adding Stefano (for imx boards) to the cc list.

Regards,
Soeren