public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Staging : lustre : replaced seq_printf() with seq_puts and changed include statement
@ 2014-12-17 17:34 Hariharan Rangasamy
  2014-12-17 17:46 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Hariharan Rangasamy @ 2014-12-17 17:34 UTC (permalink / raw)
  To: oleg.drokin, andreas.dilger, gregkh
  Cc: devel, linux-kernel, Hariharan Rangasamy

Used seq_puts instead of seq_printf()
Used #include <linux/statfs.h> instead of #include <asm/statfs.h>

Signed-off-by: Hariharan Rangasamy <hariharanrangasamy@gmail.com>
---
 drivers/staging/lustre/lustre/osc/lproc_osc.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/lustre/lustre/osc/lproc_osc.c b/drivers/staging/lustre/lustre/osc/lproc_osc.c
index 9f719bc..d64a400 100644
--- a/drivers/staging/lustre/lustre/osc/lproc_osc.c
+++ b/drivers/staging/lustre/lustre/osc/lproc_osc.c
@@ -35,7 +35,7 @@
  */
 #define DEBUG_SUBSYSTEM S_CLASS
 
-#include <asm/statfs.h>
+#include <linux/statfs.h>
 #include "../include/obd_cksum.h"
 #include "../include/obd_class.h"
 #include "../include/lprocfs_status.h"
@@ -358,7 +358,7 @@ static int osc_checksum_type_seq_show(struct seq_file *m, void *v)
 		else
 			seq_printf(m, "%s ", cksum_name[i]);
 	}
-	seq_printf(m, "\n");
+	seq_puts(m, "\n");
 	return 0;
 }
 
@@ -590,7 +590,7 @@ static int osc_rpc_stats_seq_show(struct seq_file *seq, void *v)
 	seq_printf(seq, "pending read pages:   %d\n",
 		   atomic_read(&cli->cl_pending_r_pages));
 
-	seq_printf(seq, "\n\t\t\tread\t\t\twrite\n");
+	seq_puts(seq, "\n\t\t\tread\t\t\twrite\n");
 	seq_printf(seq, "pages per rpc	 rpcs   %% cum %% |");
 	seq_printf(seq, "       rpcs   %% cum %%\n");
 
@@ -613,7 +613,7 @@ static int osc_rpc_stats_seq_show(struct seq_file *seq, void *v)
 			break;
 	}
 
-	seq_printf(seq, "\n\t\t\tread\t\t\twrite\n");
+	seq_puts(seq, "\n\t\t\tread\t\t\twrite\n");
 	seq_printf(seq, "rpcs in flight	rpcs   %% cum %% |");
 	seq_printf(seq, "       rpcs   %% cum %%\n");
 
@@ -636,7 +636,7 @@ static int osc_rpc_stats_seq_show(struct seq_file *seq, void *v)
 			break;
 	}
 
-	seq_printf(seq, "\n\t\t\tread\t\t\twrite\n");
+	seq_puts(seq, "\n\t\t\tread\t\t\twrite\n");
 	seq_printf(seq, "offset		rpcs   %% cum %% |");
 	seq_printf(seq, "       rpcs   %% cum %%\n");
 
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] Staging : lustre : replaced seq_printf() with seq_puts and changed include statement
  2014-12-17 17:34 [PATCH] Staging : lustre : replaced seq_printf() with seq_puts and changed include statement Hariharan Rangasamy
@ 2014-12-17 17:46 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2014-12-17 17:46 UTC (permalink / raw)
  To: Hariharan Rangasamy; +Cc: oleg.drokin, andreas.dilger, devel, linux-kernel

On Wed, Dec 17, 2014 at 11:04:41PM +0530, Hariharan Rangasamy wrote:
> Used seq_puts instead of seq_printf()
> Used #include <linux/statfs.h> instead of #include <asm/statfs.h>

That's two different things, which means two different patches are
needed.  Please only do one-thing-per-patch.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-12-17 17:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-17 17:34 [PATCH] Staging : lustre : replaced seq_printf() with seq_puts and changed include statement Hariharan Rangasamy
2014-12-17 17:46 ` Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox