[U-Boot] [PATCH 1/2] common/console.c: Remove unused inline functions
Bin Meng
bmeng.cn at gmail.com
Thu Jan 21 04:35:46 CET 2016
Hi Tom,
On Wed, Jan 20, 2016 at 8:23 PM, Tom Rini <trini at konsulko.com> wrote:
> clang-3.8 reports that these functions are unused, remove them.
>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> common/console.c | 48 ------------------------------------------------
> 1 file changed, 48 deletions(-)
>
> diff --git a/common/console.c b/common/console.c
> index bc37b6d..b3f5fdc 100644
> --- a/common/console.c
> +++ b/common/console.c
> @@ -227,11 +227,6 @@ static void console_puts(int file, const char *s)
> }
> }
>
> -static inline void console_printdevs(int file)
> -{
> - iomux_printdevs(file);
> -}
> -
> static inline void console_doenv(int file, struct stdio_dev *dev)
> {
> iomux_doenv(file, dev->name);
> @@ -265,11 +260,6 @@ static inline void console_puts(int file, const char *s)
> stdio_devices[file]->puts(stdio_devices[file], s);
> }
>
> -static inline void console_printdevs(int file)
> -{
> - printf("%s\n", stdio_devices[file]->name);
> -}
> -
> static inline void console_doenv(int file, struct stdio_dev *dev)
> {
> console_setfile(file, dev);
> @@ -653,44 +643,6 @@ void clear_ctrlc(void)
> ctrlc_was_pressed = 0;
> }
>
> -#ifdef CONFIG_MODEM_SUPPORT_DEBUG
> -char screen[1024];
> -char *cursor = screen;
> -int once = 0;
> -inline void dbg(const char *fmt, ...)
> -{
> - va_list args;
> - uint i;
> - char printbuffer[CONFIG_SYS_PBSIZE];
> -
> - if (!once) {
> - memset(screen, 0, sizeof(screen));
> - once++;
> - }
> -
> - va_start(args, fmt);
> -
> - /* For this to work, printbuffer must be larger than
> - * anything we ever want to print.
> - */
> - i = vsnprintf(printbuffer, sizeof(printbuffer), fmt, args);
> - va_end(args);
> -
> - if ((screen + sizeof(screen) - 1 - cursor)
> - < strlen(printbuffer) + 1) {
> - memset(screen, 0, sizeof(screen));
> - cursor = screen;
> - }
> - sprintf(cursor, printbuffer);
> - cursor += strlen(printbuffer);
> -
> -}
> -#else
> -static inline void dbg(const char *fmt, ...)
> -{
> -}
> -#endif
> -
> /** U-Boot INIT FUNCTIONS *************************************************/
>
> struct stdio_dev *search_device(int flags, const char *name)
> --
Please remove one more instance of CONFIG_MODEM_SUPPORT_DEBUG in README.
Other than that,
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
Regards,
Bin
More information about the U-Boot
mailing list