From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Sun, 9 Jul 2017 14:53:07 -0600 Subject: [U-Boot] [PATCH 24/30] env: Drop unused env_ptr variables In-Reply-To: <20170709205313.116174-1-sjg@chromium.org> References: <20170709205313.116174-1-sjg@chromium.org> Message-ID: <20170709205313.116174-25-sjg@chromium.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de This variable is declared as a global in most environment location drivers. But it is not used outside the drivers and most of the declarations are unnecessary. Also some drivers call free() on env_ptr which seems wrong since it is not in the heap. Drop the variable where possible, and all calls to free(). Signed-off-by: Simon Glass --- env/dataflash.c | 2 -- env/eeprom.c | 4 ---- env/ext4.c | 2 -- env/fat.c | 2 -- env/mmc.c | 8 -------- env/nand.c | 2 -- env/nowhere.c | 2 -- env/nvram.c | 1 - env/ubi.c | 2 -- 9 files changed, 25 deletions(-) diff --git a/env/dataflash.c b/env/dataflash.c index 9421a6a5b2..28dcf6fa1c 100644 --- a/env/dataflash.c +++ b/env/dataflash.c @@ -14,8 +14,6 @@ DECLARE_GLOBAL_DATA_PTR; -env_t *env_ptr; - static unsigned char env_dataflash_get_char(int index) { uchar c; diff --git a/env/eeprom.c b/env/eeprom.c index 2c795b9236..39adccabbe 100644 --- a/env/eeprom.c +++ b/env/eeprom.c @@ -21,8 +21,6 @@ DECLARE_GLOBAL_DATA_PTR; -env_t *env_ptr; - static int eeprom_bus_read(unsigned dev_addr, unsigned offset, uchar *buffer, unsigned cnt) { @@ -196,8 +194,6 @@ static int env_eeprom_save(void) char flag_obsolete = OBSOLETE_FLAG; #endif - BUG_ON(env_ptr != NULL); - rc = env_export(&env_new); if (rc) return rc; diff --git a/env/ext4.c b/env/ext4.c index 5acdf0eda0..ee073a8b7c 100644 --- a/env/ext4.c +++ b/env/ext4.c @@ -31,8 +31,6 @@ #include #include -env_t *env_ptr; - DECLARE_GLOBAL_DATA_PTR; #ifdef CONFIG_CMD_SAVEENV diff --git a/env/fat.c b/env/fat.c index cf33549fda..323288a0f0 100644 --- a/env/fat.c +++ b/env/fat.c @@ -19,8 +19,6 @@ #include #include -env_t *env_ptr; - DECLARE_GLOBAL_DATA_PTR; #if defined(CONFIG_CMD_SAVEENV) && !defined(CONFIG_SPL_BUILD) diff --git a/env/mmc.c b/env/mmc.c index ae10b5b1f8..cf85bc232f 100644 --- a/env/mmc.c +++ b/env/mmc.c @@ -23,12 +23,6 @@ #error CONFIG_ENV_SIZE_REDUND should be the same as CONFIG_ENV_SIZE #endif -#ifdef ENV_IS_EMBEDDED -env_t *env_ptr = &environment; -#else /* ! ENV_IS_EMBEDDED */ -env_t *env_ptr; -#endif /* ENV_IS_EMBEDDED */ - DECLARE_GLOBAL_DATA_PTR; #if !defined(CONFIG_ENV_OFFSET) @@ -284,8 +278,6 @@ static void env_mmc_load(void) gd->env_valid = ENV_VALID; } - free(env_ptr); - if (gd->env_valid == ENV_VALID) ep = tmp_env1; else diff --git a/env/nand.c b/env/nand.c index 98b1dfc618..695957ea1c 100644 --- a/env/nand.c +++ b/env/nand.c @@ -361,8 +361,6 @@ static void env_nand_load(void) gd->env_valid = ENV_VALID; } - free(env_ptr); - if (gd->env_valid == ENV_VALID) ep = tmp_env1; else diff --git a/env/nowhere.c b/env/nowhere.c index 1a17d6978b..d60de494e6 100644 --- a/env/nowhere.c +++ b/env/nowhere.c @@ -15,8 +15,6 @@ DECLARE_GLOBAL_DATA_PTR; -env_t *env_ptr; - U_BOOT_ENV_LOCATION(nowhere) = { .location = ENVL_NOWHERE, ENV_NAME("nowhere") diff --git a/env/nvram.c b/env/nvram.c index 0f4b491fd3..ac389ace62 100644 --- a/env/nvram.c +++ b/env/nvram.c @@ -36,7 +36,6 @@ DECLARE_GLOBAL_DATA_PTR; #ifdef CONFIG_SYS_NVRAM_ACCESS_ROUTINE extern void *nvram_read(void *dest, const long src, size_t count); extern void nvram_write(long dest, const void *src, size_t count); -env_t *env_ptr; #else env_t *env_ptr = (env_t *)CONFIG_ENV_ADDR; #endif diff --git a/env/ubi.c b/env/ubi.c index 0f1e5bcd22..65be2b1bbf 100644 --- a/env/ubi.c +++ b/env/ubi.c @@ -16,8 +16,6 @@ #include #undef crc32 -env_t *env_ptr; - DECLARE_GLOBAL_DATA_PTR; #ifdef CONFIG_CMD_SAVEENV -- 2.13.2.725.g09c95d1e9-goog