Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1211474

[PATCH 4/5] staging: lustre: fix SWITCH_CASE_INDENT_LEVEL checkpatch.pl errors

Show all headers | View raw

Fix occurencies of the following checkpatch.pl error:
ERROR: switch and case should be at the same indent

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
---
 drivers/staging/lustre/lnet/selftest/selftest.h    | 20 ++++++++++----------
 .../lustre/lustre/libcfs/linux/linux-tracefile.c   | 22 ++++++++++------------
 2 files changed, 20 insertions(+), 22 deletions(-)

diff --git a/drivers/staging/lustre/lnet/selftest/selftest.h b/drivers/staging/lustre/lnet/selftest/selftest.h
index dfe3342..160d68c 100644
--- a/drivers/staging/lustre/lnet/selftest/selftest.h
+++ b/drivers/staging/lustre/lnet/selftest/selftest.h
@@ -563,16 +563,16 @@ swi_state2str (int state)
 {
 #define STATE2STR(x) case x: return #x
 	switch (state) {
-		default:
-			LBUG();
-		STATE2STR(SWI_STATE_NEWBORN);
-		STATE2STR(SWI_STATE_REPLY_SUBMITTED);
-		STATE2STR(SWI_STATE_REPLY_SENT);
-		STATE2STR(SWI_STATE_REQUEST_SUBMITTED);
-		STATE2STR(SWI_STATE_REQUEST_SENT);
-		STATE2STR(SWI_STATE_REPLY_RECEIVED);
-		STATE2STR(SWI_STATE_BULK_STARTED);
-		STATE2STR(SWI_STATE_DONE);
+	default:
+		LBUG();
+	STATE2STR(SWI_STATE_NEWBORN);
+	STATE2STR(SWI_STATE_REPLY_SUBMITTED);
+	STATE2STR(SWI_STATE_REPLY_SENT);
+	STATE2STR(SWI_STATE_REQUEST_SUBMITTED);
+	STATE2STR(SWI_STATE_REQUEST_SENT);
+	STATE2STR(SWI_STATE_REPLY_RECEIVED);
+	STATE2STR(SWI_STATE_BULK_STARTED);
+	STATE2STR(SWI_STATE_DONE);
 	}
 #undef STATE2STR
 }
diff --git a/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c b/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c
index eb10e3b..87d8449 100644
--- a/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c
+++ b/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c
@@ -212,12 +212,11 @@ static char *
 dbghdr_to_err_string(struct ptldebug_header *hdr)
 {
 	switch (hdr->ph_subsys) {
-
-		case S_LND:
-		case S_LNET:
-			return "LNetError";
-		default:
-			return "LustreError";
+	case S_LND:
+	case S_LNET:
+		return "LNetError";
+	default:
+		return "LustreError";
 	}
 }
 
@@ -225,12 +224,11 @@ static char *
 dbghdr_to_info_string(struct ptldebug_header *hdr)
 {
 	switch (hdr->ph_subsys) {
-
-		case S_LND:
-		case S_LNET:
-			return "LNet";
-		default:
-			return "Lustre";
+	case S_LND:
+	case S_LNET:
+		return "LNet";
+	default:
+		return "Lustre";
 	}
 }
 
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/5] staging: lustre: fix some trivial checpatck.pl errors Mike Rapoport <mike.rapoport@gmail.com> - 2015-08-22 16:20 +0200
  [PATCH 3/5] staging: lustre: fix GLOBAL_INITIALISERS checkpatch.pl errors Mike Rapoport <mike.rapoport@gmail.com> - 2015-08-22 16:20 +0200
  [PATCH 4/5] staging: lustre: fix SWITCH_CASE_INDENT_LEVEL checkpatch.pl errors Mike Rapoport <mike.rapoport@gmail.com> - 2015-08-22 16:20 +0200
  Re: [PATCH 1/5] staging: lustre: fix whitespace errors reported by  checkpatch.pl Greg Kroah-Hartman <greg@kroah.com> - 2015-09-03 02:50 +0200
    Re: [PATCH 1/5] staging: lustre: fix whitespace errors reported by  checkpatch.pl Greg Kroah-Hartman <greg@kroah.com> - 2015-09-03 07:40 +0200

csiph-web