Re: [PATCH] staging: lustre: Fix coding style. Switch and case at the same indent.

2014-10-29 Thread Greg KH 
On Tue, Oct 28, 2014 at 09:35:15PM +0200, Evaldas Palaima wrote:
 Fix a style issue reported by checkpatch.pl for the Eudyptula challenge.
 
 Signed-off-by: Evaldas Palaima evaldas.pala...@gmail.com
 ---
  drivers/staging/lustre/lustre/libcfs/fail.c | 24 
  1 file changed, 12 insertions(+), 12 deletions(-)
 
 diff --git a/drivers/staging/lustre/lustre/libcfs/fail.c 
 b/drivers/staging/lustre/lustre/libcfs/fail.c
 index e73ca3d..92444b0 100644
 --- a/drivers/staging/lustre/lustre/libcfs/fail.c
 +++ b/drivers/staging/lustre/lustre/libcfs/fail.c
 @@ -103,18 +103,18 @@ int __cfs_fail_check_set(__u32 id, __u32 value, int set)
   }
  
   switch (set) {
 - case CFS_FAIL_LOC_NOSET:
 - case CFS_FAIL_LOC_VALUE:
 - break;
 - case CFS_FAIL_LOC_ORSET:
 - cfs_fail_loc |= value  ~(CFS_FAILED | CFS_FAIL_ONCE);
 - break;
 - case CFS_FAIL_LOC_RESET:
 - cfs_fail_loc = value;
 - break;
 - default:
 - LASSERTF(0, called with bad set %u\n, set);
 - break;
 + case CFS_FAIL_LOC_NOSET:
 + case CFS_FAIL_LOC_VALUE:
 + break;
 + case CFS_FAIL_LOC_ORSET:
 + cfs_fail_loc |= value  ~(CFS_FAILED | CFS_FAIL_ONCE);
 + break;
 + case CFS_FAIL_LOC_RESET:
 + cfs_fail_loc = value;
 + break;
 + default:
 + LASSERTF(0, called with bad set %u\n, set);
 + break;
   }
  
   return 1;

Someone else did this change before you, sorry :(
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: lustre: Fix coding style. Switch and case at the same indent.

2014-10-28 Thread Evaldas Palaima 
Fix a style issue reported by checkpatch.pl for the Eudyptula challenge.

Signed-off-by: Evaldas Palaima evaldas.pala...@gmail.com
---
 drivers/staging/lustre/lustre/libcfs/fail.c | 24 
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/drivers/staging/lustre/lustre/libcfs/fail.c 
b/drivers/staging/lustre/lustre/libcfs/fail.c
index e73ca3d..92444b0 100644
--- a/drivers/staging/lustre/lustre/libcfs/fail.c
+++ b/drivers/staging/lustre/lustre/libcfs/fail.c
@@ -103,18 +103,18 @@ int __cfs_fail_check_set(__u32 id, __u32 value, int set)
}
 
switch (set) {
-   case CFS_FAIL_LOC_NOSET:
-   case CFS_FAIL_LOC_VALUE:
-   break;
-   case CFS_FAIL_LOC_ORSET:
-   cfs_fail_loc |= value  ~(CFS_FAILED | CFS_FAIL_ONCE);
-   break;
-   case CFS_FAIL_LOC_RESET:
-   cfs_fail_loc = value;
-   break;
-   default:
-   LASSERTF(0, called with bad set %u\n, set);
-   break;
+   case CFS_FAIL_LOC_NOSET:
+   case CFS_FAIL_LOC_VALUE:
+   break;
+   case CFS_FAIL_LOC_ORSET:
+   cfs_fail_loc |= value  ~(CFS_FAILED | CFS_FAIL_ONCE);
+   break;
+   case CFS_FAIL_LOC_RESET:
+   cfs_fail_loc = value;
+   break;
+   default:
+   LASSERTF(0, called with bad set %u\n, set);
+   break;
}
 
return 1;
-- 
2.1.2

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel