From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Fri, 05 May 2017 08:50:28 +0000 Subject: [PATCH] IB/hfi1: Replace two seq_puts() calls by seq_putc() in sdma_seqfile_dump_cpu_list() Message-Id: <0bf8f61d-63c6-ba0a-df2f-00340b26afa7@users.sourceforge.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Dennis Dalessandro , Doug Ledford , Hal Rosenstock , Mike Marciniszyn , Sean Hefty Cc: LKML , kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org From: Markus Elfring 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 --- 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