[PATCH v1 1/1] cmd: nvedit: Mark a few functions static

[Simon Glass]

On Mon, 21 Oct 2024 at 15:56, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> Some functions are not used anywhere except the same file
> where they are defined. Mark them static. This helps avoiding
> the compiler warnings:
>
>   cmd/nvedit.c:201:5: warning: no previous prototype for ‘do_env_ask’ [-Wmissing-prototypes]
>   cmd/nvedit.c:315:5: warning: no previous prototype for ‘do_env_callback’ [-Wmissing-prototypes]
>   cmd/nvedit.c:384:5: warning: no previous prototype for ‘do_env_flags’ [-Wmissing-prototype ]
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> ---
>  cmd/nvedit.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>

Reviewed-by: Simon Glass <sjg at chromium.org>


> diff --git a/cmd/nvedit.c b/cmd/nvedit.c
> index 98a687bcabbd..0bd4799dc5a2 100644
> --- a/cmd/nvedit.c
> +++ b/cmd/nvedit.c
> @@ -198,7 +198,7 @@ static int do_env_set(struct cmd_tbl *cmdtp, int flag, int argc,
>   * Prompt for environment variable
>   */
>  #if defined(CONFIG_CMD_ASKENV)
> -int do_env_ask(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
> +static int do_env_ask(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
>  {
>         char message[CONFIG_SYS_CBSIZE];
>         int i, len, pos, size;
> @@ -312,8 +312,8 @@ static int print_active_callback(struct env_entry *entry)
>  /*
>   * Print the callbacks available and what they are bound to
>   */
> -int do_env_callback(struct cmd_tbl *cmdtp, int flag, int argc,
> -                   char *const argv[])
> +static int do_env_callback(struct cmd_tbl *cmdtp, int flag, int argc,
> +                          char *const argv[])
>  {
>         struct env_clbk_tbl *clbkp;
>         int i;
> @@ -381,7 +381,7 @@ static int print_active_flags(struct env_entry *entry)
>  /*
>   * Print the flags available and what variables have flags
>   */
> -int do_env_flags(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
> +static int do_env_flags(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
>  {
>         /* Print the available variable types */
>         printf("Available variable type flags (position %d):\n",
> --
> 2.43.0.rc1.1336.g36b5255a03ac
>