Re: [PATCH 0/4] exfat: minor cleanup changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2021-11-03 6:23 GMT+09:00, Christophe Vu-Brugier <cvubrugier@xxxxxxxxxxx>:
> From: Christophe Vu-Brugier <christophe.vu-brugier@xxxxxxxxxxx>
>
> Hi,
Hi Christophe,
>
> These patches contain a few minor changes I wrote while studying the
> exFAT file system driver.
There is a warning from checkpatch.pl. Please run it before sending
patch to list next time.

WARNING: Comparisons should place the constant on the right side of the test
#105: FILE: fs/exfat/fatent.c:87:
+	return EXFAT_FIRST_CLUSTER <= clus && clus < sbi->num_clusters;

I fixed directly this warning and applied to #dev branch.

Thanks for your patch!
>
> With best regards,
>
> Christophe Vu-Brugier (4):
>   exfat: simplify is_valid_cluster()
>   exfat: fix typos in comments
>   exfat: make exfat_find_location() static
>   exfat: reuse exfat_inode_info variable instead of calling EXFAT_I()
>
>  fs/exfat/dir.c      |  6 +++---
>  fs/exfat/exfat_fs.h |  2 --
>  fs/exfat/fatent.c   |  4 +---
>  fs/exfat/file.c     | 14 +++++++-------
>  fs/exfat/inode.c    | 11 +++++------
>  fs/exfat/namei.c    |  6 +++---
>  fs/exfat/super.c    |  6 +++---
>  7 files changed, 22 insertions(+), 27 deletions(-)
>
> --
> 2.20.1
>
>



[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux