summaryrefslogtreecommitdiffstats
path: root/fs/xfs
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2014-06-06 16:04:42 +1000
committerDave Chinner <david@fromorbit.com>2014-06-06 16:04:42 +1000
commit72208ee060635dfab2b3bd447a95e0f9c419e954 (patch)
tree964fbcfbd497714f5b7a19f749cdc79d1c94ae7d /fs/xfs
parent36de95567f910f5544060f50346d8677ae13ad22 (diff)
downloadlinux-72208ee060635dfab2b3bd447a95e0f9c419e954.tar.gz
linux-72208ee060635dfab2b3bd447a95e0f9c419e954.tar.bz2
linux-72208ee060635dfab2b3bd447a95e0f9c419e954.zip
xfs: small cleanup in xfs_lowbit64()
There are two checkpatch.pl complaints here because of the bad indenting and because of the assignment inside the condition. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Eric Sandeen <sandeen@redhat.com> Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs')
-rw-r--r--fs/xfs/xfs_bit.h7
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/xfs/xfs_bit.h b/fs/xfs/xfs_bit.h
index f1e3c907044d..e1649c0d3e02 100644
--- a/fs/xfs/xfs_bit.h
+++ b/fs/xfs/xfs_bit.h
@@ -66,8 +66,11 @@ static inline int xfs_lowbit64(__uint64_t v)
n = ffs(w);
} else { /* upper bits */
w = (__uint32_t)(v >> 32);
- if (w && (n = ffs(w)))
- n += 32;
+ if (w) {
+ n = ffs(w);
+ if (n)
+ n += 32;
+ }
}
return n - 1;
}