[U-Boot] [PATCHv2 2/2] stdio_names: Ensure MAX_NAMES is defined before use, don't use 3 directly

[Peter Robinson <pbrobinson@gmail.com>]

On Thu, May 3, 2018 at 2:12 PM, Tom Rini <trini@konsulko.com> wrote:
> With tighter build flags the fact that <stdio_dev.h> doesn't have a
> reference back to MAX_NAMES causes an error.  Include <stdio.h> here and
> then in common/console.c use MAX_NAMES rather than 3 when working with
> stdio_names.
>
> Reported-by: Peter Robinson <pbrobinson@gmail.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
> Changes in v2:
> - New patch
> ---
>  common/console.c    | 4 ++--
>  include/stdio_dev.h | 1 +
>  2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/common/console.c b/common/console.c
> index 0e0295514b21..f1a5e95c8f39 100644
> --- a/common/console.c
> +++ b/common/console.c
> @@ -847,7 +847,7 @@ done:
>
>  #ifdef CONFIG_SYS_CONSOLE_ENV_OVERWRITE
>         /* set the environment variables (will overwrite previous env settings) */
> -       for (i = 0; i < 3; i++) {
> +       for (i = 0; i < MAX_NAMES; i++) {

I think you mean MAX_FILES? With MAX_NAMES it fails, with MAX_FILES it builds.

With the that change, if it's correct:
Tested-by: Peter Robinson <pbrobinson@gmail.com>

>                 env_set(stdio_names[i], stdio_devices[i]->name);
>         }
>  #endif /* CONFIG_SYS_CONSOLE_ENV_OVERWRITE */
> @@ -926,7 +926,7 @@ int console_init_r(void)
>  #endif /* CONFIG_SYS_CONSOLE_INFO_QUIET */
>
>         /* Setting environment variables */
> -       for (i = 0; i < 3; i++) {
> +       for (i = 0; i < MAX_NAMES; i++) {
>                 env_set(stdio_names[i], stdio_devices[i]->name);
>         }
>
> diff --git a/include/stdio_dev.h b/include/stdio_dev.h
> index 1ea8bff47bab..c2a88b4fc416 100644
> --- a/include/stdio_dev.h
> +++ b/include/stdio_dev.h
> @@ -8,6 +8,7 @@
>  #ifndef _STDIO_DEV_H_
>  #define _STDIO_DEV_H_
>
> +#include <stdio.h>
>  #include <linux/list.h>
>
>  /*
> --
> 2.7.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot