Re: [PATCH] powerpc/boot: Remove unused sprintf()

["Christophe Leroy (CS GROUP)" <chleroy@kernel.org>]

Le 24/07/2026 à 19:21, Thorsten Blum a écrit :
> There have been no sprintf() callers in the boot wrapper since commit
> e275e023aa69 ("powerpc/44x: Warp patches for the new NDFC driver").
> 
> Remove the function definition and declaration.
> 
> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>

Reviewed-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

> ---
>   arch/powerpc/boot/stdio.c | 11 -----------
>   arch/powerpc/boot/stdio.h |  3 ---
>   2 files changed, 14 deletions(-)
> 
> diff --git a/arch/powerpc/boot/stdio.c b/arch/powerpc/boot/stdio.c
> index 31eece29f56d..b12aa2c0b8e7 100644
> --- a/arch/powerpc/boot/stdio.c
> +++ b/arch/powerpc/boot/stdio.c
> @@ -326,17 +326,6 @@ int vsprintf(char *buf, const char *fmt, va_list args)
>   	return str-buf;
>   }
>   
> -int sprintf(char * buf, const char *fmt, ...)
> -{
> -	va_list args;
> -	int i;
> -
> -	va_start(args, fmt);
> -	i=vsprintf(buf,fmt,args);
> -	va_end(args);
> -	return i;
> -}
> -
>   static char sprint_buf[1024];
>   
>   int
> diff --git a/arch/powerpc/boot/stdio.h b/arch/powerpc/boot/stdio.h
> index 884d5959a9ae..d5a7a0f54453 100644
> --- a/arch/powerpc/boot/stdio.h
> +++ b/arch/powerpc/boot/stdio.h
> @@ -12,9 +12,6 @@ extern int printf(const char *fmt, ...) __attribute__((format(printf, 1, 2)));
>   
>   #define fprintf(fmt, args...)	printf(args)
>   
> -extern int sprintf(char *buf, const char *fmt, ...)
> -	__attribute__((format(printf, 2, 3)));
> -
>   extern int vsprintf(char *buf, const char *fmt, va_list args);
>   
>   #endif				/* _PPC_BOOT_STDIO_H_ */