Commit f3fe3954 authored by Yuezhang Mo's avatar Yuezhang Mo Committed by Namjae Jeon

exfat: replace magic numbers with Macros

Code refinement, no functional changes.
Signed-off-by: default avatarYuezhang Mo <Yuezhang.Mo@sony.com>
Reviewed-by: default avatarAndy Wu <Andy.Wu@sony.com>
Reviewed-by: default avatarAoyama Wataru <wataru.aoyama@sony.com>
Reviewed-by: default avatarSungjong Seo <sj1557.seo@samsung.com>
Signed-off-by: default avatarNamjae Jeon <linkinjeon@kernel.org>
parent 3b9681ac
...@@ -44,7 +44,7 @@ static void exfat_get_uniname_from_ext_entry(struct super_block *sb, ...@@ -44,7 +44,7 @@ static void exfat_get_uniname_from_ext_entry(struct super_block *sb,
* Third entry : first file-name entry * Third entry : first file-name entry
* So, the index of first file-name dentry should start from 2. * So, the index of first file-name dentry should start from 2.
*/ */
for (i = 2; i < es.num_entries; i++) { for (i = ES_IDX_FIRST_FILENAME; i < es.num_entries; i++) {
struct exfat_dentry *ep = exfat_get_dentry_cached(&es, i); struct exfat_dentry *ep = exfat_get_dentry_cached(&es, i);
/* end of name entry */ /* end of name entry */
...@@ -336,7 +336,7 @@ int exfat_calc_num_entries(struct exfat_uni_name *p_uniname) ...@@ -336,7 +336,7 @@ int exfat_calc_num_entries(struct exfat_uni_name *p_uniname)
return -EINVAL; return -EINVAL;
/* 1 file entry + 1 stream entry + name entries */ /* 1 file entry + 1 stream entry + name entries */
return ((len - 1) / EXFAT_FILE_NAME_LEN + 3); return ES_ENTRY_NUM(len);
} }
unsigned int exfat_get_entry_type(struct exfat_dentry *ep) unsigned int exfat_get_entry_type(struct exfat_dentry *ep)
...@@ -591,13 +591,13 @@ void exfat_update_dir_chksum_with_entry_set(struct exfat_entry_set_cache *es) ...@@ -591,13 +591,13 @@ void exfat_update_dir_chksum_with_entry_set(struct exfat_entry_set_cache *es)
unsigned short chksum = 0; unsigned short chksum = 0;
struct exfat_dentry *ep; struct exfat_dentry *ep;
for (i = 0; i < es->num_entries; i++) { for (i = ES_IDX_FILE; i < es->num_entries; i++) {
ep = exfat_get_dentry_cached(es, i); ep = exfat_get_dentry_cached(es, i);
chksum = exfat_calc_chksum16(ep, DENTRY_SIZE, chksum, chksum = exfat_calc_chksum16(ep, DENTRY_SIZE, chksum,
chksum_type); chksum_type);
chksum_type = CS_DEFAULT; chksum_type = CS_DEFAULT;
} }
ep = exfat_get_dentry_cached(es, 0); ep = exfat_get_dentry_cached(es, ES_IDX_FILE);
ep->dentry.file.checksum = cpu_to_le16(chksum); ep->dentry.file.checksum = cpu_to_le16(chksum);
es->modified = true; es->modified = true;
} }
...@@ -858,7 +858,7 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es, ...@@ -858,7 +858,7 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es,
return -EIO; return -EIO;
es->bh[es->num_bh++] = bh; es->bh[es->num_bh++] = bh;
ep = exfat_get_dentry_cached(es, 0); ep = exfat_get_dentry_cached(es, ES_IDX_FILE);
if (!exfat_validate_entry(exfat_get_entry_type(ep), &mode)) if (!exfat_validate_entry(exfat_get_entry_type(ep), &mode))
goto put_es; goto put_es;
...@@ -895,7 +895,7 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es, ...@@ -895,7 +895,7 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es,
} }
/* validate cached dentries */ /* validate cached dentries */
for (i = 1; i < num_entries; i++) { for (i = ES_IDX_STREAM; i < num_entries; i++) {
ep = exfat_get_dentry_cached(es, i); ep = exfat_get_dentry_cached(es, i);
if (!exfat_validate_entry(exfat_get_entry_type(ep), &mode)) if (!exfat_validate_entry(exfat_get_entry_type(ep), &mode))
goto put_es; goto put_es;
......
...@@ -44,8 +44,8 @@ int __exfat_write_inode(struct inode *inode, int sync) ...@@ -44,8 +44,8 @@ int __exfat_write_inode(struct inode *inode, int sync)
/* get the directory entry of given file or directory */ /* get the directory entry of given file or directory */
if (exfat_get_dentry_set(&es, sb, &(ei->dir), ei->entry, ES_ALL_ENTRIES)) if (exfat_get_dentry_set(&es, sb, &(ei->dir), ei->entry, ES_ALL_ENTRIES))
return -EIO; return -EIO;
ep = exfat_get_dentry_cached(&es, 0); ep = exfat_get_dentry_cached(&es, ES_IDX_FILE);
ep2 = exfat_get_dentry_cached(&es, 1); ep2 = exfat_get_dentry_cached(&es, ES_IDX_STREAM);
ep->dentry.file.attr = cpu_to_le16(exfat_make_attr(inode)); ep->dentry.file.attr = cpu_to_le16(exfat_make_attr(inode));
......
...@@ -646,8 +646,8 @@ static int exfat_find(struct inode *dir, struct qstr *qname, ...@@ -646,8 +646,8 @@ static int exfat_find(struct inode *dir, struct qstr *qname,
dentry = hint_opt.eidx; dentry = hint_opt.eidx;
if (exfat_get_dentry_set(&es, sb, &cdir, dentry, ES_2_ENTRIES)) if (exfat_get_dentry_set(&es, sb, &cdir, dentry, ES_2_ENTRIES))
return -EIO; return -EIO;
ep = exfat_get_dentry_cached(&es, 0); ep = exfat_get_dentry_cached(&es, ES_IDX_FILE);
ep2 = exfat_get_dentry_cached(&es, 1); ep2 = exfat_get_dentry_cached(&es, ES_IDX_STREAM);
info->type = exfat_get_entry_type(ep); info->type = exfat_get_entry_type(ep);
info->attr = le16_to_cpu(ep->dentry.file.attr); info->attr = le16_to_cpu(ep->dentry.file.attr);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment