diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2014-06-06 16:04:42 +1000 |
|---|---|---|
| committer |   Dave Chinner <david@fromorbit.com> | 2014-06-06 16:04:42 +1000 |
| commit | 72208ee060635dfab2b3bd447a95e0f9c419e954 (patch) | |
| tree | 964fbcfbd497714f5b7a19f749cdc79d1c94ae7d /fs/xfs/xfs_bit.h | |
| parent | xfs: kill xfs_buf_geterror() (diff) | |
| download | linux-dev-72208ee060635dfab2b3bd447a95e0f9c419e954.tar.xz linux-dev-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/xfs_bit.h')
| -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) 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; } |