Re: [PATCH] ntfs: dir.c: fix kernel-doc function parameter warnings

From: Namjae Jeon
Date: Tue Dec 19 2023 - 02:45:34 EST


2023-12-19 13:54 GMT+09:00, Randy Dunlap <rdunlap@xxxxxxxxxxxxx>:
> 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@xxxxxxxxxxxxx>
> Cc: Anton Altaparmakov <anton@xxxxxxxxxx>
> Cc: Namjae Jeon <linkinjeon@xxxxxxxxxx>
> Cc: linux-ntfs-dev@xxxxxxxxxxxxxxxxxxxxx
> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
As you may already know, the same warning is probably coming from
ntfs_file_fsync() in file.c.
Reviewed-by: Namjae Jeon <linkinjeon@xxxxxxxxxx>
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
>