* [PATCH] IB/hfi1: Replace two seq_puts() calls by seq_putc() in sdma_seqfile_dump_cpu_list()
@ 2017-05-05 8:50 SF Markus Elfring
0 siblings, 0 replies; only message in thread
From: SF Markus Elfring @ 2017-05-05 8:50 UTC (permalink / raw)
To: linux-rdma-u79uwXL29TY76Z2rM5mHXA, Dennis Dalessandro,
Doug Ledford, Hal Rosenstock, Mike Marciniszyn, Sean Hefty
Cc: LKML, kernel-janitors-u79uwXL29TY76Z2rM5mHXA
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Fri, 5 May 2017 10:40:03 +0200
Two single characters should be put into a sequence.
Thus use the corresponding function "seq_putc".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
drivers/infiniband/hw/hfi1/sdma.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/infiniband/hw/hfi1/sdma.c b/drivers/infiniband/hw/hfi1/sdma.c
index bfd0d5187e9b..7611a88b86cf 100644
--- a/drivers/infiniband/hw/hfi1/sdma.c
+++ b/drivers/infiniband/hw/hfi1/sdma.c
@@ -1130,7 +1130,7 @@ void sdma_seqfile_dump_cpu_list(struct seq_file *s,
continue;
if (j > 0)
- seq_puts(s, ",");
+ seq_putc(s, ',');
seq_printf(s, " sdma%2d",
rht_node->map[i]->sde[j]->this_idx);
@@ -1138,7 +1138,7 @@ void sdma_seqfile_dump_cpu_list(struct seq_file *s,
seq_puts(s, " ]");
}
- seq_puts(s, "\n");
+ seq_putc(s, '\n');
}
/*
--
2.12.2
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2017-05-05 8:50 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-05 8:50 [PATCH] IB/hfi1: Replace two seq_puts() calls by seq_putc() in sdma_seqfile_dump_cpu_list() SF Markus Elfring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox