From: Simon Glass Date: Fri, 11 Mar 2022 23:22:38 +0000 (-0700) Subject: env: Move the doc comment to the code X-Git-Url: http://git.dujemihanovic.xyz/?a=commitdiff_plain;h=9d4e88c02163e1f1824afa3ff142ec7a116fbe5c;p=u-boot.git env: Move the doc comment to the code This doesn't really make much sense in the documentation. Add a code comment instead. Suggested-by: Heinrich Schuchardt Signed-off-by: Simon Glass --- diff --git a/common/autoboot.c b/common/autoboot.c index b8861d5621..63f2587941 100644 --- a/common/autoboot.c +++ b/common/autoboot.c @@ -446,6 +446,11 @@ const char *bootdelay_process(void) s = env_get("bootdelay"); bootdelay = s ? (int)simple_strtol(s, NULL, 10) : CONFIG_BOOTDELAY; + /* + * Does it really make sense that the devicetree overrides the user + * setting? It is possibly helpful for security since the device tree + * may be signed whereas the environment is often loaded from storage. + */ if (IS_ENABLED(CONFIG_OF_CONTROL)) bootdelay = ofnode_conf_read_int("bootdelay", bootdelay); diff --git a/doc/usage/environment.rst b/doc/usage/environment.rst index d295cc8987..4b228c1de3 100644 --- a/doc/usage/environment.rst +++ b/doc/usage/environment.rst @@ -120,7 +120,6 @@ bootdelay The default value is defined by CONFIG_BOOTDELAY. The value of 'bootdelay' is overridden by the /config/bootdelay value in the device-tree if CONFIG_OF_CONTROL=y. - Does it really make sense that the devicetree overrides the user setting? bootcmd The command that is run if the user does not enter the shell during the