]> git.dujemihanovic.xyz Git - u-boot.git/commit
xilinx: Do not call private env_get_location() when !ENV_IS_NOWHERE
authorMichal Simek <michal.simek@amd.com>
Fri, 22 Mar 2024 11:43:17 +0000 (12:43 +0100)
committerMichal Simek <michal.simek@amd.com>
Tue, 2 Apr 2024 13:07:22 +0000 (15:07 +0200)
commit097ccdf9623d650aebe32ea98828d81503f63a10
treeadb2e849b6753fe46f9a1f215792b092a4a86c91
parent8e462bf3bdea694dbc359de5933fd3aabd8c7ac3
xilinx: Do not call private env_get_location() when !ENV_IS_NOWHERE

Private function for finding out location of environment is not working
when ENV_IS_NOWHERE is disabled.
The reason is that current fallback is ENVL_UNKNOWN when
CONFIG_ENV_IS_NOWHERE is not enabled. The code could be updated like this

-               return ENVL_NOWHERE;
+               if (IS_ENABLED(CONFIG_ENV_IS_NOWHERE))
+                       return ENVL_NOWHERE;
+               return ENVL_UNKNOWN;

But then boot is still not working because ENVL_UNKNOWN has no driver and
env_init() returns -ENODEV. That's why it is better not to define board
specific env_get_location() because then weak env_get_location() is used
which is going over env_locations[] and returning locations which are
actually enabled in u-boot instance.

Signed-off-by: Michal Simek <michal.simek@amd.com>
Link: https://lore.kernel.org/r/830b9c93d30688cfa8a5cbb0388e79deb7731f29.1711107795.git.michal.simek@amd.com
board/xilinx/versal-net/board.c
board/xilinx/versal/board.c
board/xilinx/zynqmp/zynqmp.c