On Aug 21, 2008, at 6:35 PM, Wolfgang Denk wrote:
Dear Kumar Gala,
In message <1219350079-25416-1-git-send-email-galak@kernel.crashing.org
you wrote: Provide a boot command for being able to boot from a hard drive
Signed-off-by: Kumar Gala galak@kernel.crashing.org
common/env_common.c | 3 +++ common/environment.c | 3 +++ tools/env/fw_env.c | 3 +++ 3 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/common/env_common.c b/common/env_common.c index d51c211..271e71f 100644 --- a/common/env_common.c +++ b/common/env_common.c @@ -64,6 +64,9 @@ uchar default_environment[] = { #ifdef CONFIG_BOOTCOMMAND "bootcmd=" CONFIG_BOOTCOMMAND "\0" #endif +#ifdef CONFIG_HDBOOTCOMMAND
- "hdboot=" CONFIG_HDBOOTCOMMAND "\0"
+#endif #ifdef CONFIG_RAMBOOTCOMMAND "ramboot=" CONFIG_RAMBOOTCOMMAND "\0" #endif
The percentage of boards that actually have a HDD is so small that I do not think such an addition to these global files is justified, especially since it is trivial to add a definition of such an environment variable through a CONFIG_EXTRA_ENV_SETTINGS definition.
Or is there a specific reason why CONFIG_EXTRA_ENV_SETTINGS would not work for you?
Nope, that solution works as good as any.
- k