diff options
author | Kumar Gala <galak@kernel.crashing.org> | 2010-10-22 03:18:13 -0500 |
---|---|---|
committer | Wolfgang Denk <wd@denx.de> | 2010-10-23 22:10:54 +0200 |
commit | c4b115f536b0cb41b0d863ab00fe52d7772433a0 (patch) | |
tree | add5d84b93ed44959f093bfa6b1bf39dd89e1924 /common/hwconfig.c | |
parent | 3f7ffa440a60d5b37d1eabbee1b4c2af60eb104b (diff) | |
download | bootable_bootloader_goldelico_gta04-c4b115f536b0cb41b0d863ab00fe52d7772433a0.zip bootable_bootloader_goldelico_gta04-c4b115f536b0cb41b0d863ab00fe52d7772433a0.tar.gz bootable_bootloader_goldelico_gta04-c4b115f536b0cb41b0d863ab00fe52d7772433a0.tar.bz2 |
hwconfig: Utilize getenv_f before relocation to allow for larger buffer
Since we use hwconfig in cases before relocation (like getting DDR
params on FSL PPC systems), we can have strings that exceed the early
small (32 byte) buffer size that getenv will handle.
So we explicitly allocate our own buffer on the stack and use if to
handle getting the hwconfig env string. We currently utilize a string
length of 128 bytes.
This allows us to get rid of boot messages like:
env_buf too small [32]
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
Diffstat (limited to 'common/hwconfig.c')
-rw-r--r-- | common/hwconfig.c | 21 |
1 files changed, 20 insertions, 1 deletions
diff --git a/common/hwconfig.c b/common/hwconfig.c index 1f9f4a0..3c9759f 100644 --- a/common/hwconfig.c +++ b/common/hwconfig.c @@ -26,6 +26,8 @@ #define min(a, b) (((a) < (b)) ? (a) : (b)) #endif /* HWCONFIG_TEST */ +DECLARE_GLOBAL_DATA_PTR; + static const char *hwconfig_parse(const char *opts, size_t maxlen, const char *opt, char *stopchs, char eqch, size_t *arglen) @@ -69,9 +71,26 @@ next: const char *cpu_hwconfig __attribute__((weak)); const char *board_hwconfig __attribute__((weak)); +#define HWCONFIG_PRE_RELOC_BUF_SIZE 128 + static const char *__hwconfig(const char *opt, size_t *arglen) { - const char *env_hwconfig = getenv("hwconfig"); + const char *env_hwconfig = NULL; + char buf[HWCONFIG_PRE_RELOC_BUF_SIZE]; + + if (gd->flags & GD_FLG_ENV_READY) { + env_hwconfig = getenv("hwconfig"); + } else { + /* + * Use our own on stack based buffer before relocation to allow + * accessing longer hwconfig strings that might be in the + * environment before we've relocated. This is pretty fragile + * on both the use of stack and if the buffer is big enough. + * However we will get a warning from getenv_f for the later. + */ + if ((getenv_f("hwconfig", buf, sizeof(buf))) > 0) + env_hwconfig = buf; + } if (env_hwconfig) return hwconfig_parse(env_hwconfig, strlen(env_hwconfig), |