diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-06-06 02:04:42 -0400 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2014-06-06 02:04:42 -0400 |
commit | 72208ee060635dfab2b3bd447a95e0f9c419e954 (patch) | |
tree | 964fbcfbd497714f5b7a19f749cdc79d1c94ae7d | |
parent | 36de95567f910f5544060f50346d8677ae13ad22 (diff) |
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>
-rw-r--r-- | fs/xfs/xfs_bit.h | 7 |
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) | |||
66 | n = ffs(w); | 66 | n = ffs(w); |
67 | } else { /* upper bits */ | 67 | } else { /* upper bits */ |
68 | w = (__uint32_t)(v >> 32); | 68 | w = (__uint32_t)(v >> 32); |
69 | if (w && (n = ffs(w))) | 69 | if (w) { |
70 | n += 32; | 70 | n = ffs(w); |
71 | if (n) | ||
72 | n += 32; | ||
73 | } | ||
71 | } | 74 | } |
72 | return n - 1; | 75 | return n - 1; |
73 | } | 76 | } |