From: Marek Behún marek.behun@nic.cz
Hello Simon, Pali,
this is v2 of the series that adds support for board specific runtime determined default environment variables.
(Went through CI on github.)
As requested, I converted it to use sysinfo (and added support for some new things into sysinfo).
I haven't tested the change on ESPRESSObin yet, because I don't have the board at home. But I copy-pasted the code to turris_mox and it worked as expected there. Nonetheless it should be tested, I or Pali can do it today or tomorrow.
Marek
Marek Behún (12): env: Don't set ready flag if import failed in env_set_default() env: Fix env_get() when returning empty string using env_get_f() env: Simplify env_get_default() sysinfo: Make sysinfo_get_str() behave like snprintf() test: Use ut_asserteq_str() instead of ut_assertok(strcmp()) sysinfo: Add get_str_list() method sysinfo: Make .detect() non-mandatory sysinfo: Add support for iterating string list env: Change return behaviour of env_set_default_vars() env: Add support for overwriting default environment via sysinfo arm: mvebu: Espressobin: Use new API for setting default env at runtime env: Remove support for read-write default_environment[]
board/Marvell/mvebu_armada-37xx/board.c | 135 ++++++++++------ board/google/chromebook_coral/coral.c | 13 +- common/board_info.c | 2 +- configs/mvebu_espressobin-88f3720_defconfig | 1 + drivers/sysinfo/gpio.c | 2 +- drivers/sysinfo/rcar3.c | 2 +- drivers/sysinfo/sandbox.c | 20 ++- drivers/sysinfo/sysinfo-uclass.c | 99 +++++++++++- env/common.c | 165 +++++++++++++++++--- include/configs/mvebu_armada-37xx.h | 17 +- include/env.h | 1 + include/env_default.h | 2 - include/env_internal.h | 4 - include/sysinfo.h | 163 ++++++++++++++++++- lib/smbios.c | 2 +- test/dm/cpu.c | 4 +- test/dm/soc.c | 4 +- test/dm/sysinfo-gpio.c | 12 +- test/dm/sysinfo.c | 66 ++++++-- test/dm/usb.c | 2 +- test/dm/virtio.c | 2 +- test/print_ut.c | 32 ++-- 22 files changed, 595 insertions(+), 155 deletions(-)