From 122bc08845c25367d23e3d7a74c29b7d1ac07234 Mon Sep 17 00:00:00 2001 From: Daniel Hobi Date: Wed, 15 Sep 2010 19:46:26 +0200 Subject: [PATCH] tools/env: allow option "-n" for fw_printenv In commit bd7b26f8 (Tools: set multiple variable with fw_setenv utility), the option parsing was changed to getopt_long(3), but option "-n" of fw_printenv was not included. This leads to an error message "invalid option -- 'n'" on stderr, although the output on stdout is correct. Signed-off-by: Daniel Hobi --- tools/env/fw_env_main.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tools/env/fw_env_main.c b/tools/env/fw_env_main.c index 82116b4b29..baf3a4d909 100644 --- a/tools/env/fw_env_main.c +++ b/tools/env/fw_env_main.c @@ -59,7 +59,7 @@ void usage(void) fprintf(stderr, "fw_printenv/fw_setenv, " "a command line interface to U-Boot environment\n\n" - "usage:\tfw_printenv\n" + "usage:\tfw_printenv [-n] [variable name]\n" "\tfw_setenv [variable name] [variable value]\n" "\tfw_setenv -s [ file ]\n" "\tfw_setenv -s - < [ file ]\n\n" @@ -93,9 +93,12 @@ main(int argc, char *argv[]) cmdname = p + 1; } - while ((c = getopt_long (argc, argv, "s:h", + while ((c = getopt_long (argc, argv, "ns:h", long_options, NULL)) != EOF) { switch (c) { + case 'n': + /* handled in fw_printenv */ + break; case 's': script_file = optarg; break; -- 2.25.1