diff options
| author | Tom Rini <[email protected]> | 2026-03-23 13:51:06 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2026-04-07 11:30:41 -0600 |
| commit | 67e72e9686e702b85c38f62e4fc02f8efe3ffe82 (patch) | |
| tree | cb9f70d74098c77a34137fc8a8f413511bdf8d54 | |
| parent | 8af4e124875932567cea627e04f26cf5cc529e25 (diff) | |
env: remote: Disallow CMD_SAVEENV
Looking at how the saveenv portion of this driver was implemented, it
does not appear that it could actually result in changes being saved on
the remote end. Update Kconfig to disallow CMD_SAVEENV for
ENV_IS_IN_REMOTE and then remove the relevant code.
Signed-off-by: Tom Rini <[email protected]>
| -rw-r--r-- | cmd/Kconfig | 1 | ||||
| -rw-r--r-- | env/remote.c | 13 |
2 files changed, 1 insertions, 13 deletions
diff --git a/cmd/Kconfig b/cmd/Kconfig index f47ce7f45ce..ff6ad37fbe4 100644 --- a/cmd/Kconfig +++ b/cmd/Kconfig @@ -717,6 +717,7 @@ config CMD_GREPENV config CMD_SAVEENV bool "saveenv" + depends on !(SRIO_PCIE_BOOT_SLAVE && ENV_IS_IN_REMOTE) default y help Save all environment variables into the compiled-in persistent diff --git a/env/remote.c b/env/remote.c index 0cc383c2360..5dd929155a4 100644 --- a/env/remote.c +++ b/env/remote.c @@ -31,18 +31,6 @@ static int env_remote_init(void) return -ENOENT; } -#ifdef CONFIG_CMD_SAVEENV -static int env_remote_save(void) -{ -#ifdef CONFIG_SRIO_PCIE_BOOT_SLAVE - printf("Can not support the 'saveenv' when boot from SRIO or PCIE!\n"); - return 1; -#else - return 0; -#endif -} -#endif /* CONFIG_CMD_SAVEENV */ - static int env_remote_load(void) { #ifndef ENV_IS_EMBEDDED @@ -56,6 +44,5 @@ U_BOOT_ENV_LOCATION(remote) = { .location = ENVL_REMOTE, ENV_NAME("Remote") .load = env_remote_load, - .save = env_save_ptr(env_remote_save), .init = env_remote_init, }; |