diff options
| author |   Gulsah Kose <gulsah.1004@gmail.com> | 2014-09-30 21:59:21 +0300 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-10-02 09:56:45 -0700 |
| commit | fa44eae464430475334264e3045d42b708c5a6d7 (patch) | |
| tree | 9c7040923e87873b7b2886798f9d65d65b4e14c8 /drivers/staging | |
| parent | staging: lustre: include: linux: libcfs: linux: Used "linux" instead of "asm". (diff) | |
| download | linux-dev-fa44eae464430475334264e3045d42b708c5a6d7.tar.xz linux-dev-fa44eae464430475334264e3045d42b708c5a6d7.zip | |
staging: lustre: include: linux: libcfs: Removed unnecessary else expression.
This patch fixes "else is not generally useful after a break or return"
checkpatch.pl warning libcfs.h
Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging')
| -rw-r--r-- | drivers/staging/lustre/include/linux/libcfs/libcfs.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs.h b/drivers/staging/lustre/include/linux/libcfs/libcfs.h index 1122ae963677..a6b2f906bb1a 100644 --- a/drivers/staging/lustre/include/linux/libcfs/libcfs.h +++ b/drivers/staging/lustre/include/linux/libcfs/libcfs.h @@ -173,8 +173,7 @@ static inline void *__container_of(void *ptr, unsigned long shift) { if (unlikely(IS_ERR(ptr) || ptr == NULL)) return ptr; - else - return (char *)ptr - shift; + return (char *)ptr - shift; } #define container_of0(ptr, type, member) \ |