Re: [PATCH] common: Kconfig: Add dependency for default variables strings

On Wed, Aug 12, 2020 at 09:28:24AM +0200, Michal Simek wrote:

Kconfig provides several config options for setting up default variables but these are unused when variables are passed to U-Boot via file. That's why cover this dependency in Kconfig.

Signed-off-by: Michal Simek michal.simek@xilinx.com

There are likely other Kconfig entries which should be fixed too.

---

common/Kconfig | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/common/Kconfig b/common/Kconfig index 62d78c5bd739..b7d5a8f28cf8 100644 --- a/common/Kconfig +++ b/common/Kconfig @@ -355,7 +355,7 @@ endmenu config BOOTDELAY int "delay in seconds before automatically booting" default 2

• depends on AUTOBOOT

• depends on AUTOBOOT && !USE_DEFAULT_ENV_FILE help Delay before automatically running bootcmd; set to 0 to autoboot with no delay, but you can stop it by key input.

The rest are fine but BOOTDELAY is used directly in the code outside of just the environment.