staging: lustre: include: linux: libcfs: Removed unnecessary else expression.
authorGulsah Kose <gulsah.1004@gmail.com>
Tue, 30 Sep 2014 18:59:21 +0000 (21:59 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Oct 2014 16:56:45 +0000 (09:56 -0700)
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>
drivers/staging/lustre/include/linux/libcfs/libcfs.h

index 1122ae963677640ce7ee79b78fa268e151fcff37..a6b2f906bb1ab49fa2c84b31791db6f83f587d7c 100644 (file)
@@ -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) \