diff options
| author | Simon Glass <sjg@chromium.org> | 2022-03-11 16:22:38 -0700 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2022-04-07 16:50:53 -0400 |
| commit | 9d4e88c02163e1f1824afa3ff142ec7a116fbe5c (patch) | |
| tree | cab70bdf6de2c947a564afbef2cdb076cd175239 /common | |
| parent | cc5a490cf46565d3a42f86beaac05f56f4f40741 (diff) | |
| download | u-boot-9d4e88c02163e1f1824afa3ff142ec7a116fbe5c.tar.gz | |
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>
Diffstat (limited to 'common')
| -rw-r--r-- | common/autoboot.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/common/autoboot.c b/common/autoboot.c index b8861d56218..63f2587941d 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); |