]> git.baikalelectronics.ru Git - uboot.git/commit
Revert "env: Load env when ENV_IS_NOWHERE is only location selected"
authorMarek Vasut <marex@denx.de>
Sun, 10 Apr 2022 04:46:52 +0000 (06:46 +0200)
committerTom Rini <trini@konsulko.com>
Sun, 10 Apr 2022 15:20:16 +0000 (11:20 -0400)
commit1e8547fb4eaa964a27fbdfe5a87b4a9d4c28b1b7
treede1c05d2aa1b8058fdb84b5d1134b5e62370a92a
parentacd1b15a0117d1f096fe4f5886ca45c0ee5d9afe
Revert "env: Load env when ENV_IS_NOWHERE is only location selected"

This reverts commit 50292199aa5c8b1b073c93e4b349f52040d2abdf.

This commit broke environment on literally every board I have access
to, with this revert in place, environment works as it should again.
The problem I observe with this patch is that saved environment in
either SPI NOR or eMMC is never used, the system always falls back
to default environment. The 'saveenv' command does succeed, but then
after reset, the default env is again used.

Furthermore, the commit introduced duplicate code in env_init(), this:
"
if (!prio) {
gd->env_addr = (ulong)&default_environment[0];
gd->env_valid = ENV_INVALID;

return 0;
}

if (ret == -ENOENT) {
gd->env_addr = (ulong)&default_environment[0];
gd->env_valid = ENV_INVALID;

return 0;
}
"

Furthermore, the commit is missing DCO SoB line.

Also note that upstream does not support UltraZed EG board, so
this might have been a patch pulled from downstream which did
depend on some other downstream behavior.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Felix.Vietmeyer@jila.colorado.edu <felix.vietmeyer@jila.colorado.edu>
Cc: Tom Rini <trini@konsulko.com>
env/env.c