From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxx> --- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html