LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] powerpc/boot: Remove unused sprintf()
@ 2026-07-24 17:21 Thorsten Blum
  2026-07-31  7:17 ` Christophe Leroy (CS GROUP)
  0 siblings, 1 reply; 2+ messages in thread
From: Thorsten Blum @ 2026-07-24 17:21 UTC (permalink / raw)
  To: Madhavan Srinivasan, Michael Ellerman, Nicholas Piggin,
	Christophe Leroy (CS GROUP)
  Cc: Thorsten Blum, linuxppc-dev, linux-kernel

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>
---
 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_ */


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] powerpc/boot: Remove unused sprintf()
  2026-07-24 17:21 [PATCH] powerpc/boot: Remove unused sprintf() Thorsten Blum
@ 2026-07-31  7:17 ` Christophe Leroy (CS GROUP)
  0 siblings, 0 replies; 2+ messages in thread
From: Christophe Leroy (CS GROUP) @ 2026-07-31  7:17 UTC (permalink / raw)
  To: Thorsten Blum, Madhavan Srinivasan, Michael Ellerman,
	Nicholas Piggin
  Cc: linuxppc-dev, linux-kernel



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_ */



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-07-31  7:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-24 17:21 [PATCH] powerpc/boot: Remove unused sprintf() Thorsten Blum
2026-07-31  7:17 ` Christophe Leroy (CS GROUP)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox