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 <xypron.glpk@gmx.de>

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2022-03-11 16:22:38 -07:00 committed by Tom Rini
parent cc5a490cf4
commit 9d4e88c021
2 changed files with 5 additions and 1 deletions

View file

@ -446,6 +446,11 @@ const char *bootdelay_process(void)
s = env_get("bootdelay"); s = env_get("bootdelay");
bootdelay = s ? (int)simple_strtol(s, NULL, 10) : CONFIG_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)) if (IS_ENABLED(CONFIG_OF_CONTROL))
bootdelay = ofnode_conf_read_int("bootdelay", bootdelay); bootdelay = ofnode_conf_read_int("bootdelay", bootdelay);

View file

@ -120,7 +120,6 @@ bootdelay
The default value is defined by CONFIG_BOOTDELAY. The default value is defined by CONFIG_BOOTDELAY.
The value of 'bootdelay' is overridden by the /config/bootdelay value in The value of 'bootdelay' is overridden by the /config/bootdelay value in
the device-tree if CONFIG_OF_CONTROL=y. the device-tree if CONFIG_OF_CONTROL=y.
Does it really make sense that the devicetree overrides the user setting?
bootcmd bootcmd
The command that is run if the user does not enter the shell during the The command that is run if the user does not enter the shell during the