[U-Boot] [PATCH v2 05/40] env: Create a new file for environment functions

[Simon Goldschmidt]

Am 01.08.2019 um 17:46 schrieb Simon Glass:
> At present we have environment.h but this file includes all the
> environment-related header files as well as internals such as
> default_environment.
> 
> It seems desirable to have a new header to hold the commonly used
> environment functions, so that most files can avoid including all of this
> unnecessary stuff.
> 
> Create a new env.h header and move one function over to it.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Acked-by: Joe Hershberger <joe.hershberger at ni.com>

Reviewed-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>

> ---
> 
> Changes in v2: None
> 
>   common/command.c |  1 +
>   include/common.h |  4 ----
>   include/env.h    | 28 ++++++++++++++++++++++++++++
>   3 files changed, 29 insertions(+), 4 deletions(-)
>   create mode 100644 include/env.h
> 
> diff --git a/common/command.c b/common/command.c
> index db25bf54e0..4b887a267f 100644
> --- a/common/command.c
> +++ b/common/command.c
> @@ -11,6 +11,7 @@
>   #include <common.h>
>   #include <command.h>
>   #include <console.h>
> +#include <env.h>
>   #include <linux/ctype.h>
>   
>   /*
> diff --git a/include/common.h b/include/common.h
> index 4d0dd3f1b7..6b890470c1 100644
> --- a/include/common.h
> +++ b/include/common.h
> @@ -247,10 +247,6 @@ static inline int env_set_addr(const char *varname, const void *addr)
>   	return env_set_hex(varname, (ulong)addr);
>   }
>   
> -#ifdef CONFIG_AUTO_COMPLETE
> -int env_complete(char *var, int maxv, char *cmdv[], int maxsz, char *buf,
> -		 bool dollar_comp);
> -#endif
>   int get_env_id (void);
>   
>   void	pci_init_board(void);
> diff --git a/include/env.h b/include/env.h
> new file mode 100644
> index 0000000000..157ee9b724
> --- /dev/null
> +++ b/include/env.h
> @@ -0,0 +1,28 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
> +/*
> + * Common environment functions
> + *
> + * (C) Copyright 2000-2009
> + * Wolfgang Denk, DENX Software Engineering, wd at denx.de.
> + */
> +
> +#ifndef __ENV_H
> +#define __ENV_H
> +
> +#include <stdbool.h>
> +
> +/**
> + * env_complete() - return an auto-complete for environment variables
> + *
> + * @var: partial name to auto-complete
> + * @maxv: Maximum number of matches to return
> + * @cmdv: Returns a list of possible matches
> + * @maxsz: Size of buffer to use for matches
> + * @buf: Buffer to use for matches
> + * @dollar_comp: non-zero to wrap each match in ${...}
> + * @return number of matches found (in @cmdv)
> + */
> +int env_complete(char *var, int maxv, char *cmdv[], int maxsz, char *buf,
> +		 bool dollar_comp);
> +
> +#endif
>