diff options
| author | Michal Simek <[email protected]> | 2024-03-22 12:43:17 +0100 |
|---|---|---|
| committer | Michal Simek <[email protected]> | 2024-04-02 15:07:22 +0200 |
| commit | 097ccdf9623d650aebe32ea98828d81503f63a10 (patch) | |
| tree | adb2e849b6753fe46f9a1f215792b092a4a86c91 /scripts/checkpatch.pl | |
| parent | 8e462bf3bdea694dbc359de5933fd3aabd8c7ac3 (diff) | |
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 <[email protected]>
Link: https://lore.kernel.org/r/830b9c93d30688cfa8a5cbb0388e79deb7731f29.1711107795.git.michal.simek@amd.com
Diffstat (limited to 'scripts/checkpatch.pl')
0 files changed, 0 insertions, 0 deletions
