env: Drop the env_name_spec global
Add a name to the driver and use that instead of the global variable
declared by each driver.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
diff --git a/cmd/nvedit.c b/cmd/nvedit.c
index cd17db6..75df997 100644
--- a/cmd/nvedit.c
+++ b/cmd/nvedit.c
@@ -706,7 +706,9 @@
static int do_env_save(cmd_tbl_t *cmdtp, int flag, int argc,
char * const argv[])
{
- printf("Saving Environment to %s...\n", env_name_spec);
+ struct env_driver *env = env_driver_lookup_default();
+
+ printf("Saving Environment to %s...\n", env->name);
return saveenv() ? 1 : 0;
}
diff --git a/env/dataflash.c b/env/dataflash.c
index 8ab482b..9421a6a 100644
--- a/env/dataflash.c
+++ b/env/dataflash.c
@@ -16,8 +16,6 @@
env_t *env_ptr;
-char *env_name_spec = "dataflash";
-
static unsigned char env_dataflash_get_char(int index)
{
uchar c;
@@ -70,6 +68,7 @@
U_BOOT_ENV_LOCATION(dataflash) = {
.location = ENVL_DATAFLASH,
+ ENV_NAME("dataflash")
.get_char = env_dataflash_get_char,
.load = env_dataflash_load,
.save = env_save_ptr(env_dataflash_save),
diff --git a/env/eeprom.c b/env/eeprom.c
index 3cc4126..2c795b9 100644
--- a/env/eeprom.c
+++ b/env/eeprom.c
@@ -23,8 +23,6 @@
env_t *env_ptr;
-char *env_name_spec = "EEPROM";
-
static int eeprom_bus_read(unsigned dev_addr, unsigned offset,
uchar *buffer, unsigned cnt)
{
@@ -233,6 +231,7 @@
U_BOOT_ENV_LOCATION(eeprom) = {
.location = ENVL_EEPROM,
+ ENV_NAME("EEPROM")
.get_char = env_eeprom_get_char,
.load = env_eeprom_load,
.save = env_save_ptr(env_eeprom_save),
diff --git a/env/env.c b/env/env.c
index 12cd475..53a085c 100644
--- a/env/env.c
+++ b/env/env.c
@@ -54,7 +54,7 @@
return ENVL_UNKNOWN;
}
-static struct env_driver *env_driver_lookup_default(void)
+struct env_driver *env_driver_lookup_default(void)
{
enum env_location loc = env_get_default_location();
struct env_driver *drv;
diff --git a/env/ext4.c b/env/ext4.c
index 25a5cbe..5acdf0e 100644
--- a/env/ext4.c
+++ b/env/ext4.c
@@ -31,8 +31,6 @@
#include <ext4fs.h>
#include <mmc.h>
-char *env_name_spec = "EXT4";
-
env_t *env_ptr;
DECLARE_GLOBAL_DATA_PTR;
@@ -121,6 +119,7 @@
U_BOOT_ENV_LOCATION(ext4) = {
.location = ENVL_EXT4,
+ ENV_NAME("EXT4")
.load = env_ext4_load,
.save = env_save_ptr(env_ext4_save),
};
diff --git a/env/fat.c b/env/fat.c
index 1ff3d9d..eda18db 100644
--- a/env/fat.c
+++ b/env/fat.c
@@ -31,8 +31,6 @@
# endif
#endif
-char *env_name_spec = "FAT";
-
env_t *env_ptr;
DECLARE_GLOBAL_DATA_PTR;
@@ -116,6 +114,7 @@
U_BOOT_ENV_LOCATION(fat) = {
.location = ENVL_FAT,
+ ENV_NAME("FAT")
#ifdef LOADENV
.load = env_fat_load,
#endif
diff --git a/env/flash.c b/env/flash.c
index 6ec9b61..2d72c51 100644
--- a/env/flash.c
+++ b/env/flash.c
@@ -45,8 +45,6 @@
#define INITENV
#endif
-char *env_name_spec = "Flash";
-
#ifdef ENV_IS_EMBEDDED
env_t *env_ptr = &environment;
@@ -359,6 +357,7 @@
U_BOOT_ENV_LOCATION(flash) = {
.location = ENVL_FLASH,
+ ENV_NAME("Flash")
#ifdef LOADENV
.load = env_flash_load,
#endif
diff --git a/env/mmc.c b/env/mmc.c
index 966f620..18da6dc 100644
--- a/env/mmc.c
+++ b/env/mmc.c
@@ -23,8 +23,6 @@
#error CONFIG_ENV_SIZE_REDUND should be the same as CONFIG_ENV_SIZE
#endif
-char *env_name_spec = "MMC";
-
#ifdef ENV_IS_EMBEDDED
env_t *env_ptr = &environment;
#else /* ! ENV_IS_EMBEDDED */
@@ -318,6 +316,7 @@
U_BOOT_ENV_LOCATION(mmc) = {
.location = ENVL_MMC,
+ ENV_NAME("MMC")
.load = env_mmc_load,
#ifndef CONFIG_SPL_BUILD
.save = env_save_ptr(env_mmc_save),
diff --git a/env/nand.c b/env/nand.c
index 596bdd3..e74a8c6 100644
--- a/env/nand.c
+++ b/env/nand.c
@@ -40,8 +40,6 @@
#define CONFIG_ENV_RANGE CONFIG_ENV_SIZE
#endif
-char *env_name_spec = "NAND";
-
#if defined(ENV_IS_EMBEDDED)
env_t *env_ptr = &environment;
#elif defined(CONFIG_NAND_ENV_DST)
@@ -398,6 +396,7 @@
U_BOOT_ENV_LOCATION(nand) = {
.location = ENVL_NAND,
+ ENV_NAME("NAND")
.load = env_nand_load,
#if defined(CMD_SAVEENV)
.save = env_save_ptr(env_nand_save),
diff --git a/env/nowhere.c b/env/nowhere.c
index 1d2ae85..1a17d69 100644
--- a/env/nowhere.c
+++ b/env/nowhere.c
@@ -19,4 +19,5 @@
U_BOOT_ENV_LOCATION(nowhere) = {
.location = ENVL_NOWHERE,
+ ENV_NAME("nowhere")
};
diff --git a/env/nvram.c b/env/nvram.c
index 09091b8..0f4b491 100644
--- a/env/nvram.c
+++ b/env/nvram.c
@@ -41,8 +41,6 @@
env_t *env_ptr = (env_t *)CONFIG_ENV_ADDR;
#endif
-char *env_name_spec = "NVRAM";
-
#ifdef CONFIG_SYS_NVRAM_ACCESS_ROUTINE
static uchar env_nvram_get_char(int index)
{
@@ -115,6 +113,7 @@
U_BOOT_ENV_LOCATION(nvram) = {
.location = ENVL_NVRAM,
+ ENV_NAME("NVRAM")
#ifdef CONFIG_SYS_NVRAM_ACCESS_ROUTINE
.get_char = env_nvram_get_char,
#endif
diff --git a/env/onenand.c b/env/onenand.c
index cb1ab53..319f553 100644
--- a/env/onenand.c
+++ b/env/onenand.c
@@ -21,8 +21,6 @@
#include <linux/mtd/mtd.h>
#include <linux/mtd/onenand.h>
-char *env_name_spec = "OneNAND";
-
#define ONENAND_MAX_ENV_SIZE CONFIG_ENV_SIZE
#define ONENAND_ENV_SIZE(mtd) (ONENAND_MAX_ENV_SIZE - ENV_HEADER_SIZE)
@@ -108,6 +106,7 @@
U_BOOT_ENV_LOCATION(onenand) = {
.location = ENVL_ONENAND,
+ ENV_NAME("OneNAND")
.load = env_onenand_load,
.save = env_save_ptr(env_onenand_save),
};
diff --git a/env/remote.c b/env/remote.c
index c5dce5b..0d8865b 100644
--- a/env/remote.c
+++ b/env/remote.c
@@ -11,8 +11,6 @@
#include <environment.h>
#include <linux/stddef.h>
-char *env_name_spec = "Remote";
-
#ifdef ENV_IS_EMBEDDED
env_t *env_ptr = &environment;
#else /* ! ENV_IS_EMBEDDED */
@@ -57,6 +55,7 @@
U_BOOT_ENV_LOCATION(remote) = {
.location = ENVL_REMOTE,
+ ENV_NAME("Remote")
.load = env_remote_load,
.save = env_save_ptr(env_remote_save),
.init = env_remote_init,
diff --git a/env/sata.c b/env/sata.c
index a5ff54c..16d8f93 100644
--- a/env/sata.c
+++ b/env/sata.c
@@ -24,8 +24,6 @@
#error CONFIG_ENV_OFFSET or CONFIG_ENV_SIZE not defined
#endif
-char *env_name_spec = "SATA";
-
DECLARE_GLOBAL_DATA_PTR;
__weak int sata_get_env_dev(void)
@@ -119,6 +117,7 @@
U_BOOT_ENV_LOCATION(sata) = {
.location = ENVL_ESATA,
+ ENV_NAME("SATA")
.load = env_sata_load,
.save = env_save_ptr(env_sata_save),
};
diff --git a/env/sf.c b/env/sf.c
index a07641b..07386c6 100644
--- a/env/sf.c
+++ b/env/sf.c
@@ -48,8 +48,6 @@
DECLARE_GLOBAL_DATA_PTR;
-char *env_name_spec = "SPI Flash";
-
static struct spi_flash *env_flash;
static int setup_flash_device(void)
@@ -346,6 +344,7 @@
U_BOOT_ENV_LOCATION(sf) = {
.location = ENVL_SPI_FLASH,
+ ENV_NAME("SPI Flash")
.load = env_sf_load,
#ifdef CMD_SAVEENV
.save = env_save_ptr(env_sf_save),
diff --git a/env/ubi.c b/env/ubi.c
index 4811f97..4d5ae13 100644
--- a/env/ubi.c
+++ b/env/ubi.c
@@ -16,8 +16,6 @@
#include <ubi_uboot.h>
#undef crc32
-char *env_name_spec = "UBI";
-
env_t *env_ptr;
DECLARE_GLOBAL_DATA_PTR;
diff --git a/include/environment.h b/include/environment.h
index 7eaba72..584fd72 100644
--- a/include/environment.h
+++ b/include/environment.h
@@ -143,10 +143,6 @@
# define ENV_HEADER_SIZE (sizeof(uint32_t))
#endif
-#if defined(CONFIG_CMD_SAVEENV) && !defined(CONFIG_ENV_IS_NOWHERE)
-extern char *env_name_spec;
-#endif
-
#ifdef CONFIG_ENV_AES
/* Make sure the payload is multiple of AES block size */
#define ENV_SIZE ((CONFIG_ENV_SIZE - ENV_HEADER_SIZE) & ~(16 - 1))
@@ -224,6 +220,7 @@
};
struct env_driver {
+ const char *name;
enum env_location location;
/**
@@ -269,6 +266,13 @@
#define U_BOOT_ENV_LOCATION(__name) \
ll_entry_declare(struct env_driver, __name, env_driver)
+/* Declare the name of a location */
+#ifdef CONFIG_CMD_SAVEENV
+#define ENV_NAME(_name) .name = _name,
+#else
+#define ENV_NAME(_name)
+#endif
+
#ifdef CONFIG_CMD_SAVEENV
#define env_save_ptr(x) x
#else
@@ -303,6 +307,13 @@
int env_import_redund(const char *buf1, const char *buf2);
#endif
+/**
+ * env_driver_lookup_default() - Look up the default environment driver
+ *
+ * @return pointer to driver, or NULL if none (which should not happen)
+ */
+struct env_driver *env_driver_lookup_default(void);
+
#endif /* DO_DEPS_ONLY */
#endif /* _ENVIRONMENT_H_ */