diff options
| author | Joe Hershberger <joe.hershberger@ni.com> | 2012-10-03 09:38:46 +0000 | 
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2012-10-15 11:54:05 -0700 | 
| commit | 497f2053f8cdd5df412e8dd01db525eb04201ebb (patch) | |
| tree | 40fda235cd1febe9731cb8aac847563ccb962a76 /tools/env/fw_env.c | |
| parent | 47708457f8e3e155fc656d9b6d176aa53ced4440 (diff) | |
| download | olio-uboot-2014.01-497f2053f8cdd5df412e8dd01db525eb04201ebb.tar.xz olio-uboot-2014.01-497f2053f8cdd5df412e8dd01db525eb04201ebb.zip | |
tools/env: Use a board-specific default env
Originally added in aa701b94336b358798d676eef12a7b90bdac23f5
Before this patch, there was a hard-coded env that was used as default
if the env in flash is detected as invalid.  Now this tool (compiled
for a given board) will share the default env with the u-boot for the
board.
Fix include of config.h
Need to define "TEXT_BASE" when building the fw_env tool so that the
default env will be correct for environments which use it.
Define __ASSEMBLY__ when calling #include <config.h> so that we only
get #defines (all we're interested in).
Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
Diffstat (limited to 'tools/env/fw_env.c')
| -rw-r--r-- | tools/env/fw_env.c | 28 | 
1 files changed, 13 insertions, 15 deletions
| diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index 9f8dfdccb..57ad8c9db 100644 --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c @@ -45,8 +45,6 @@  #include "fw_env.h" -#include <config.h> -  #define WHITESPACE(c) ((c == '\t') || (c == ' '))  #define min(x, y) ({				\ @@ -81,7 +79,7 @@ static int dev_current;  #define ENVSECTORS(i) envdevices[(i)].env_sectors  #define DEVTYPE(i)    envdevices[(i)].mtd_type -#define CONFIG_ENV_SIZE ENVSIZE(dev_current) +#define CUR_ENVSIZE ENVSIZE(dev_current)  #define ENV_SIZE      getenvsize() @@ -226,7 +224,7 @@ static int get_config (char *);  #endif  static inline ulong getenvsize (void)  { -	ulong rc = CONFIG_ENV_SIZE - sizeof (long); +	ulong rc = CUR_ENVSIZE - sizeof(long);  	if (HaveRedundEnv)  		rc -= sizeof (char); @@ -442,7 +440,7 @@ int fw_env_write(char *name, char *value)  		++env;  	/*  	 * Overflow when: -	 * "name" + "=" + "val" +"\0\0"  > CONFIG_ENV_SIZE - (env-environment) +	 * "name" + "=" + "val" +"\0\0"  > CUR_ENVSIZE - (env-environment)  	 */  	len = strlen (name) + 2;  	/* add '=' for first arg, ' ' for all others */ @@ -957,8 +955,8 @@ static int flash_write (int fd_current, int fd_target, int dev_target)  	printf ("Writing new environment at 0x%lx on %s\n",  		DEVOFFSET (dev_target), DEVNAME (dev_target));  #endif -	rc = flash_write_buf (dev_target, fd_target, environment.image, -			      CONFIG_ENV_SIZE, DEVOFFSET (dev_target), +	rc = flash_write_buf(dev_target, fd_target, environment.image, +			      CUR_ENVSIZE, DEVOFFSET(dev_target),  			      DEVTYPE(dev_target));  	if (rc < 0)  		return rc; @@ -997,10 +995,10 @@ static int flash_read (int fd)  	DEVTYPE(dev_current) = mtdinfo.type; -	rc = flash_read_buf (dev_current, fd, environment.image, CONFIG_ENV_SIZE, +	rc = flash_read_buf(dev_current, fd, environment.image, CUR_ENVSIZE,  			     DEVOFFSET (dev_current), mtdinfo.type); -	return (rc != CONFIG_ENV_SIZE) ? -1 : 0; +	return (rc != CUR_ENVSIZE) ? -1 : 0;  }  static int flash_io (int mode) @@ -1097,11 +1095,11 @@ int fw_env_open(void)  	if (parse_config ())		/* should fill envdevices */  		return -1; -	addr0 = calloc (1, CONFIG_ENV_SIZE); +	addr0 = calloc(1, CUR_ENVSIZE);  	if (addr0 == NULL) { -		fprintf (stderr, +		fprintf(stderr,  			"Not enough memory for environment (%ld bytes)\n", -			CONFIG_ENV_SIZE); +			CUR_ENVSIZE);  		return -1;  	} @@ -1136,11 +1134,11 @@ int fw_env_open(void)  		flag0 = *environment.flags;  		dev_current = 1; -		addr1 = calloc (1, CONFIG_ENV_SIZE); +		addr1 = calloc(1, CUR_ENVSIZE);  		if (addr1 == NULL) { -			fprintf (stderr, +			fprintf(stderr,  				"Not enough memory for environment (%ld bytes)\n", -				CONFIG_ENV_SIZE); +				CUR_ENVSIZE);  			return -1;  		}  		redundant = addr1; |