diff options
| author | Patrice Chotard <patrice.chotard@st.com> | 2019-01-23 18:12:48 +0100 |
|---|---|---|
| committer | Patrice Chotard <patrice.chotard@st.com> | 2019-06-06 17:40:14 +0200 |
| commit | e6b7afe737f95f61f347a340acdda1a48d29ed50 (patch) | |
| tree | bd8cf0476527d49abd3b517b9c1bfce061eb272f /drivers/mtd | |
| parent | 8f24b1a4a97cbeb8f2756be62bd524f166423406 (diff) | |
| download | u-boot-e6b7afe737f95f61f347a340acdda1a48d29ed50.zip u-boot-e6b7afe737f95f61f347a340acdda1a48d29ed50.tar.gz u-boot-e6b7afe737f95f61f347a340acdda1a48d29ed50.tar.bz2 | |
mtd: Fix get_mtdparts()
When ENV_IS_IN_UBI is enable, get_mtdparts is called before relocation.
During first get_mtdparts() call, mtdparts is not available in environment,
it can be retrieved by calling board_mtdparts_default(), but following
env_set() do nothing as we are before relocation. Finally mtdparts is
still not available in environment.
At second get_mtdparts() call, use_defaults is false, but mtdparts is still
not in environment and is NULL.
Remove use_defaults bool, only mtdparts criteria is useful.
Fixes: commit 5ffcd50612f6 ("mtd: Use default mtdparts/mtids when not defined
in the environment")
Signed-off-by: Patrice Chotard <patrice.chotard@st.com>
Diffstat (limited to 'drivers/mtd')
| -rw-r--r-- | drivers/mtd/mtd_uboot.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/mtd/mtd_uboot.c b/drivers/mtd/mtd_uboot.c index d638f70..0a41ed4 100644 --- a/drivers/mtd/mtd_uboot.c +++ b/drivers/mtd/mtd_uboot.c @@ -122,7 +122,6 @@ static const char *get_mtdparts(void) { __maybe_unused const char *mtdids = NULL; static char tmp_parts[MTDPARTS_MAXLEN]; - static bool use_defaults = true; const char *mtdparts = NULL; if (gd->flags & GD_FLG_ENV_READY) @@ -130,7 +129,7 @@ static const char *get_mtdparts(void) else if (env_get_f("mtdparts", tmp_parts, sizeof(tmp_parts)) != -1) mtdparts = tmp_parts; - if (mtdparts || !use_defaults) + if (mtdparts) return mtdparts; #if defined(CONFIG_SYS_MTDPARTS_RUNTIME) @@ -144,8 +143,6 @@ static const char *get_mtdparts(void) if (mtdparts) env_set("mtdparts", mtdparts); - use_defaults = false; - return mtdparts; } |