On Thu, 26 Oct 2017 14:57:53 +0200
Stefan Agner <ste...@agner.ch> wrote:

> From: Stefan Agner <stefan.ag...@toradex.com>
> 
> The fw_env utility family has a default environment compiled in
> which ties it quite strongly to the U-Boot source/config it has
> been built with. Allow to display the U-Boot version it has been
> built with using the -v/--version argument.
> 
> Signed-off-by: Stefan Agner <stefan.ag...@toradex.com>
> ---
> 
>  tools/env/fw_env_main.c | 14 +++++++++++---
>  1 file changed, 11 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/env/fw_env_main.c b/tools/env/fw_env_main.c
> index 6e278ca80b..0b9063742c 100644
> --- a/tools/env/fw_env_main.c
> +++ b/tools/env/fw_env_main.c
> @@ -34,6 +34,7 @@
>  #include <stdlib.h>
>  #include <sys/file.h>
>  #include <unistd.h>
> +#include <version.h>
>  #include "fw_env_private.h"
>  #include "fw_env.h"
>  
> @@ -48,6 +49,7 @@ static struct option long_options[] = {
>       {"script", required_argument, NULL, 's'},
>       {"noheader", required_argument, NULL, 'n'},
>       {"lock", required_argument, NULL, 'l'},
> +     {"version", no_argument, NULL, 'v'},
>       {NULL, 0, NULL, 0}
>  };
>  
> @@ -67,6 +69,7 @@ void usage_printenv(void)
>               "Print variables from U-Boot environment\n"
>               "\n"
>               " -h, --help           print this help.\n"
> +             " -v, --version        display version\n"
>  #ifdef CONFIG_ENV_AES
>               " -a, --aes            aes key to access
> environment\n" #endif
> @@ -85,6 +88,7 @@ void usage_env_set(void)
>               "Modify variables in U-Boot environment\n"
>               "\n"
>               " -h, --help           print this help.\n"
> +             " -v, --version        display version\n"
>  #ifdef CONFIG_ENV_AES
>               " -a, --aes            aes key to access
> environment\n" #endif
> @@ -123,7 +127,7 @@ static void parse_common_args(int argc, char
> *argv[]) env_opts.config_file = CONFIG_FILE;
>  #endif
>  
> -     while ((c = getopt_long(argc, argv, ":a:c:l:h",
> long_options, NULL)) !=
> +     while ((c = getopt_long(argc, argv, ":a:c:l:h:v",
> long_options, NULL)) != EOF) {
>               switch (c) {
>               case 'a':
> @@ -145,6 +149,10 @@ static void parse_common_args(int argc, char
> *argv[]) do_printenv ? usage_printenv() : usage_env_set();
>                       exit(EXIT_SUCCESS);
>                       break;
> +             case 'v':
> +                     fprintf(stderr, "Compiled with "
> U_BOOT_VERSION "\n");
> +                     exit(EXIT_SUCCESS);
> +                     break;
>               default:
>                       /* ignore unknown options */
>                       break;
> @@ -162,7 +170,7 @@ int parse_printenv_args(int argc, char *argv[])
>  
>       parse_common_args(argc, argv);
>  
> -     while ((c = getopt_long(argc, argv, "a:c:ns:l:h",
> long_options, NULL))
> +     while ((c = getopt_long(argc, argv, "a:c:ns:l:h:v",
> long_options, NULL)) != EOF) {
>               switch (c) {
>               case 'n':
> @@ -189,7 +197,7 @@ int parse_setenv_args(int argc, char *argv[])
>  
>       parse_common_args(argc, argv);
>  
> -     while ((c = getopt_long(argc, argv, "a:c:ns:l:h",
> long_options, NULL))
> +     while ((c = getopt_long(argc, argv, "a:c:ns:l:h:v",
> long_options, NULL)) != EOF) {
>               switch (c) {
>               case 's':

Reviewed-by: Lukasz Majewski <lu...@denx.de>


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to