staging: lustre: lustre: obdclass: lprocfs_status.c: Tidied up some code in a case...

Tidied up some code in a case statement after a discussion.

This was partly found using a static code analysis program called cppcheck.

Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c b/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
index 572ecada0f9353d0a8d5a207af8eb4a84e712bce..ddab94d7ee82649cf2764e6a66fa4b0b5f07523c 100644 (file)
--- a/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
+++ b/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
@@ -45,6 +45,7 @@
 #include "../include/lprocfs_status.h"
 #include "../include/lustre/lustre_idl.h"
 #include <linux/seq_file.h>
+#include <linux/ctype.h>
 
 static const char * const obd_connect_names[] = {
        "read_only",
@@ -1896,16 +1897,16 @@ int lprocfs_write_frac_u64_helper(const char *buffer, unsigned long count,
        }
 
        units = 1;
-       switch (*end) {
-       case 'p': case 'P':
+       switch (tolower(*end)) {
+       case 'p':
                units <<= 10;
-       case 't': case 'T':
+       case 't':
                units <<= 10;
-       case 'g': case 'G':
+       case 'g':
                units <<= 10;
-       case 'm': case 'M':
+       case 'm':
                units <<= 10;
-       case 'k': case 'K':
+       case 'k':
                units <<= 10;
        }
        /* Specified units override the multiplier */