[PATCH] console: Use printascii for lowlevel debug string

Sverdlin, Alexander alexander.sverdlin at siemens.com
Fri Nov 8 10:04:31 CET 2024


Hi Kever,

On Fri, 2024-11-08 at 16:36 +0800, Kever Yang wrote:
> The lowlevel debug with CONFIG_DEBUG_UART has printascii for string
> print, and no need to convert to use printch now.
> 
> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>

LGTM,
Reviewed-by: Alexander Sverdlin <alexander.sverdlin at siemens.com>

> ---
> 
>  common/console.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/common/console.c b/common/console.c
> index 22224701e45..863ac6aa9dc 100644
> --- a/common/console.c
> +++ b/common/console.c
> @@ -745,11 +745,7 @@ void puts(const char *s)
>  	}
>  
>  	if (IS_ENABLED(CONFIG_DEBUG_UART) && !(gd->flags & GD_FLG_SERIAL_READY)) {
> -		while (*s) {
> -			int ch = *s++;
> -
> -			printch(ch);
> -		}
> +		printascii(s);
>  		return;
>  	}

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com


More information about the U-Boot mailing list