The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [PATCH] ntfs: dir.c: fix kernel-doc function parameter warnings
@ 2023-12-19  4:54 Randy Dunlap
  2023-12-19  7:45 ` Namjae Jeon
  2023-12-19 13:07 ` Christian Brauner
  0 siblings, 2 replies; 3+ messages in thread
From: Randy Dunlap @ 2023-12-19  4:54 UTC (permalink / raw)
  To: linux-kernel
  Cc: Randy Dunlap, Anton Altaparmakov, Namjae Jeon, linux-ntfs-dev,
	Andrew Morton

Correct the kernel-doc function parameter warnings for function
ntfs_dir_fsync() to prevent the following kernel-doc warnings:

dir.c:1489: warning: Function parameter or member 'start' not described in 'ntfs_dir_fsync'
dir.c:1489: warning: Function parameter or member 'end' not described in 'ntfs_dir_fsync'
dir.c:1489: warning: Excess function parameter 'dentry' description in 'ntfs_dir_fsync'

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Anton Altaparmakov <anton@tuxera.com>
Cc: Namjae Jeon <linkinjeon@kernel.org>
Cc: linux-ntfs-dev@lists.sourceforge.net
Cc: Andrew Morton <akpm@linux-foundation.org>
---
 fs/ntfs/dir.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff -- a/fs/ntfs/dir.c b/fs/ntfs/dir.c
--- a/fs/ntfs/dir.c
+++ b/fs/ntfs/dir.c
@@ -1462,7 +1462,8 @@ static int ntfs_dir_open(struct inode *v
 /**
  * ntfs_dir_fsync - sync a directory to disk
  * @filp:	directory to be synced
- * @dentry:	dentry describing the directory to sync
+ * @start:	offset in bytes of the beginning of data range to sync
+ * @end:	offset in bytes of the end of data range (inclusive)
  * @datasync:	if non-zero only flush user data and not metadata
  *
  * Data integrity sync of a directory to disk.  Used for fsync, fdatasync, and

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

* Re: [PATCH] ntfs: dir.c: fix kernel-doc function parameter warnings
  2023-12-19  4:54 [PATCH] ntfs: dir.c: fix kernel-doc function parameter warnings Randy Dunlap
@ 2023-12-19  7:45 ` Namjae Jeon
  2023-12-19 13:07 ` Christian Brauner
  1 sibling, 0 replies; 3+ messages in thread
From: Namjae Jeon @ 2023-12-19  7:45 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: linux-kernel, Anton Altaparmakov, linux-ntfs-dev, Andrew Morton,
	Christian Brauner

2023-12-19 13:54 GMT+09:00, Randy Dunlap <rdunlap@infradead.org>:
> Correct the kernel-doc function parameter warnings for function
> ntfs_dir_fsync() to prevent the following kernel-doc warnings:
>
> dir.c:1489: warning: Function parameter or member 'start' not described in
> 'ntfs_dir_fsync'
> dir.c:1489: warning: Function parameter or member 'end' not described in
> 'ntfs_dir_fsync'
> dir.c:1489: warning: Excess function parameter 'dentry' description in
> 'ntfs_dir_fsync'
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Anton Altaparmakov <anton@tuxera.com>
> Cc: Namjae Jeon <linkinjeon@kernel.org>
> Cc: linux-ntfs-dev@lists.sourceforge.net
> Cc: Andrew Morton <akpm@linux-foundation.org>
As you may already know, the same warning is probably coming from
ntfs_file_fsync() in file.c.
Reviewed-by: Namjae Jeon <linkinjeon@kernel.org>
Thanks!
> ---
>  fs/ntfs/dir.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff -- a/fs/ntfs/dir.c b/fs/ntfs/dir.c
> --- a/fs/ntfs/dir.c
> +++ b/fs/ntfs/dir.c
> @@ -1462,7 +1462,8 @@ static int ntfs_dir_open(struct inode *v
>  /**
>   * ntfs_dir_fsync - sync a directory to disk
>   * @filp:	directory to be synced
> - * @dentry:	dentry describing the directory to sync
> + * @start:	offset in bytes of the beginning of data range to sync
> + * @end:	offset in bytes of the end of data range (inclusive)
>   * @datasync:	if non-zero only flush user data and not metadata
>   *
>   * Data integrity sync of a directory to disk.  Used for fsync, fdatasync,
> and
>

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

* Re: [PATCH] ntfs: dir.c: fix kernel-doc function parameter warnings
  2023-12-19  4:54 [PATCH] ntfs: dir.c: fix kernel-doc function parameter warnings Randy Dunlap
  2023-12-19  7:45 ` Namjae Jeon
@ 2023-12-19 13:07 ` Christian Brauner
  1 sibling, 0 replies; 3+ messages in thread
From: Christian Brauner @ 2023-12-19 13:07 UTC (permalink / raw)
  To: linux-kernel, Randy Dunlap
  Cc: Christian Brauner, Anton Altaparmakov, Namjae Jeon,
	linux-ntfs-dev, Andrew Morton

On Mon, 18 Dec 2023 20:54:14 -0800, Randy Dunlap wrote:
> Correct the kernel-doc function parameter warnings for function
> ntfs_dir_fsync() to prevent the following kernel-doc warnings:
> 
> dir.c:1489: warning: Function parameter or member 'start' not described in 'ntfs_dir_fsync'
> dir.c:1489: warning: Function parameter or member 'end' not described in 'ntfs_dir_fsync'
> dir.c:1489: warning: Excess function parameter 'dentry' description in 'ntfs_dir_fsync'
> 
> [...]

Applied to the vfs.misc branch of the vfs/vfs.git tree.
Patches in the vfs.misc branch should appear in linux-next soon.

Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.

It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.

Note that commit hashes shown below are subject to change due to rebase,
trailer updates or similar. If in doubt, please check the listed branch.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.misc

[1/1] ntfs: dir.c: fix kernel-doc function parameter warnings
      https://git.kernel.org/vfs/vfs/c/be4887a964e2

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

end of thread, other threads:[~2023-12-19 13:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-19  4:54 [PATCH] ntfs: dir.c: fix kernel-doc function parameter warnings Randy Dunlap
2023-12-19  7:45 ` Namjae Jeon
2023-12-19 13:07 ` Christian Brauner

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