staging: lustre: o2iblnd: fix misleading indentation
authorLuis de Bethencourt <luisbg@osg.samsung.com>
Wed, 21 Oct 2015 17:40:40 +0000 (18:40 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 25 Oct 2015 01:52:21 +0000 (18:52 -0700)
The code is correct, the indentation is misleading. Only the the return rc
is part of the conditional statement if rc != 0.

Fix a smatch warning:
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c:2157
kiblnd_hdev_setup_mrs() warn: curly braces intended?

Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c

index c2cc4e4..7c730e3 100644 (file)
@@ -2154,23 +2154,23 @@ static int kiblnd_hdev_setup_mrs(kib_hca_dev_t *hdev)
        if (rc != 0)
                return rc;
 
-               LIBCFS_ALLOC(hdev->ibh_mrs, 1 * sizeof(*hdev->ibh_mrs));
-               if (hdev->ibh_mrs == NULL) {
-                       CERROR("Failed to allocate MRs table\n");
-                       return -ENOMEM;
-               }
+       LIBCFS_ALLOC(hdev->ibh_mrs, 1 * sizeof(*hdev->ibh_mrs));
+       if (hdev->ibh_mrs == NULL) {
+               CERROR("Failed to allocate MRs table\n");
+               return -ENOMEM;
+       }
 
-               hdev->ibh_mrs[0] = NULL;
-               hdev->ibh_nmrs   = 1;
+       hdev->ibh_mrs[0] = NULL;
+       hdev->ibh_nmrs   = 1;
 
-               mr = ib_get_dma_mr(hdev->ibh_pd, acflags);
-               if (IS_ERR(mr)) {
-                       CERROR("Failed ib_get_dma_mr : %ld\n", PTR_ERR(mr));
-                       kiblnd_hdev_cleanup_mrs(hdev);
-                       return PTR_ERR(mr);
-               }
+       mr = ib_get_dma_mr(hdev->ibh_pd, acflags);
+       if (IS_ERR(mr)) {
+               CERROR("Failed ib_get_dma_mr : %ld\n", PTR_ERR(mr));
+               kiblnd_hdev_cleanup_mrs(hdev);
+               return PTR_ERR(mr);
+       }
 
-               hdev->ibh_mrs[0] = mr;
+       hdev->ibh_mrs[0] = mr;
 
        return 0;
 }