[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 10 Nov 2021 17:49:55 -0800
From: Jakub Kicinski <kuba@...nel.org>
To: Randy Dunlap <rdunlap@...radead.org>
Cc: netdev@...r.kernel.org, kernel test robot <lkp@...el.com>,
Min Li <min.li.xe@...esas.com>,
Richard Cochran <richardcochran@...il.com>,
"David S. Miller" <davem@...emloft.net>
Subject: Re: [PATCH] ptp: ptp_clockmatrix: repair non-kernel-doc comment
On Wed, 10 Nov 2021 14:53:06 -0800 Randy Dunlap wrote:
> Do not use "/**" to begin a comment that is not in kernel-doc format.
>
> Prevents this docs build warning:
>
> drivers/ptp/ptp_clockmatrix.c:1679: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> * Maximum absolute value for write phase offset in picoseconds
>
> Fixes: 794c3dffacc16 ("ptp: ptp_clockmatrix: Add support for FW 5.2 (8A34005)")
> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
> Reported-by: kernel test robot <lkp@...el.com>
> Cc: Min Li <min.li.xe@...esas.com>
> Cc: Richard Cochran <richardcochran@...il.com>
> Cc: "David S. Miller" <davem@...emloft.net>
> Cc: Jakub Kicinski <kuba@...nel.org>
> ---
> drivers/ptp/ptp_clockmatrix.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-next-20211110.orig/drivers/ptp/ptp_clockmatrix.c
> +++ linux-next-20211110/drivers/ptp/ptp_clockmatrix.c
> @@ -1699,7 +1699,7 @@ static int initialize_dco_operating_mode
>
> /* PTP Hardware Clock interface */
>
> -/**
> +/*
> * Maximum absolute value for write phase offset in picoseconds
> *
> * @channel: channel
Looks like it documents parameters to the function, should we either
fix it to make it valid kdoc or remove the params (which TBH aren't
really adding much value)?
Powered by blists - more mailing lists