Linux cryptographic layer development
 help / color / mirror / Atom feed
* [PATCH] crypto: ccp - don't abuse kernel-doc comment format
@ 2026-07-30  5:17 Randy Dunlap
  2026-07-30 15:50 ` Tom Lendacky
  2026-08-10  8:30 ` Herbert Xu
  0 siblings, 2 replies; 3+ messages in thread
From: Randy Dunlap @ 2026-07-30  5:17 UTC (permalink / raw)
  To: linux-kernel
  Cc: Randy Dunlap, Ashish Kalra, Borislav Petkov (AMD), Tom Lendacky,
	Herbert Xu, David S. Miller, linux-crypto

Use plain C "/*" notation for comments that are not in kernel-doc format
to avoid kernel-doc warnings:

Warning: include/uapi/linux/psp-sfs.h:18 expecting prototype for SFS().
 Prototype was for PAYLOAD_NAME_SIZE() instead
Warning: include/uapi/linux/psp-sfs.h:46 This comment starts with '/**',
 but isn't a kernel-doc comment.
 * Seamless Firmware Support (SFS) IOC

Fixes: 648dbccc03a0 ("crypto: ccp - Add AMD Seamless Firmware Servicing (SFS) driver")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
---
Cc: Ashish Kalra <ashish.kalra@amd.com>
Cc: Borislav Petkov (AMD) <bp@alien8.de>
Cc: Tom Lendacky <thomas.lendacky@amd.com>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: linux-crypto@vger.kernel.org

 include/uapi/linux/psp-sfs.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

--- linux-next-20260729.orig/include/uapi/linux/psp-sfs.h
+++ linux-next-20260729/include/uapi/linux/psp-sfs.h
@@ -12,7 +12,7 @@
 
 #include <linux/types.h>
 
-/**
+/*
  * SFS: AMD Seamless Firmware Support (SFS) interface
  */
 
@@ -43,7 +43,7 @@ struct sfs_user_update_package {
 	__u32	sfs_extended_status;
 } __packed;
 
-/**
+/*
  * Seamless Firmware Support (SFS) IOC
  *
  * possible return codes for all SFS IOCTLs:

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

* Re: [PATCH] crypto: ccp - don't abuse kernel-doc comment format
  2026-07-30  5:17 [PATCH] crypto: ccp - don't abuse kernel-doc comment format Randy Dunlap
@ 2026-07-30 15:50 ` Tom Lendacky
  2026-08-10  8:30 ` Herbert Xu
  1 sibling, 0 replies; 3+ messages in thread
From: Tom Lendacky @ 2026-07-30 15:50 UTC (permalink / raw)
  To: Randy Dunlap, linux-kernel
  Cc: Ashish Kalra, Borislav Petkov (AMD), Herbert Xu, David S. Miller,
	linux-crypto

On 7/30/26 00:17, Randy Dunlap wrote:
> Use plain C "/*" notation for comments that are not in kernel-doc format
> to avoid kernel-doc warnings:
> 
> Warning: include/uapi/linux/psp-sfs.h:18 expecting prototype for SFS().
>  Prototype was for PAYLOAD_NAME_SIZE() instead
> Warning: include/uapi/linux/psp-sfs.h:46 This comment starts with '/**',
>  but isn't a kernel-doc comment.
>  * Seamless Firmware Support (SFS) IOC
> 
> Fixes: 648dbccc03a0 ("crypto: ccp - Add AMD Seamless Firmware Servicing (SFS) driver")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>

Acked-by: Tom Lendacky <thomas.lendacky@amd.com>

> ---
> Cc: Ashish Kalra <ashish.kalra@amd.com>
> Cc: Borislav Petkov (AMD) <bp@alien8.de>
> Cc: Tom Lendacky <thomas.lendacky@amd.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: linux-crypto@vger.kernel.org
> 
>  include/uapi/linux/psp-sfs.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> --- linux-next-20260729.orig/include/uapi/linux/psp-sfs.h
> +++ linux-next-20260729/include/uapi/linux/psp-sfs.h
> @@ -12,7 +12,7 @@
>  
>  #include <linux/types.h>
>  
> -/**
> +/*
>   * SFS: AMD Seamless Firmware Support (SFS) interface
>   */
>  
> @@ -43,7 +43,7 @@ struct sfs_user_update_package {
>  	__u32	sfs_extended_status;
>  } __packed;
>  
> -/**
> +/*
>   * Seamless Firmware Support (SFS) IOC
>   *
>   * possible return codes for all SFS IOCTLs:


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

* Re: [PATCH] crypto: ccp - don't abuse kernel-doc comment format
  2026-07-30  5:17 [PATCH] crypto: ccp - don't abuse kernel-doc comment format Randy Dunlap
  2026-07-30 15:50 ` Tom Lendacky
@ 2026-08-10  8:30 ` Herbert Xu
  1 sibling, 0 replies; 3+ messages in thread
From: Herbert Xu @ 2026-08-10  8:30 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: linux-kernel, Ashish Kalra, Borislav Petkov (AMD), Tom Lendacky,
	David S. Miller, linux-crypto

On Wed, Jul 29, 2026 at 10:17:10PM -0700, Randy Dunlap wrote:
> Use plain C "/*" notation for comments that are not in kernel-doc format
> to avoid kernel-doc warnings:
> 
> Warning: include/uapi/linux/psp-sfs.h:18 expecting prototype for SFS().
>  Prototype was for PAYLOAD_NAME_SIZE() instead
> Warning: include/uapi/linux/psp-sfs.h:46 This comment starts with '/**',
>  but isn't a kernel-doc comment.
>  * Seamless Firmware Support (SFS) IOC
> 
> Fixes: 648dbccc03a0 ("crypto: ccp - Add AMD Seamless Firmware Servicing (SFS) driver")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> ---
> Cc: Ashish Kalra <ashish.kalra@amd.com>
> Cc: Borislav Petkov (AMD) <bp@alien8.de>
> Cc: Tom Lendacky <thomas.lendacky@amd.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: linux-crypto@vger.kernel.org
> 
>  include/uapi/linux/psp-sfs.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Patch applied.  Thanks.
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

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

end of thread, other threads:[~2026-08-10  8:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-30  5:17 [PATCH] crypto: ccp - don't abuse kernel-doc comment format Randy Dunlap
2026-07-30 15:50 ` Tom Lendacky
2026-08-10  8:30 ` Herbert Xu

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