[PATCH v2 3/7] IOMUX: move search_device() to console.h

Simon Glass sjg at chromium.org
Sat Dec 19 03:29:17 CET 2020


On Wed, 16 Dec 2020 at 16:16, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> search_device() is defined in console.c. Move its declaration
> to an appropriate header file.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> ---
> v2: new patch
>  include/console.h | 2 ++
>  include/iomux.h   | 1 -
>  2 files changed, 2 insertions(+), 1 deletion(-)

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

>
> diff --git a/include/console.h b/include/console.h
> index 233ff323e1ee..79f9534a9535 100644
> --- a/include/console.h
> +++ b/include/console.h
> @@ -25,6 +25,8 @@ void clear_ctrlc(void);       /* clear the Control-C condition */
>  int disable_ctrlc(int);        /* 1 to disable, 0 to enable Control-C detect */
>  int confirm_yesno(void);        /*  1 if input is "y", "Y", "yes" or "YES" */
>
> +struct stdio_dev *search_device(int flags, const char *name);
> +
>  #ifdef CONFIG_CONSOLE_RECORD
>  /**
>   * console_record_init() - set up the console recording buffers
> diff --git a/include/iomux.h b/include/iomux.h
> index e6e1097db5b2..da7ff697d218 100644
> --- a/include/iomux.h
> +++ b/include/iomux.h
> @@ -26,6 +26,5 @@ extern int cd_count[MAX_FILES];
>
>  int iomux_doenv(const int, const char *);
>  void iomux_printdevs(const int);
> -struct stdio_dev *search_device(int, const char *);

Please add a full function comment

Also consider renaming it to console_search_dev() or similar, since it
is in console.h

>
>  #endif /* _IO_MUX_H */
> --
> 2.29.2
>

Regards,
Simon


More information about the U-Boot mailing list