[PATCH] lib: utils/serial: Optimize semihosting_putc implementation

[Xiang W]

在 2023-10-17星期二的 19:53 +0800，cp0613 at linux.alibaba.com写道：
> From: Chen Pei <cp0613 at linux.alibaba.com>
> 
> For some debuggers that do not implement SYSWRITEC and SYSREADC
> operations, we can use SYSWRITE and SYSREAD instead like the
> implementation of semihosting_getc().
> 
> Signed-off-by: Chen Pei <cp0613 at linux.alibaba.com>
> ---
>  lib/utils/serial/semihosting.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/utils/serial/semihosting.c b/lib/utils/serial/semihosting.c
> index ce65887..9bbf2e2 100644
> --- a/lib/utils/serial/semihosting.c
> +++ b/lib/utils/serial/semihosting.c
> @@ -162,7 +162,10 @@ static long semihosting_write(long fd, const void *memp, size_t len)
>  
>  static void semihosting_putc(char ch)
>  {
> -	semihosting_trap(SYSWRITEC, &ch);
> +	if (semihosting_outfd < 0)
> +		semihosting_trap(SYSWRITEC, &ch);
> +	else
> +		semihosting_write(semihosting_outfd, &ch, 1);
The code can be simplified by calling semihosting_puts directly, otherwise look good to me.

Reviewed-by: Xiang W <wxjstz at 126.com>
>  }
>  
>  static unsigned long semihosting_puts(const char *str, unsigned long len)
> -- 
> 2.25.1
> 
>