staging: i2o: Remove use of seq_printf return value
authorJoe Perches <joe@perches.com>
Sun, 22 Feb 2015 02:53:44 +0000 (18:53 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 7 Mar 2015 00:35:54 +0000 (16:35 -0800)
The seq_printf return value, because it's frequently misused,
will eventually be converted to void.

See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
     seq_has_overflowed() and make public")

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/i2o/i2o_proc.c

index 27d6b824aff84d186155d682ad89f147411d9dcb..780fee3224ead8beb1397c626d20de53955aeb87 100644 (file)
@@ -264,16 +264,22 @@ static int i2o_report_query_status(struct seq_file *seq, int block_status,
 {
        switch (block_status) {
        case -ETIMEDOUT:
-               return seq_printf(seq, "Timeout reading group %s.\n", group);
+               seq_printf(seq, "Timeout reading group %s.\n", group);
+               break;
        case -ENOMEM:
-               return seq_printf(seq, "No free memory to read the table.\n");
+               seq_puts(seq, "No free memory to read the table.\n");
+               break;
        case -I2O_PARAMS_STATUS_INVALID_GROUP_ID:
-               return seq_printf(seq, "Group %s not supported.\n", group);
+               seq_printf(seq, "Group %s not supported.\n", group);
+               break;
        default:
-               return seq_printf(seq,
-                                 "Error reading group %s. BlockStatus 0x%02X\n",
-                                 group, -block_status);
+               seq_printf(seq,
+                          "Error reading group %s. BlockStatus 0x%02X\n",
+                          group, -block_status);
+               break;
        }
+
+       return 0;
 }
 
 static char *bus_strings[] = {